X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~yarrgweb/git?p=ypp-sc-tools.db-live.git;a=blobdiff_plain;f=yarrg%2Fweb%2Fquery_route;h=393e7a668755145d05262a56b56ed005330669c1;hp=8f6d99c49d29ed353f5f2fcdf6d88dc55180afac;hb=59bee7afb77216585b904bd20f17e71005e9778c;hpb=9d01242d0991d15f7ea84454264c868e1c4ed8ad

diff --git a/yarrg/web/query_route b/yarrg/web/query_route
index 8f6d99c..393e7a6 100644
--- a/yarrg/web/query_route
+++ b/yarrg/web/query_route
@@ -35,35 +35,64 @@
 </%doc>
 <%args>
 $quri
+$dbh
+$prselector
 $routestring => '';
+$someresults
+$emsgokorprint
 </%args>
-<%perl>
 
+<%perl>
+my $emsg;
 my @archipelagoes;
 my @islandids;
 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' };
 
-%#---------- textbox, user enters route as string ----------
-% if (!$qa->{Dropdowns}) {
+</%perl>
 
 <h1>Specify route</h1>
 
+% $prselector->('ShowStalls');
+
+%#---------- textbox, user enters route as string ----------
+% if (!$qa->{Dropdowns}) {
+
 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, thingstring => 'routestring' &>
+<&| qtextstring, qa => $qa, dbh => $dbh,
+    thingstring => 'routestring', emsgstore => \$emsg,
+    perresult => sub {
+	my ($canonname, $island, $arch) = @_;
+	push @islandids, $island;
+	push @archipelagoes, defined $island ? undef : $arch;
+    }
+ &>
  size=80
 </&>
 
 % } else { #---------- dropdowns, user selects from menus ----------
 
+% $startform->(grep {
+%		defined $ARGS{"archipelago$_"} ||
+%		defined $ARGS{"islandid$_"}
+%	} (0..$qa->{Dropdowns}-1));
+
 <%perl>
-my ($sth,$row);;
+my ($sth,$row);
 my @archlistdata;
 my %islandlistdata;
 $islandlistdata{'none'}= [ [ "none", "Select island..." ] ];
@@ -81,8 +110,6 @@ my $optionlistmap= sub {
 	return $out;
 };
 
-my $dbh= dbw_connect($qa->{Ocean});
-
 $sth= $dbh->prepare("SELECT DISTINCT archipelago FROM islands
 			    ORDER BY archipelago;");
 $sth->execute();
@@ -111,8 +138,6 @@ foreach my $arch (keys %islandlistdata) {
 		$optionlistmap->($islandlistdata{$arch}, '');
 }
 
-$dbh->rollback();
-
 </%perl>
 
 <&| script &>
@@ -155,42 +180,14 @@ function ms_Setarch(dd) {
 
 % } #---------- end of dropdowns, now common middle of page code ----------
 
-<input type=submit name=submit value="Go">
-% my $ours= sub { $_[0] =~ m/^island|^archipelago|^routestring/; };
+<input type=submit name=submit value="<% $goupdate->() %>">
+% my $ours= sub { $_[0] =~ m/^island|^archipelago|^routestring|^[RT]/; };
 <& "lookup:formhidden", ours => $ours &>
-</form>
 
 <%perl>
-#========== result computations ==========
-
-my $results_head;
-$results_head= sub {
-	print "<h1>Results</h1>\n";
-	$results_head= sub { };
-};
-
-#---------- result computation - textstring ----------
-if (!$qa->{Dropdowns}) {
-  if (length $routestring) {
-	$results_head->();
-	my ($emsg,$canonstring,$results)= $m->comp('qtextstringcheck',
-		what => 'routestring',
-		ocean => $qa->{Ocean},
-		string => $routestring,
-		format => 'return'
-	);
-	if (length $emsg) {
-		print encode_entities($emsg);
-	} else {
-		foreach my $entry (@$results) {
-			push @islandids, $entry->[1];
-			push @archipelagoes,
-				defined $entry->[1] ? undef : $entry->[2];
-		} 
-	}
-  }
+#========== results ==========
 
-} else { #---------- results - dropdowns ----------
+$emsgokorprint->($emsg) or @islandids=();
 
 my $argorundef= sub {
 	my ($dd,$base) = @_;
@@ -207,7 +204,7 @@ for my $dd (0..$qa->{Dropdowns}-1) {
 		my $ii= $islandid2{$island};
 		my $iarch= $ii->{Arch};
 		if ($iarch ne $arch) {
-			$results_head->();
+			$someresults->();
 </%perl>
  Specified archipelago <% $arch %> but
  island <% $ii->{Name} %>
@@ -220,12 +217,15 @@ for my $dd (0..$qa->{Dropdowns}-1) {
 	push @islandids, $island;
 }
 
-}#---------- result processing, common stuff
 </%perl>
 
 % if (@islandids) {
-%	$results_head->();
-
-<& routetrade, islandids => \@islandids, archipelagoes => \@archipelagoes &>
-
+%	$someresults->('Relevant trades');
+<& routetrade,
+   dbh => $dbh,
+   islandids => \@islandids,
+   archipelagoes => \@archipelagoes,
+   qa => $qa
+ &>
+</form>
 % }