Merge pull request #5927 from christianbeeznest/ofaj-22197

Internal: Filter student count in session status update script - refs BT#22197

Author: christianbeeznest

src/CoreBundle/Command/UpdateSessionStatusCommand.php

@@ -50,9 +50,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $id = $session->getId();
             $start = $session->getDisplayStartDate();
             $end = $session->getDisplayEndDate();
-            $userCount = $this->sessionRepository->countUsersBySession($session->getId());
+            $studentCount = $this->sessionRepository->countUsersBySession($session->getId());
 
-            $status = $this->determineSessionStatus($start, $end, $userCount, $now);
+            $status = $this->determineSessionStatus($start, $end, $studentCount, $now);
 
             if ($debug) {
                 $startFormatted = $start ? $start->format('Y-m-d H:i:s') : 'N/A';

src/CoreBundle/Repository/SessionRepository.php

@@ -464,13 +464,15 @@ public function getSubscribedSessionsOfUserInUrl(
         return array_filter($sessions, $filterSessions);
     }
 
-    public function countUsersBySession(int $sessionId): int
+    public function countUsersBySession(int $sessionId, int $relationType = Session::STUDENT): int
     {
         $qb = $this->createQueryBuilder('s');
         $qb->select('COUNT(sru.id)')
             ->innerJoin('s.users', 'sru')
             ->where('s.id = :sessionId')
-            ->setParameter('sessionId', $sessionId);
+            ->andWhere('sru.relationType = :relationType')
+            ->setParameter('sessionId', $sessionId)
+            ->setParameter('relationType', $relationType);
 
         return (int) $qb->getQuery()->getSingleScalarResult();
     }