From 69f6d0b4b2b013bdec98912d118ec0ffa6751f5c Mon Sep 17 00:00:00 2001
From: Andreas Grabs <info@grabs-edv.de>
Date: Fri, 2 Apr 2010 22:09:00 +0000
Subject: [PATCH] convert old dml syntax in the new one in
 feedback_get_recent_mod_activity()

---
 mod/feedback/lib.php | 41 ++++++++++++++++++++++-------------------
 1 file changed, 22 insertions(+), 19 deletions(-)

diff --git a/mod/feedback/lib.php b/mod/feedback/lib.php
index 1f1587a9923..3f1b0b02ab3 100644
--- a/mod/feedback/lib.php
+++ b/mod/feedback/lib.php
@@ -231,33 +231,36 @@ function feedback_get_recent_mod_activity(&$activities, &$index, $timemodified,
     $modinfo =& get_fast_modinfo($course);
 
     $cm = $modinfo->cms[$cmid];
+    
+    $sqlargs = array();
+
+    $sql = " SELECT fk . * , fc . * , u.firstname, u.lastname, u.email, u.picture
+                                            FROM {feedback_completed} fc
+                                                JOIN {feedback} fk ON fk.id = fc.feedback
+                                                JOIN {user} u ON u.id = fc.userid ";
+
+    if ($groupid) {
+        $sql .= " JOIN {groups_members} gm ON  gm.userid=u.id ";
+    }
+    
+    $sql .= " WHERE fc.timemodified > ? AND fk.id = ? ";
+    $sqlargs[] = $timemodified;
+    $sqlargs[] = $cm->instace;
 
     if ($userid) {
-        $userselect = "AND u.id = $userid";
-    } else {
-        $userselect = "";
+        $sql .= " AND u.id = ? ";
+        $sqlargs[] = $userid;
     }
 
     if ($groupid) {
-        $groupselect = "AND gm.groupid = $groupid";
-        $groupjoin   = "JOIN {groups_members} gm ON  gm.userid=u.id";
-    } else {
-        $groupselect = "";
-        $groupjoin   = "";
+        $sql .= " AND gm.groupid = ? ";
+        $sqlargs[] = $groupid;
     }
-
-    if (!$feedbackitems = $DB->get_records_sql("SELECT fk . * , fc . * , u.firstname, u.lastname, u.email, u.picture
-                                            FROM {feedback_completed} fc
-                                                JOIN {feedback} fk ON fk.id = fc.feedback
-                                                JOIN {user} u ON u.id = fc.userid
-                                                $groupjoin
-                                            WHERE fc.timemodified > $timemodified AND fk.id = $cm->instance
-                                                $userselect $groupselect
-                                            ORDER BY fc.timemodified DESC")) {
-         return;
+    
+    if (!$feedbackitems = $DB->get_records_sql($sql, $sqlargs)) {
+        return;
     }
 
-
     $cm_context      = get_context_instance(CONTEXT_MODULE, $cm->id);
     $accessallgroups = has_capability('moodle/site:accessallgroups', $cm_context);
     $viewfullnames   = has_capability('moodle/site:viewfullnames', $cm_context);