Use POST for routetrade update to avoid overly long URLs due to many tickyboxes

[ypp-sc-tools.db-test.git] / yarrg / web / query_route

diff --git a/yarrg/web/query_route b/yarrg/web/query_route
index be1196e899ab3fa8e99c8c9b975a59b4ff6ebc10..ee457f57abd41502ad3d54f58385b16638e8ced5 100644 (file)
--- a/yarrg/web/query_route
+++ b/yarrg/web/query_route
@@ -49,6 +49,16 @@ my @islandids;
 my %islandid2;
 
 my $qa= \%ARGS;
 my %islandid2;
 
 my $qa= \%ARGS;

+
+my $be_post;
+my $startform= sub {
+       ($be_post)= @_;
+</%perl>
+<form action="<% $quri->() |h %>" method="<% $be_post ? 'post' : 'get' %>">
+<%perl>
+};
+my $goupdate= sub { $be_post ? 'Update' : 'Go' };
+

 </%perl>
 
 <h1>Specify route</h1>
 </%perl>
 
 <h1>Specify route</h1>


@@ -61,7 +71,7 @@ my $qa= \%ARGS;
 Enter route (islands, or archipelagoes, separated by |s or commas;
  abbreviations are OK):<br>
 
 Enter route (islands, or archipelagoes, separated by |s or commas;
  abbreviations are OK):<br>
 

-<form action="<% $quri->() |h %>" method="get">
+% $startform->($routestring =~ m/\S/);

 
 <&| qtextstring, qa => $qa, dbh => $dbh,
     thingstring => 'routestring', emsgstore => \$emsg,
 
 <&| qtextstring, qa => $qa, dbh => $dbh,
     thingstring => 'routestring', emsgstore => \$emsg,


@@ -76,6 +86,11 @@ Enter route (islands, or archipelagoes, separated by |s or commas;
 
 % } else { #---------- dropdowns, user selects from menus ----------
 
 
 % } else { #---------- dropdowns, user selects from menus ----------
 

+$startform->(grep {
+               defined %ARGS{"archipelago$dd"} ||
+               defined %ARGS{"islandid$dd"}
+       } (0..$qa->{Dropdowns}-1));
+

 <%perl>
 my ($sth,$row);
 my @archlistdata;
 <%perl>
 my ($sth,$row);
 my @archlistdata;


@@ -165,7 +180,7 @@ function ms_Setarch(dd) {
 
 % } #---------- end of dropdowns, now common middle of page code ----------
 
 
 % } #---------- end of dropdowns, now common middle of page code ----------
 

-<input type=submit name=submit value="Go">
+<input type=submit name=submit value="<% $goupdate->() %>">

 % my $ours= sub { $_[0] =~ m/^island|^archipelago|^routestring|^[RT]/; };
 <& "lookup:formhidden", ours => $ours &>
 
 % my $ours= sub { $_[0] =~ m/^island|^archipelago|^routestring|^[RT]/; };
 <& "lookup:formhidden", ours => $ours &>