Skip to content

Commit 285ec9f

Browse files
nickygerritsenvmcj
authored andcommitted
Only set internal error on judging if we have one.
Diskspace errors are not linked to any judging. (cherry picked from commit ab07c2b)
1 parent c2b36f5 commit 285ec9f

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

webapp/src/Controller/API/JudgehostController.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -820,7 +820,9 @@ public function internalErrorAction(Request $request): ?int
820820
$this->em->persist($error);
821821
// Even if there are no remaining judge tasks for this judging open (which is covered by the transaction below),
822822
// we need to mark this judging as internal error.
823-
$judging->setInternalError($error);
823+
if ($judging) {
824+
$judging->setInternalError($error);
825+
}
824826
$this->em->flush();
825827

826828
if ($field_name !== null) {

0 commit comments

Comments
 (0)