highlighting sloppy globals hacks
authorPetr Skoda <skodak@moodle.org>
Sat, 18 Sep 2010 13:55:56 +0000 (13:55 +0000)
committerPetr Skoda <skodak@moodle.org>
Sat, 18 Sep 2010 13:55:56 +0000 (13:55 +0000)
mod/survey/lib.php
mod/survey/save.php
mod/survey/view.php

index 14c9b61..bc52cf2 100644 (file)
@@ -525,7 +525,7 @@ function survey_shorten_name ($name, $numwords) {
  * @param object $question
  */
 function survey_print_multi($question) {
-    global $USER, $DB, $qnum, $checklist, $DB, $OUTPUT;
+    global $USER, $DB, $qnum, $checklist, $DB, $OUTPUT; //TODO: this is sloppy globals abuse
 
     $stripreferthat = get_string("ipreferthat", "survey");
     $strifoundthat = get_string("ifoundthat", "survey");
index d87f431..6878bf6 100644 (file)
@@ -74,8 +74,8 @@
 
     $answers = array();
 
-    foreach ($formdata as $key => $val) {        
-        if ($key <> "userid" && $key <> "id") { 
+    foreach ($formdata as $key => $val) {
+        if ($key <> "userid" && $key <> "id") {
             if ( substr($key,0,1) == "q") {
                 $key = clean_param(substr($key,1), PARAM_ALPHANUM);   // keep everything but the 'q', number or Pnumber
             }
@@ -94,6 +94,7 @@
     $timenow = time();
     foreach ($answers as $key => $val) {
         if ($key != 'sesskey') {
+            $newdata = new object();
             $newdata->time = $timenow;
             $newdata->userid = $USER->id;
             $newdata->survey = $survey->id;
index 55ef262..b75fab6 100644 (file)
 
 // Cycle through all the questions in order and print them
 
+    global $qnum;  //TODO: ugly globals hack for survey_print_*()
+    global $checklist; //TODO: ugly globals hack for survey_print_*()
     $qnum = 0;
+    $checklist = array(); 
     foreach ($questionorder as $key => $val) {
         $question = $questions["$val"];
         $question->id = $val;