the code generating the sandbox link at the top of the page lost some of its arguments like a bad politician before the elections. The users controller suffered the most, but it's now sandbox-aware. The corrected code won't make anyone president, but it fixes bug 373193.

git-svn-id: http://svn.mozilla.org/addons/trunk@2490 4eb1ac78-321c-0410-a911-ec516a8615a5

1 files changed, 5 insertions, 4 deletions

diff --git a/site/app/app_controller.php b/site/app/app_controller.php
index d011966..a062e56 100644
--- a/site/app/app_controller.php
+++ b/site/app/app_controller.php
@@ -42,8 +42,8 @@ class AppController extends Controller
     var $uses = array('User','Group');
     var $view = 'Addons';
 
-    // allow named arguments, default off
-    var $namedArgs = false;
+    // allow named arguments, default on
+    var $namedArgs = true;
     var $argSeparator = ":";
     var $beforeFilter = array('getNamedArgs');
 
@@ -54,7 +54,8 @@ class AppController extends Controller
 
     /**
     * Enables sandbox access when requested, via the status named argument.
-    * add "_checkSandbox" to beforeFilter to enable sandbox context in a controller
+    * add "_checkSandbox" to beforeFilter array to enable sandbox context
+    * in a controller
     */
     
     function _checkSandbox() {
@@ -87,7 +88,7 @@ class AppController extends Controller
 
         // This is either PUBLIC or SANDBOX and is used for view switching.
         // Since morgamic is a jerk we have to check the status array.
-        if (in_array(STATUS_SANDBOX,$this->status)) {
+        if (in_array(STATUS_SANDBOX, $this->status)) {
             $this->set('addonStatus', STATUS_SANDBOX);
         } else {
             $this->set('addonStatus', STATUS_PUBLIC);