Fix some warnings

diff --git a/yarrg/web/check_routestring b/yarrg/web/check_routestring
index 2cbe719cf4eec1afa16860f947da2b5044ee1490..e66cbfbfd35e855ef3a514806cf693426ceae24b 100755 (executable)
--- a/yarrg/web/check_routestring
+++ b/yarrg/web/check_routestring
@@ -89,7 +89,7 @@ foreach my $each (split m#[/|,]#, $string) {
        push @results, $results->[0];
 };
 
-$dbh->disconnect();
+$dbh->rollback();
 
 return  $emsg,
        (join ' | ', map { $_->[2] } @results),

diff --git a/yarrg/web/qtextstring b/yarrg/web/qtextstring
index 400716f119556e6bc0c893c82b5314ba6ac6206c..6efe368e650dfc396b020f5fa58f5679d3c86426 100644 (file)
--- a/yarrg/web/qtextstring
+++ b/yarrg/web/qtextstring
@@ -37,6 +37,10 @@
 $thingstring
 $qa => $m->caller_args(1)->{'qa'}
 </%args>
+<%perl>
+my $stringval= $qa->{$thingstring};
+$stringval='' if !defined $stringval;
+</%perl>
 
 Enter route (islands, or archipelagoes, separated by |s or commas;
  abbreviations are OK):<br>
@@ -86,7 +90,7 @@ window.onload= ts_Needed;
 <input type="text" <% $m->content %>
  id="<% $thingstring %>" name="<% $thingstring %>"
  onchange="ts_Needed();" onkeyup="ts_Later();"
- value="<% $qa->{$thingstring} |h %>"
+ value="<% $stringval |h %>"
  >
 <br>
 <div id="ts_results">&nbsp;</div><br>

diff --git a/yarrg/web/query_age b/yarrg/web/query_age
index c5009d3bac56498fb0ae9cc5ac78482aa791af40..922e29f0c90031b97e1c485bb6fcb456946f8018 100644 (file)
--- a/yarrg/web/query_age
+++ b/yarrg/web/query_age
@@ -130,3 +130,6 @@ use POSIX;
 use CommodsWeb;
 my $dbh= dbw_connect('Midnight');
 </%init>
+<%cleanup>
+$dbh->rollback();
+</%cleanup>

diff --git a/yarrg/web/query_route b/yarrg/web/query_route
index dc1ebc9917608eaf7a596e7d3eaf95e58165676a..ea94d61c8f20ddf3428f2d873232c4e2f1d27557 100644 (file)
--- a/yarrg/web/query_route
+++ b/yarrg/web/query_route
@@ -108,6 +108,8 @@ foreach my $arch (keys %islandlistdata) {
                $optionlistmap->($islandlistdata{$arch}, '');
 }
 
+$dbh->rollback();
+
 </%perl>
 
 <input type=hidden name=dropdowns value="<% $qa->{Dropdowns} |h %>">