Refactor reengagement_get_startusers function to use array_filter for…

… filtering startcandidates

lib.php

@@ -778,14 +778,11 @@ function reengagement_get_startusers($reengagement) {
     $information = '';
 
     $startusers = $DB->get_records_sql($sql, $params);
-    foreach ($startusers as $startcandidate) {
-        // Exclude users who can't see this activity.
-        if (!$ainfomod->is_available($information, false, $startcandidate->id, $modinfo)) {
-            unset($startusers[$startcandidate->id]);
-        }
-    }
+    $startcandidates = array_filter($startusers, function ($startcandidate) use ($ainfomod, $information, $modinfo) {
+        return $ainfomod->is_available($information, false, $startcandidate->id, $modinfo);
+    });
 
-    return $startusers;
+    return $startcandidates;
 }
 
 
@@ -796,7 +793,7 @@ function reengagement_get_startusers($reengagement) {
  * @return mixed True if module supports feature, null if doesn't know
  */
 function reengagement_supports($feature) {
-    switch($feature) {
+    switch ($feature) {
         case FEATURE_GROUPS:
             return false;
         case FEATURE_GROUPINGS: