Clean up message handling after importing a zipped problem.

Nowadays, the messages is a keyed array with the keys being the warning
level. Previously, it was a flat data structure and apparently we didn't
clean up all of them.

There is still a code duplication between the two controller, but moving
this to the Utils class is ugly because it would require passing in the
controller.

Fixes #2633.

(cherry picked from commit a6640d5)

Author: meisterT

webapp/src/Controller/Jury/ImportExportController.php

@@ -184,7 +184,7 @@ public function indexAction(Request $request): Response
                     $this->dj->auditlog('problem', $newProblem->getProbid(), 'upload zip',
                         $clientName);
                 } else {
-                    $this->addFlash('danger', implode("\n", $allMessages));
+                    $this->postMessages($allMessages);
                     return $this->redirectToRoute('jury_problems');
                 }
             } catch (Exception $e) {
@@ -194,12 +194,7 @@ public function indexAction(Request $request): Response
                     $zip->close();
                 }
             }
-
-            foreach (['info', 'warning', 'danger'] as $type) {
-                if (!empty($allMessages[$type])) {
-                    $this->addFlash($type, implode("\n", $allMessages[$type]));
-                }
-            }
+            $this->postMessages($allMessages);
 
             if ($newProblem !== null) {
                 return $this->redirectToRoute('jury_problem', ['probId' => $newProblem->getProbid()]);
@@ -575,4 +570,16 @@ protected function getClarificationsHtml(): Response
             'problems' => $contestProblems,
         ]);
     }
+
+    /**
+     * @param array<string, string[]> $allMessages
+     */
+    private function postMessages(array $allMessages): void
+    {
+        foreach (['info', 'warning', 'danger'] as $type) {
+            if (!empty($allMessages[$type])) {
+                $this->addFlash($type, implode("\n", $allMessages[$type]));
+            }
+        }
+    }
 }

webapp/src/Controller/Jury/ProblemController.php

@@ -972,12 +972,7 @@ public function editAction(Request $request, int $probId): Response
                     $zip->close();
                 }
             }
-
-            foreach (['info', 'warning', 'danger'] as $type) {
-                if (!empty($messages[$type])) {
-                    $this->addFlash($type, implode("\n", $messages[$type]));
-                }
-            }
+            $this->postMessages($messages);
 
             return $this->redirectToRoute('jury_problem', ['probId' => $problem->getProbid()]);
         }
@@ -1172,4 +1167,16 @@ public function requestRemainingRunsWholeProblemAction(string $probId): Redirect
         $this->judgeRemaining($judgings);
         return $this->redirect($this->generateUrl('jury_problem', ['probId' => $probId]));
     }
+
+    /**
+     * @param array<string, string[]> $allMessages
+     */
+    private function postMessages(array $allMessages): void
+    {
+        foreach (['info', 'warning', 'danger'] as $type) {
+            if (!empty($allMessages[$type])) {
+                $this->addFlash($type, implode("\n", $allMessages[$type]));
+            }
+        }
+    }
 }