Add more left joins for internal error judging tables since they all depend on judgings.

Also fix logic when we have no affected judgings.

(cherry picked from commit 9a05f11)

Author: nickygerritsen

webapp/src/Controller/Jury/InternalErrorController.php

@@ -154,9 +154,9 @@ public function handleAction(Request $request, ?Profiler $profiler, int $errorId
         $internalError = $this->em->createQueryBuilder()
             ->from(InternalError::class, 'e')
             ->leftJoin('e.affectedJudgings', 'j')
-            ->join('j.submission', 's')
-            ->join('j.contest', 'c')
-            ->join('s.team', 't')
+            ->leftJoin('j.submission', 's')
+            ->leftJoin('j.contest', 'c')
+            ->leftJoin('s.team', 't')
             ->leftJoin('s.rejudging', 'r')
             ->select('e, j, s, c, t, r')
             ->where('e.errorid = :id')
@@ -195,7 +195,7 @@ public function handleAction(Request $request, ?Profiler $profiler, int $errorId
                         sprintf('internal error: %s', InternalErrorStatusType::STATUS_RESOLVED));
 
                     $affectedJudgings = $internalError->getAffectedJudgings();
-                    if (!empty($affectedJudgings)) {
+                    if (!$affectedJudgings->isEmpty()) {
                         $skipped          = [];
                         $rejudging        = $this->rejudgingService->createRejudging(
                             'Internal Error ' . $internalError->getErrorid() . ' resolved',
@@ -220,6 +220,8 @@ public function handleAction(Request $request, ?Profiler $profiler, int $errorId
                             );
                             $progressReporter(100, '', $message);
                         }
+                    } else {
+                        $progressReporter(100, '', 'No affected judgings.');
                     }
                 });
             });