From: mjollnir_ <mjollnir_>
Date: Tue, 23 Nov 2004 01:05:11 +0000 (+0000)
Subject: Merged from MOODlE_14_STABLE: Fix for invalid sql (for postgres) in cron cleanup... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=df7ebd69ab0ef3021305c6cefc485b347b8da865;p=moodle.git

Merged from MOODlE_14_STABLE: Fix for invalid sql (for postgres) in cron cleanup of duplicate enrolments. Possibly soon to be overridden by a unique index
---

diff --git a/admin/cron.php b/admin/cron.php
index 574886e632..b6e6a81b91 100644
--- a/admin/cron.php
+++ b/admin/cron.php
@@ -103,9 +103,9 @@
         /// Delete duplicate enrolments (don't know what causes these yet - expired sessions?)
     
         if ($users = get_records_select("user_students", "userid > 0 GROUP BY course, userid ".
-                                        "HAVING count(*) > 1", "", "*,count(*)")) {
+                                        "HAVING count(*) > 1", "", "max(id) as id, userid, course ,count(*)")) {
             foreach ($users as $user) {
-               delete_records_select("user_students", "userid = '$user->userid' ".
+                delete_records_select("user_students", "userid = '$user->userid' ".
                                      "AND course = '$user->course' AND id <> '$user->id'");
             }
         }