OpenPKG CVS Repository
  http://cvs.openpkg.org/
  ____________________________________________________________________________

  Server: cvs.openpkg.org                  Name:   Thomas Lotterer
  Root:   /v/openpkg/cvs                   Email:  [EMAIL PROTECTED]
  Module: openpkg-registry                 Date:   27-Nov-2005 00:31:49
  Branch: HEAD                             Handle: 2005112623314800

  Modified files:
    openpkg-registry        registry-ui.pl

  Log:
    switch page handling from hidden form fields to url params

  Summary:
    Revision    Changes     Path
    1.27        +22 -40     openpkg-registry/registry-ui.pl
  ____________________________________________________________________________

  patch -p0 <<'@@ .'
  Index: openpkg-registry/registry-ui.pl
  ============================================================================
  $ cvs diff -u -r1.26 -r1.27 registry-ui.pl
  --- openpkg-registry/registry-ui.pl   26 Nov 2005 20:46:01 -0000      1.26
  +++ openpkg-registry/registry-ui.pl   26 Nov 2005 23:31:48 -0000      1.27
  @@ -71,6 +71,7 @@
   
$cfg->{canvas}->{url}="http://meta.openpkg.org/?path=project.registry.manage";;
   $cfg->{canvas}->{mark_head}="<!-- CANVAS: HEAD -->";
   $cfg->{canvas}->{mark_body}="<!-- CANVAS: BODY -->";
  +$cfg->{page}->{default} = undef;
   
   my $ase;
   $ase = undef;
  @@ -82,6 +83,9 @@
   #   create objects
   my $cgi  = new CGI;
   my $myurl = $cgi->url(-relative => 1) || ".";
  +my $requestedpage = $cgi->url_param("page");
  +$requestedpage = $cfg->{page}->{default} if (not defined $requestedpage);
  +$cgi->delete(-name=>'page');
   
   #   database handle and scratch variables
   #
  @@ -128,21 +132,21 @@
   
   #   first check for pages which do not require database access
   #
  -if    ($cgi->param("page") eq "css") {
  +if    ($requestedpage eq "css") {
       my $out;
       $out = '';
       $out .= &viewhttp('text/css', '+10s');
       $out .= &viewcss();
       print STDOUT $out;
   }
  -elsif ($cgi->param("page") eq "jpg") {
  +elsif ($requestedpage eq "jpg") {
       my $out;
       $out = '';
       $out .= &viewhttp('image/jpg', '+600s');
       $out .= &viewjpg($cgi->param("name"));
       print STDOUT $out;
   }
  -elsif ($cgi->param("page") eq "gif") {
  +elsif ($requestedpage eq "gif") {
       my $out;
       $out = '';
       $out .= &viewhttp('image/gif', '+600s');
  @@ -209,25 +213,25 @@
   
   #   continue to the pages that require database access
   #
  -if    ($cgi->param("page") eq "login") {
  +if    ($requestedpage eq "login") {
       my $out;
       $out = '';
       $out .= &viewlogin();
       print STDOUT &canvas($out);
   }
  -elsif ($cgi->param("page") eq "logout") {
  +elsif ($requestedpage eq "logout") {
       my $out;
       $out = '';
       $out .= &viewlogout();
       print STDOUT &canvas($out);
   }
  -elsif ($cgi->param("page") eq "asecomeback") {
  +elsif ($requestedpage eq "asecomeback") {
       my $out;
       $out = '';
       $out .= &viewasecomeback();
       print STDOUT &canvas($out);
   }
  -elsif ($cgi->param("page") eq "association") {
  +elsif ($requestedpage eq "association") {
       my $out;
       $out = '';
       $out .= viewhttpauthrequired("association");
  @@ -236,7 +240,7 @@
       $out .= &viewhtmltail();
       print STDOUT &canvas($out);
   }
  -elsif (($cgi->param("page") eq "dropxml" or $cgi->param("page") eq "" ) and 
not &uao()) {
  +elsif ($requestedpage eq "dropxml" and not &uao()) {
       my $out;
       $out = '';
       $out .= &viewhttp();
  @@ -245,14 +249,14 @@
       $out .= &viewhtmltail();
       print STDOUT &canvas($out);
   }
  -elsif ($cgi->param("page") eq "dropxml" and &uao()) {
  +elsif ($requestedpage eq "dropxml" and &uao()) {
       my $out;
       $out = '';
       $out .= &viewhttp("plain/text");
       $out .= &viewdropxml();
       print STDOUT $out;
   }
  -elsif ($cgi->param("page") eq "ase") {
  +elsif ($requestedpage eq "ase") {
       my $out;
       $out = '';
       $out .= &viewhttp();
  @@ -911,9 +915,7 @@
       my ($html);
       $html = '';
       $html .= '<td>';
  -    $html .= $cgi->start_form();
  -    $cgi->delete(-name=>'page');
  -    $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "association") 
. "</div>";
  +    $html .= $cgi->start_form(-action => "$myurl?page=association");
       $html .= "<div>" . $cgi->submit('submit','association') . "</div>";
       $html .= $cgi->end_form;
       $html .= '</td>';
  @@ -926,12 +928,7 @@
       $html = '';
       $html .= '<td>';
       if    ($cfg->{identification}->{mode} eq "ase") {
  -        $html .= $cgi->start_form(-action => $ase->url(-action => "login"));
  -        $cgi->delete(-name=>'page');
  -        $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "ase") . 
"</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-action',      -value => 
"login") . "</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-mode_during', -value => 
"ase") . "</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-mode_after',  -value => 
"asecomeback") . "</div>";
  +        $html .= $cgi->start_form(-action => $ase->url(-action => "login", 
-mode_during => "ase", -mode_after => "asecomeback"));
           $html .= "<div>" . $cgi->submit('submit','login') . "</div>";
           $html .= $cgi->end_form;
       }
  @@ -939,9 +936,7 @@
           # nop
       }
       elsif ($cfg->{identification}->{mode} eq "naive") {
  -        $html .= $cgi->start_form();
  -        $cgi->delete(-name=>'page');
  -        $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "login") . 
"</div>";
  +        $html .= $cgi->start_form(-action => "$myurl?page=login");
           $html .= "<div>" . $cgi->submit('submit','login') . "</div>";
           $html .= $cgi->textfield(
               -name      => 'username',
  @@ -1077,12 +1072,7 @@
       $html = '';
       $html .= '<td>';
       if    ($cfg->{identification}->{mode} eq "ase") {
  -        $html .= $cgi->start_form(-action => $ase->url(-action => "logout"));
  -        $cgi->delete(-name=>'page');
  -        $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "ase") . 
"</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-action',      -value => 
"logout") . "</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-mode_during', -value => 
"ase") . "</div>";
  -        $html .= "<div>" . $cgi->hidden(-name=>'ase-mode_after',  -value => 
"dropxml") . "</div>";
  +        $html .= $cgi->start_form(-action => $ase->url(-action => "logout", 
-mode_during => "ase", -mode_after => "asecomeback"));
           $html .= "<div>" . $cgi->submit('submit','logout') . "</div>";
           $html .= $cgi->end_form;
       }
  @@ -1090,9 +1080,7 @@
           # nop
       }
       elsif ($cfg->{identification}->{mode} eq "naive") {
  -        $html .= $cgi->start_form();
  -        $cgi->delete(-name=>'page');
  -        $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "logout") . 
"</div>";
  +        $html .= $cgi->start_form(-action => "$myurl?page=logout");
           $html .= "<div>" . $cgi->submit('submit','logout') . "</div>";
           $html .= $cgi->end_form;
       }
  @@ -1132,9 +1120,7 @@
       my $html;
       $html = '';
       $html .= '<td>';
  -    $html .= $cgi->start_form();
  -    $cgi->delete(-name=>'page');
  -    $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "dropxml") . 
"</div>";
  +    $html .= $cgi->start_form(-action => "$myurl?page=dropxml");
       $html .= "<div>" . $cgi->submit('submit','dropxml') . "</div>";
       $html .= $cgi->end_form;
       $html .= '</td>';
  @@ -1271,7 +1257,7 @@
       $html .= "<table class=\"association\">\n";
       $html .= "<tr>";
       $html .= "<td colspan=\"4\">";
  -    $html .= $cgi->start_form();
  +    $html .= $cgi->start_form(-action => "$myurl?page=association");
       $html .= "<h3>$headline</h3>";
       $html .= "</td>";
       $html .= "</tr>\n";
  @@ -1339,8 +1325,6 @@
           $html .= "<td colspan=\"2\">";
           $cgi->delete(-name=>'username');
           $html .= "<div>" . $cgi->hidden(-name=>'username', -value => 
$username) . "</div>";
  -        $cgi->delete(-name=>'page');
  -        $html .= "<div>" . $cgi->hidden(-name=>'page', -value => 
"association") . "</div>";
           $html .= "<div>" . $cgi->submit("$mode/delete",'delete') . "</div>";
           $html .= "</td>";
           $html .= "<td colspan=\"2\">";
  @@ -1603,9 +1587,7 @@
       $html .= "      <table class=\"menu\">\n";
       $html .= "          <tr>\n";
       $html .= "              <td>\n";
  -    $html .= $cgi->start_form();
  -    $cgi->delete(-name=>'page');
  -    $html .= "<div>" . $cgi->hidden(-name=>'page', -value => "dropxml") . 
"</div>";
  +    $html .= $cgi->start_form(-action => "$myurl?page=dropxml");
       $html .= "<div>" . $cgi->textarea(
           -name       => 'data',
           -columns    => 80,
  @@ .
______________________________________________________________________
The OpenPKG Project                                    www.openpkg.org
CVS Repository Commit List                     openpkg-cvs@openpkg.org

Reply via email to