MDL-59970 competencies: performance
authorDamyon Wiese <damyon@moodle.com>
Wed, 22 Nov 2017 07:35:20 +0000 (15:35 +0800)
committerDamyon Wiese <damyon@moodle.com>
Wed, 22 Nov 2017 07:35:20 +0000 (15:35 +0800)
Remove useless get_records call in get_least_proficient_competencies_for_course function.
On big sites this would cause huge memory usage.

competency/classes/user_competency_course.php

index 392e1c5..ad97f6c 100644 (file)
@@ -282,7 +282,6 @@ class user_competency_course extends persistent {
               ORDER BY p.timesproficient ASC, c.id DESC';
 
         $results = $DB->get_records_sql($sql, $params, $skip, $limit);
-        $a = $DB->get_records_sql('SELECT * from {' . self::TABLE . '}');
 
         $comps = array();
         foreach ($results as $r) {