chiark / gitweb /
Break out generation of commodity and route form entry boxes into enter_* components
authorIan Jackson <ian@liberator.relativity.greenend.org.uk>
Sun, 20 Sep 2009 18:59:51 +0000 (19:59 +0100)
committerIan Jackson <ian@liberator.relativity.greenend.org.uk>
Sun, 20 Sep 2009 18:59:51 +0000 (19:59 +0100)
yarrg/web/enter_commod [new file with mode: 0644]
yarrg/web/enter_route [new file with mode: 0644]
yarrg/web/qtextstringcheck
yarrg/web/query_commod
yarrg/web/query_route

diff --git a/yarrg/web/enter_commod b/yarrg/web/enter_commod
new file mode 100644 (file)
index 0000000..2ccc9ab
--- /dev/null
@@ -0,0 +1,72 @@
+<%doc>
+
+ This is part of the YARRG website.  YARRG is a tool and website
+ for assisting players of Yohoho Puzzle Pirates.
+
+ Copyright (C) 2009 Ian Jackson <ijackson@chiark.greenend.org.uk>
+ Copyright (C) 2009 Clare Boothby
+
+  YARRG's client code etc. is covered by the ordinary GNU GPL (v3 or later).
+  The YARRG website is covered by the GNU Affero GPL v3 or later, which
+   basically means that every installation of the website will let you
+   download the source.
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU Affero General Public License as
+ published by the Free Software Foundation, either version 3 of the
+ License, or (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU Affero General Public License for more details.
+
+ You should have received a copy of the GNU Affero General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+ Yohoho and Puzzle Pirates are probably trademarks of Three Rings and
+ are used without permission.  This program is not endorsed or
+ sponsored by Three Rings.
+
+
+ This Mason component generates form contents for selecting a commodity.
+
+
+</%doc>
+<%args>
+$qa
+$dbh
+$emsg_r
+
+$commodname_r
+$cmid_r
+</%args>
+
+%#---------- textbox, user enters commodity as string ----------
+% if (!$qa->{Dropdowns}) {
+
+Enter commodity (abbreviations are OK):<br>
+
+<&| qtextstring, qa => $qa, dbh => $dbh,
+    thingstring => 'commodstring', emsgstore => $emsg_r,
+    onresults => sub { ($$commodname_r,$$cmid_r)= @{ $_[0] } if @_ }
+ &>
+ size=80
+</&>
+
+% } else { #---------- dropdowns, user selects from menus ----------
+
+%      my $sth= $dbh->prepare("SELECT commodname,commodid FROM commods
+%                                      ORDER BY commodname");
+%      $sth->execute();
+%      my $row;
+<select name="commodid">
+<option value="">Select commodity...</option>
+%      while ($row= $sth->fetchrow_arrayref) {
+%              my $selected= $qa->{'commodid'} eq $row->[1] ? 'selected' : '';
+<option value="<% $row->[1] %>" <% $selected %>><% $row->[0] |h %></option>
+%              ($$commodname_r,$$cmid_r) = @$row if $selected;
+%      }
+</select>
+
+% } #---------- end of dropdowns, now common middle of page code ----------
diff --git a/yarrg/web/enter_route b/yarrg/web/enter_route
new file mode 100644 (file)
index 0000000..58b29f4
--- /dev/null
@@ -0,0 +1,190 @@
+<%doc>
+
+ This is part of the YARRG website.  YARRG is a tool and website
+ for assisting players of Yohoho Puzzle Pirates.
+
+ Copyright (C) 2009 Ian Jackson <ijackson@chiark.greenend.org.uk>
+ Copyright (C) 2009 Clare Boothby
+
+  YARRG's client code etc. is covered by the ordinary GNU GPL (v3 or later).
+  The YARRG website is covered by the GNU Affero GPL v3 or later, which
+   basically means that every installation of the website will let you
+   download the source.
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU Affero General Public License as
+ published by the Free Software Foundation, either version 3 of the
+ License, or (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU Affero General Public License for more details.
+
+ You should have received a copy of the GNU Affero General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+ Yohoho and Puzzle Pirates are probably trademarks of Three Rings and
+ are used without permission.  This program is not endorsed or
+ sponsored by Three Rings.
+
+
+ This Mason component generates form contents for selecting a list
+ of locations (eg, a route).
+
+
+</%doc>
+<%args>
+$qa
+$dbh
+$emsg_r
+$warningfs_r
+
+$enterwhat
+$islandids_r
+$archipelagoes_r
+</%args>
+
+%#---------- textbox, user enters route as string ----------
+% if (!$qa->{Dropdowns}) {
+
+<% $enterwhat %> (islands, or archipelagoes, separated by |s or commas;
+ abbreviations are OK):<br>
+
+<&| qtextstring, qa => $qa, dbh => $dbh,
+    thingstring => 'routestring', emsgstore => $emsg_r,
+    onresults => sub {
+       foreach (@_) {
+       my ($canonname, $island, $arch) = @$_;
+               push @$islandids_r, $island;
+               push @$archipelagoes_r, defined $island ? undef : $arch;
+       }
+    }
+ &>
+ size=80
+</&>
+
+% } else { #---------- dropdowns, user selects from menus ----------
+
+<%perl>
+my %islandid2;
+my ($sth,$row);
+my @archlistdata;
+my %islandlistdata;
+$islandlistdata{'none'}= [ [ "none", "Select island..." ] ];
+
+my $optionlistmap= sub {
+       my ($optlist, $selected) = @_;
+       my $out='';
+       foreach my $entry (@$optlist) {
+               $out.= sprintf('<option value="%s" %s>%s</option>',
+                       encode_entities($entry->[0]),
+                       defined $selected && $entry->[0] eq $selected
+                               ? 'selected' : '',
+                       encode_entities($entry->[1]));
+       }
+       return $out;
+};
+
+$sth= $dbh->prepare("SELECT DISTINCT archipelago FROM islands
+                           ORDER BY archipelago;");
+$sth->execute();
+
+while ($row=$sth->fetchrow_arrayref) {
+       my ($arch)= @$row;
+       push @archlistdata, [ $arch, $arch ];
+       $islandlistdata{$arch}= [ [ "none", "Whole arch" ] ];
+}
+
+$sth= $dbh->prepare("SELECT islandid,islandname,archipelago
+                            FROM islands
+                           ORDER BY islandname;");
+$sth->execute();
+
+while ($row=$sth->fetchrow_arrayref) {
+       my $arch= $row->[2];
+       push @{ $islandlistdata{'none'} }, [ @$row ];
+       push @{ $islandlistdata{$arch} }, [ @$row ];
+       $islandid2{$row->[0]}= { Name => $row->[1], Arch => $arch };
+}
+
+my %resetislandlistdata;
+foreach my $arch (keys %islandlistdata) {
+       $resetislandlistdata{$arch}=
+               $optionlistmap->($islandlistdata{$arch}, '');
+}
+
+</%perl>
+
+<&| script &>
+ms_lists= <% to_json_protecttags(\%resetislandlistdata) %>;
+function ms_Setarch(dd) {
+  debug('ms_SetArch '+dd+' arch='+arch);
+  var arch= document.getElementsByName('archipelago'+dd).item(0).value;
+  var got= ms_lists[arch];
+  if (got == undefined) return; // unknown arch ?  hrm
+  debug('ms_SetArch '+dd+' arch='+arch+' got ok');
+  var select= document.getElementsByName('islandid'+dd).item(0);
+  select.innerHTML= got;
+  debug('ms_SetArch '+dd+' arch='+arch+' innerHTML set');
+}
+</&script>
+
+<table style="table-layout:fixed; width:90%;">
+
+<tr>
+%      for my $dd (0..$qa->{Dropdowns}-1) {
+<td>
+<select name="archipelago<% $dd %>" onchange="ms_Setarch(<% $dd %>)">
+<option value="none">Whole ocean</option>
+<% $optionlistmap->(\@archlistdata, $qa->{"archipelago$dd"}) %></select></td>
+%      }
+</tr>
+
+<tr>
+%      for my $dd (0..$qa->{Dropdowns}-1) {
+%              my $arch= $qa->{"archipelago$dd"};
+%              $arch= 'none' if !defined $arch;
+<td>
+<select name="islandid<% $dd %>">
+<% $optionlistmap->($islandlistdata{$arch}, $qa->{"islandid$dd"}) %>
+</select></td>
+%      }
+</tr>
+
+</table>
+
+<%perl>
+
+my $argorundef= sub {
+       my ($dd,$base) = @_;
+       my $thing= $qa->{"${base}${dd}"};
+       $thing= undef if defined $thing and $thing eq 'none';
+       return $thing;
+};
+
+for my $dd (0..$qa->{Dropdowns}-1) {
+       my $arch= $argorundef->($dd,'archipelago');
+       my $island= $argorundef->($dd,'islandid');
+       next unless defined $arch or defined $island;
+       if (defined $island and defined $arch) {
+               my $ii= $islandid2{$island};
+               my $iarch= $ii->{Arch};
+               if ($iarch ne $arch) {
+                       push @$warningfs_r, sub {
+</%perl>
+ Specified archipelago <% $arch %> but
+ island <% $ii->{Name} %>
+ which is in <% $iarch %>; using the island.<p>
+<%perl>
+                       };
+               }
+               $arch= undef;
+       }
+       push @$archipelagoes_r, $arch;
+       push @$islandids_r, $island;
+}
+
+</%perl>
+
+% }
index 4adb606ea0b2c3e25dc8251dac335a47ee24e7e6..3b4fdd7470443e040a09882a1e66eba621c3ac7e 100755 (executable)
@@ -95,7 +95,7 @@ if ($chk->method_exists('execute')) {
        my $sqlstmt_nqs= @sqlstmt_nqs;
 
        my @specs= $chk->attr('multiple')
-               ? (split m#[/|,]#, $string)
+               ? (split m#\s*[/|,]\s*#, $string)
                : ($string);
 
        foreach my $each (@specs) {
index 81e8208d1b34abd3fda1bb38e0181eb29949762c..5db6330c6cb2df4999c7c4bac682a301745be79d 100644 (file)
@@ -57,34 +57,10 @@ my $qa= \%ARGS;
 
 <form action="<% $quri->() |h %>" method="get">
 
-%#---------- textbox, user enters route as string ----------
-% if (!$qa->{Dropdowns}) {
-
-Enter commodity (abbreviations are OK):<br>
-
-<&| qtextstring, qa => $qa, dbh => $dbh,
-    thingstring => 'commodstring', emsgstore => \$emsg,
-    onresults => sub { ($commodname,$cmid)= @{ $_[0] }; }
+<& enter_commod, qa => $qa, dbh => $dbh, emsg_r => \$emsg,
+       commodname_r => \$commodname,
+       cmid_r => \$cmid
  &>
- size=80
-</&>
-
-% } else { #---------- dropdowns, user selects from menus ----------
-
-%      my $sth= $dbh->prepare("SELECT commodname,commodid FROM commods
-%                                      ORDER BY commodname");
-%      $sth->execute();
-%      my $row;
-<select name="commodid">
-<option value="">Select commodity...</option>
-%      while ($row= $sth->fetchrow_arrayref) {
-%              my $selected= $commodid eq $row->[1] ? 'selected' : '';
-<option value="<% $row->[1] %>" <% $selected %>><% $row->[0] |h %></option>
-%              ($commodname,$cmid) = @$row if $selected;
-%      }
-</select>
-
-% } #---------- end of dropdowns, now common middle of page code ----------
 
 <input type=submit name=submit value="Go">
 % my $ours= sub { $_[0] =~ m/^commodstring|^commodid/; };
index 8b491d93a99aca47c41b8a77306648c10975acf1..26b83a58b1308f3a595712ff76d1918bbee46146 100644 (file)
@@ -47,9 +47,9 @@ $emsgokorprint
 
 <%perl>
 my $emsg;
+my @warningfs;
 my @archipelagoes;
 my @islandids;
-my %islandid2;
 my ($max_volume, $max_mass);
 my $lossperleaguepct;
 my $capital;
@@ -69,28 +69,28 @@ my $goupdate= sub { $be_post ? 'Update' : 'Go' };
 
 <h1>Specify route</h1>
 
-% $prselector->('ShowStalls');
-
-%#---------- textbox, user enters route as string ----------
+% # Sadly we need to do this rather hacky thing to make it be a POST
+% #  form if the user has already selected some thing(s)
 % if (!$qa->{Dropdowns}) {
+%     $startform->($routestring =~ m/\S/);
+% } else {
+%     $startform->(grep {
+%              defined $qa->{"archipelago$_"} ||
+%              defined $qa->{"islandid$_"}
+%      } (0..$qa->{Dropdowns}-1));
+% }
 
-Enter route (islands, or archipelagoes, separated by |s or commas;
- abbreviations are OK):<br>
+% $prselector->('ShowStalls');
 
-% $startform->($routestring =~ m/\S/);
+<& enter_route, qa => $qa, dbh => $dbh, emsg_r => \$emsg,
+       warningfs_r => \@warningfs,
+       enterwhat => 'Enter route',
+       islandids_r => \@islandids,
+       archipelagoes_r => \@archipelagoes
+&>
 
-<&| qtextstring, qa => $qa, dbh => $dbh,
-    thingstring => 'routestring', emsgstore => \$emsg,
-    onresults => sub {
-       foreach (@_) {
-       my ($canonname, $island, $arch) = @$_;
-               push @islandids, $island;
-               push @archipelagoes, defined $island ? undef : $arch;
-       }
-    }
- &>
- size=80
-</&>
+%#---------- textboxes, user enters details as strings ----------
+% if (!$qa->{Dropdowns}) {
 
 <strong>Advanced options - you may leave these blank:</strong>
 <p>
@@ -137,100 +137,6 @@ Expected losses:
 </tr>
 </table>
 
-% } else { #---------- dropdowns, user selects from menus ----------
-
-% $startform->(grep {
-%              defined $ARGS{"archipelago$_"} ||
-%              defined $ARGS{"islandid$_"}
-%      } (0..$qa->{Dropdowns}-1));
-
-<%perl>
-my ($sth,$row);
-my @archlistdata;
-my %islandlistdata;
-$islandlistdata{'none'}= [ [ "none", "Select island..." ] ];
-
-my $optionlistmap= sub {
-       my ($optlist, $selected) = @_;
-       my $out='';
-       foreach my $entry (@$optlist) {
-               $out.= sprintf('<option value="%s" %s>%s</option>',
-                       encode_entities($entry->[0]),
-                       defined $selected && $entry->[0] eq $selected
-                               ? 'selected' : '',
-                       encode_entities($entry->[1]));
-       }
-       return $out;
-};
-
-$sth= $dbh->prepare("SELECT DISTINCT archipelago FROM islands
-                           ORDER BY archipelago;");
-$sth->execute();
-
-while ($row=$sth->fetchrow_arrayref) {
-       my ($arch)= @$row;
-       push @archlistdata, [ $arch, $arch ];
-       $islandlistdata{$arch}= [ [ "none", "Whole arch" ] ];
-}
-
-$sth= $dbh->prepare("SELECT islandid,islandname,archipelago
-                            FROM islands
-                           ORDER BY islandname;");
-$sth->execute();
-
-while ($row=$sth->fetchrow_arrayref) {
-       my $arch= $row->[2];
-       push @{ $islandlistdata{'none'} }, [ @$row ];
-       push @{ $islandlistdata{$arch} }, [ @$row ];
-       $islandid2{$row->[0]}= { Name => $row->[1], Arch => $arch };
-}
-
-my %resetislandlistdata;
-foreach my $arch (keys %islandlistdata) {
-       $resetislandlistdata{$arch}=
-               $optionlistmap->($islandlistdata{$arch}, '');
-}
-
-</%perl>
-
-<&| script &>
-ms_lists= <% to_json_protecttags(\%resetislandlistdata) %>;
-function ms_Setarch(dd) {
-  debug('ms_SetArch '+dd+' arch='+arch);
-  var arch= document.getElementsByName('archipelago'+dd).item(0).value;
-  var got= ms_lists[arch];
-  if (got == undefined) return; // unknown arch ?  hrm
-  debug('ms_SetArch '+dd+' arch='+arch+' got ok');
-  var select= document.getElementsByName('islandid'+dd).item(0);
-  select.innerHTML= got;
-  debug('ms_SetArch '+dd+' arch='+arch+' innerHTML set');
-}
-</&script>
-
-<table style="table-layout:fixed; width:90%;">
-
-<tr>
-%      for my $dd (0..$qa->{Dropdowns}-1) {
-<td>
-<select name="archipelago<% $dd %>" onchange="ms_Setarch(<% $dd %>)">
-<option value="none">Whole ocean</option>
-<% $optionlistmap->(\@archlistdata, $ARGS{"archipelago$dd"}) %></select></td>
-%      }
-</tr>
-
-<tr>
-%      for my $dd (0..$qa->{Dropdowns}-1) {
-%              my $arch= $ARGS{"archipelago$dd"};
-%              $arch= 'none' if !defined $arch;
-<td>
-<select name="islandid<% $dd %>">
-<% $optionlistmap->($islandlistdata{$arch}, $ARGS{"islandid$dd"}) %>
-</select></td>
-%      }
-</tr>
-
-</table>
-
 % } #---------- end of dropdowns, now common middle of page code ----------
 
 <input type=submit name=submit value="<% $goupdate->() %>">
@@ -244,32 +150,9 @@ function ms_Setarch(dd) {
 
 $emsgokorprint->($emsg) or @islandids=();
 
-my $argorundef= sub {
-       my ($dd,$base) = @_;
-       my $thing= $ARGS{"${base}${dd}"};
-       $thing= undef if defined $thing and $thing eq 'none';
-       return $thing;
-};
-
-for my $dd (0..$qa->{Dropdowns}-1) {
-       my $arch= $argorundef->($dd,'archipelago');
-       my $island= $argorundef->($dd,'islandid');
-       next unless defined $arch or defined $island;
-       if (defined $island and defined $arch) {
-               my $ii= $islandid2{$island};
-               my $iarch= $ii->{Arch};
-               if ($iarch ne $arch) {
-                       $someresults->();
-</%perl>
- Specified archipelago <% $arch %> but
- island <% $ii->{Name} %>
- which is in <% $iarch %>; using the island.<br>
-<%perl>
-               }
-               $arch= undef;
-       }
-       push @archipelagoes, $arch;
-       push @islandids, $island;
+foreach my $warningf (@warningfs) {
+       $someresults->();
+       $warningf->();
 }
 
 </%perl>