Fixing one SQL that could cause problems with coincidences

in the associative array. MDL-10787 Merged from MOODLE_18_STABLE
moodle · Aug 10, 2007 · 87f47be · 87f47be
1 parent 8207a81
commit 87f47be
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/mod/survey/lib.php b/mod/survey/lib.php
@@ -224,14 +224,13 @@ function survey_get_responses($surveyid, $groupid) {
         $groupsql = "";
     }
 
-    return get_records_sql("SELECT MAX(a.time) as time, 
-                                   u.id, u.firstname, u.lastname, u.picture
-                              FROM {$CFG->prefix}survey_answers a, 
-                                   {$CFG->prefix}user u   $groupsdb
-                             WHERE a.survey = $surveyid 
-                                   AND a.userid = u.id $groupsql
-                          GROUP BY u.id, u.firstname, u.lastname, u.picture
-                          ORDER BY time ASC");
+    return get_records_sql("SELECT u.id, u.firstname, u.lastname, u.picture, MAX(a.time) as time
+                            FROM {$CFG->prefix}survey_answers a,
+                                 {$CFG->prefix}user u   $groupsdb
+                            WHERE a.survey = $surveyid
+                              AND a.userid = u.id $groupsql
+                            GROUP BY u.id, u.firstname, u.lastname, u.picture
+                            ORDER BY time ASC");
 }
 
 function survey_get_analysis($survey, $user) {