Skip to content

Commit

Permalink
get_fast_modinfo once per course and caching it
Browse files Browse the repository at this point in the history
  • Loading branch information
rajandangi committed Jun 18, 2024
1 parent 5280db8 commit 892cc89
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions lib.php
Original file line number Diff line number Diff line change
Expand Up @@ -771,12 +771,14 @@ function reengagement_get_startusers($reengagement) {
AND u.id NOT IN ($alreadycompletionsql)
AND u.id NOT IN ($alreadyripsql)";

// Load modinfo once per course and cache it to avoid multiple calls to get_fast_modinfo.
$modinfo = get_fast_modinfo($reengagement->courseid);
$cm = $modinfo->get_cm($reengagement->cmid);
$ainfomod = new \core_availability\info_module($cm);
$information = '';

$startusers = $DB->get_records_sql($sql, $params);
foreach ($startusers as $startcandidate) {
$modinfo = get_fast_modinfo($reengagement->courseid, $startcandidate->id);
$cm = $modinfo->get_cm($reengagement->cmid);
$ainfomod = new \core_availability\info_module($cm);
$information = '';
// Exclude users who can't see this activity.
if (!$ainfomod->is_available($information, false, $startcandidate->id, $modinfo)) {
unset($startusers[$startcandidate->id]);
Expand Down

0 comments on commit 892cc89

Please sign in to comment.