Changeset 1057

Timestamp:

06/10/07 16:21:02 (18 months ago)

Author:

dom

Message:

If an unknown action is used redirect to action=display, preserving other args (see #102)

Location:

trunk

Files:

• Changes (modified) (1 diff)
• wiki.cgi (modified) (2 diffs)

trunk/Changes

@@ -15 +15 @@
           (#192).
         Respect redirect=0 (#104).
+        If an unknown action is supplied to wiki.cgi, redirect to
+          action=display (partial fix for #102)
 
 0.60    13 May 2007

trunk/wiki.cgi

@@ -187 +187 @@
     } elsif ($action eq 'about') {
         $guide->display_about(format => $format);
-    } else { # Default is to display a node.
+    } elsif ($action eq 'display') { 
         if ( $format and $format eq "rdf" ) {
             display_node_rdf( node => $node );
@@ -221 +221 @@
             }
         }
-    }
+    } else { 
+        # Fallback: redirect to the display page, preserving all vars
+        # except for the action, which we override.
+        # Note: $q->Vars needs munging if we need to support any
+        # multi-valued params
+        my $params = $q->Vars;
+        $params->{'action'} = 'display';
+        my $redir_target = $script_url . $script_name . '?';
+        my @args = map { "$_=" . $params->{$_} } keys %{$params};
+        $redir_target .= join ';', @args;
+        
+        print $q->redirect(
+            -uri => $redir_target,
+            -status => 303
+        );
+    }
+
 };