Order by order count. Courses that have many orders are emailed first. Merged from...

diff --git a/enrol/authorize/enrol.php b/enrol/authorize/enrol.php
index ba7dd9fed5576f5c07c4219ae0d52804cea3f2e3..b1f62b1f6c7a6dfe917a88b8ab04f4b9cfbca7aa 100755 (executable)
--- a/enrol/authorize/enrol.php
+++ b/enrol/authorize/enrol.php
@@ -666,16 +666,16 @@ class enrolment_plugin_authorize
                     $adminuser = get_admin();
                     email_to_user($adminuser, $adminuser, $subject, $message);
                     if (!empty($CFG->an_teachermanagepay) and !empty($CFG->an_emailexpiredteacher)) {
-                        $sql = "SELECT DISTINCT E.courseid, COUNT(E.courseid) AS count " .
+                        $sql = "SELECT E.courseid, COUNT(E.courseid) AS cnt " .
                                "FROM {$CFG->prefix}enrol_authorize E " .
-                               "WHERE $select GROUP BY E.courseid";
+                               "WHERE $select GROUP BY E.courseid ORDER BY cnt DESC";
                         $message = ''; $subject = '';
                         $lastcourse = 0; $lastcount = 0;
                         $courseidandcounts = get_records_sql($sql);
                         foreach($courseidandcounts as $courseidandcount) {
                             if ($lastcourse != $courseidandcount->courseid) {
                                 $lastcourse = $courseidandcount->courseid;
-                                $lastcount = $courseidandcount->count;
+                                $lastcount = $courseidandcount->cnt;
                                 $a = new stdClass;
                                 $a->pending = $lastcount;
                                 $a->days = $CFG->an_emailexpired;