Use two different variables for error tracking so the logic is more clear.

Author: nickygerritsen

webapp/src/Service/ImportExportService.php

@@ -330,10 +330,9 @@ public function importProblemsData(Contest $contest, array $problems, array &$id
                 ->setTimelimit($problemData['time_limit'] ?? 10)
                 ->setExternalid($problemData['id'] ?? $problemData['short-name'] ?? $problemLabel ?? null);
 
-            $hasErrors = false;
-            $errors    = $this->validator->validate($problem);
-            if ($errors->count()) {
-                $hasErrors = true;
+            $errors           = $this->validator->validate($problem);
+            $hasProblemErrors = $errors->count();
+            if ($hasProblemErrors) {
                 /** @var ConstraintViolationInterface $error */
                 foreach ($errors as $error) {
                     $messages['danger'][] = sprintf(
@@ -353,9 +352,9 @@ public function importProblemsData(Contest $contest, array $problems, array &$id
                 ->setProblem($problem)
                 ->setContest($contest);
 
-            $errors    = $this->validator->validate($contestProblem);
-            if ($errors->count()) {
-                $hasErrors = true;
+            $errors                  = $this->validator->validate($contestProblem);
+            $hasContestProblemErrors = $errors->count();
+            if ($hasContestProblemErrors) {
                 /** @var ConstraintViolationInterface $error */
                 foreach ($errors as $error) {
                     $messages['danger'][] = sprintf(
@@ -367,7 +366,7 @@ public function importProblemsData(Contest $contest, array $problems, array &$id
                 }
             }
 
-            if ($hasErrors) {
+            if ($hasProblemErrors || $hasContestProblemErrors) {
                 return false;
             }