removed JS submission from stats code MDL-7492

diff --git a/admin/report/stats/index.php b/admin/report/stats/index.php
index 602fab8acef7200b249ec9e59c80db54cae65d5f..a526314be53bd0a6044b74ea63b3641729ae7af5 100644 (file)
--- a/admin/report/stats/index.php
+++ b/admin/report/stats/index.php
@@ -59,7 +59,7 @@
 //    $strreports = get_string("reports");
 //    $strstats = get_string('stats');
 //
-//    $menu = report_stats_mode_menu($course, $mode, $time);
+//    $menu = report_stats_mode_menu($course, $mode, $time); // add 4th $url parameter if uncommented!
 //
 //
 //    $crumb = "<a href=\"{$CFG->wwwroot}/admin\">".get_string('administration')."</a> ->

diff --git a/course/report/stats/index.php b/course/report/stats/index.php
index 3d819a18b21cb309804f8fa8bff28ebe0fd7e710..ddf11eb5e23eeb5789c3038851bb92f7bdcfa12b 100644 (file)
--- a/course/report/stats/index.php
+++ b/course/report/stats/index.php
@@ -52,7 +52,7 @@
     $strreports = get_string("reports");
     $strstats = get_string('stats');
 
-    $menu = report_stats_mode_menu($course, $mode, $time);
+    $menu = report_stats_mode_menu($course, $mode, $time, "$CFG->wwwroot/course/report/stats/index.php");
 
     $crumb = "<a href=\"../../view.php?id=$course->id\">$course->shortname</a> ->
               <a href=\"../../report.php?id=$course->id\">$strreports</a> ->

diff --git a/course/report/stats/lib.php b/course/report/stats/lib.php
index f22a7432dc6047f02443960c66ab9f47d7d0685d..f281d210f1d984ed999dbd963c5bde2bcab9dc13 100644 (file)
--- a/course/report/stats/lib.php
+++ b/course/report/stats/lib.php
@@ -8,7 +8,7 @@
     require_once($CFG->dirroot.'/lib/statslib.php');
 
 
-    function report_stats_mode_menu($course, $mode, $time) {
+    function report_stats_mode_menu($course, $mode, $time, $url) {
         global $CFG;
         /*        
         $reportoptions = stats_get_report_options($course->id, $mode);
@@ -24,15 +24,8 @@
         if (has_capability('moodle/site:viewreports', get_context_instance(CONTEXT_SYSTEM, SITEID))) {
             $options[STATS_MODE_RANKED] = get_string('reports');
         }
-
-        $menu = choose_from_menu($options,'mode',$mode,'','this.form.submit();',0,true);
-
-        $menu = '<form action="index.php" method="post">'."\n"
-            .'<input type="hidden" name="course" value="'.$course->id.'" />'."\n"
-            .'<input type="hidden" name="time" value="'.$time.'" />'."\n"
-            .$menu."\n".'</form>';
         
-        return $menu;
+        return popup_form($url."?course=$course->id&amp;time=$time&amp;mode=", $options, 'switchmode', $mode, '', '', '', true);
     }