Changeset 1292 for trunk/wifidog-auth/wifidog/classes/Statistics.php

Timestamp:

09/13/07 11:23:24 (6 years ago)

Author:

benoitg

Message:

• Scratch a few itches: Fix the link from node admin to stats, minor CSS fixes

Files:

• trunk/wifidog-auth/wifidog/classes/Statistics.php (modified) (3 diffs)

trunk/wifidog-auth/wifidog/classes/Statistics.php

@@ -369 +369 @@
         $html = '';
 
-        $name = "statistics_selected_nodes[]";
+        $name = "selected_nodes[]";
         $user = User :: getCurrentUser();
 
@@ -398 +398 @@
      */
     private function processSelectedNodesUI() {
-        $name = "statistics_selected_nodes";
+        $name = "selected_nodes";
         //pretty_print_r($_REQUEST[$name]);
         $this->report_selected_nodes = array ();
@@ -445 +445 @@
     /**
      * Process the date range selection UI
+     * @return true id options were set, false otherwise
      */
     private function processDistinguishUsersByUI() {
-        if (!isset ($this->user_distinguish_by_options[$_REQUEST['distinguish_users_by']]))
-        throw new exception(_("Invalid parameter"));
-        $this->report_distinguish_users_by = $_REQUEST['distinguish_users_by'];
+        $retval = false;
+        if (empty($_REQUEST['distinguish_users_by'])) {
+            //Keep whatever was set before
+        }
+        else if (!isset ($this->user_distinguish_by_options[$_REQUEST['distinguish_users_by']])) {
+            throw new exception(_("Invalid parameter"));
+        }
+        else {
+            $this->report_distinguish_users_by = $_REQUEST['distinguish_users_by'];
+                    $retval = true;
+        }
+        return $retval;
+        
     }