Merge pull request #6288 from christianbeeznest/fide-22571-4

christianbeeznest · web-flow · commit adf4702a2c60 · 2025-05-13T00:51:16.000-05:00
Attendance: Fix PDF export crash when no teacher is assigned to course or session - refs BT#22571
diff --git a/src/CoreBundle/Controller/AttendanceController.php b/src/CoreBundle/Controller/AttendanceController.php
@@ -5,7 +5,9 @@
 namespace Chamilo\CoreBundle\Controller;
 
 use Chamilo\CoreBundle\Entity\Course;
+use Chamilo\CoreBundle\Entity\CourseRelUser;
 use Chamilo\CoreBundle\Entity\Session;
+use Chamilo\CoreBundle\Entity\SessionRelCourseRelUser;
 use Chamilo\CoreBundle\Entity\User;
 use Chamilo\CoreBundle\Repository\Node\UserRepository;
 use Chamilo\CourseBundle\Entity\CAttendance;
@@ -167,11 +169,20 @@ public function exportToPdf(int $id, Request $request): Response
 
         if ($sessionId) {
             $session = $this->em->getRepository(Session::class)->find($sessionId);
-            $teacher = $session?->getCourseCoachesSubscriptions()
+            $rel = $session?->getCourseCoachesSubscriptions()
                 ->filter(fn($rel) => $rel->getCourse()?->getId() === $courseId)
-                ->first()?->getUser()?->getFullname();
+                ->first();
+
+            $teacher = $rel instanceof SessionRelCourseRelUser
+                ? $rel->getUser()?->getFullname()
+                : null;
         } else {
-            $teacher = $course?->getTeachersSubscriptions()?->first()?->getUser()?->getFullname();
+            $rel = $course?->getTeachersSubscriptions()?->first();
+
+            $teacher = $rel instanceof CourseRelUser
+                ? $rel->getUser()?->getFullname()
+                : null;
+            ;
         }
 
         // Header
