Remove use of doctrine forAll see BT#16186

main/lp/lp_subscribe_users.php

@@ -68,12 +68,13 @@ if (!$session) {
         ->getQuery()
         ->getResult();
 } else {
-    $session->getUserCourseSubscriptionsByStatus($course, Session::STUDENT)
-        ->forAll(
-            function ($i, SessionRelCourseRelUser $sessionCourseUser) use (&$subscribedUsers) {
-                $subscribedUsers[$i] = $sessionCourseUser->getUser();
-            }
-        );
+    $list = $session->getUserCourseSubscriptionsByStatus($course, Session::STUDENT);
+    if ($list) {
+        /** @var SessionRelCourseRelUser $sessionCourseUser */
+        foreach ($list as $sessionCourseUser) {
+            $subscribedUsers[$sessionCourseUser->getUser()->getUserId()] = $sessionCourseUser->getUser();
+        }
+    }
 }
 
 // Getting all users in a nice format.

main/session/resume_session.php

@@ -178,16 +178,17 @@ if ($session->getNbrCourses() === 0) {
             $course,
             $session
         );
+
         // Get coachs of the courses in session
         $namesOfCoaches = [];
-        $coachSubscriptions = $session->getUserCourseSubscriptionsByStatus($course, Session::COACH)
-            ->forAll(
-                function ($index, SessionRelCourseRelUser $subscription) use (&$namesOfCoaches) {
-                    $namesOfCoaches[] = $subscription->getUser()->getCompleteNameWithUserName();
+        $coachSubscriptions = $session->getUserCourseSubscriptionsByStatus($course, Session::COACH);
 
-                    return true;
-                }
-            );
+        if ($coachSubscriptions) {
+            /** @var SessionRelCourseRelUser $subscription */
+            foreach ($coachSubscriptions as $subscription) {
+                $namesOfCoaches[] = $subscription->getUser()->getCompleteNameWithUserName();
+            }
+        }
 
         $orderButtons = '';
         if (SessionManager::orderCourseIsEnabled()) {