Unify exceptions into one.

nickygerritsen · nickygerritsen · commit b13a771b5119 · 2024-04-29T19:48:34.000+02:00
diff --git a/webapp/src/Controller/API/ProblemController.php b/webapp/src/Controller/API/ProblemController.php
@@ -105,10 +105,11 @@ public function addProblemsAction(Request $request): array
         if ($this->importExportService->importProblemsData($contest, $data, $ids, $messages)) {
             return $ids;
         }
+        $message = "Error while adding problems";
         if (!empty($messages)) {
-            throw new BadRequestHttpException($this->dj->jsonEncode($messages));
+            $message .= ': ' . $this->dj->jsonEncode($messages);
         }
-        throw new BadRequestHttpException("Error while adding problems");
+        throw new BadRequestHttpException($message);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -105,10 +105,11 @@ public function addProblemsAction(Request $request): array`
`105`	`105`	`if ($this->importExportService->importProblemsData($contest, $data, $ids, $messages)) {`
`106`	`106`	`return $ids;`
`107`	`107`	`}`
	`108`	`+ $message = "Error while adding problems";`
`108`	`109`	`if (!empty($messages)) {`
`109`		`- throw new BadRequestHttpException($this->dj->jsonEncode($messages));`
	`110`	`+ $message .= ': ' . $this->dj->jsonEncode($messages);`
`110`	`111`	`}`
`111`		`- throw new BadRequestHttpException("Error while adding problems");`
	`112`	`+ throw new BadRequestHttpException($message);`
`112`	`113`	`}`
`113`	`114`
`114`	`115`	`/**`