new iteration

dunglas · dunglas · commit 2a83b5745534 · 2021-04-13T18:30:58.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php b/src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php
@@ -297,29 +297,28 @@ protected function stream(string $view, array $parameters = [], StreamedResponse
      * * if the form is submitted but invalid, the Twig template passed in $view is rendered and 422 HTTP status code is set
      * * if the form is submitted and valid, the entity is saved (only if it is managed by Doctrine ORM), a 306 HTTP status code is set and the Location HTTP header is set to the value of $redirectUrl
      */
-    public function handleForm(Request $request, FormInterface $form, string $view, ?string $redirectUrl = null, ?callable $onSuccess = null, array $viewParameters = [], ?Response $response = null): Response
+
+    /**
+     * Handles a form.
+     *
+     * * if the form is not submitted, $render is called
+     * * if the form is submitted but invalid, $render is called and a 422 HTTP status code is set if the current status isn't in the 4XX range
+     * * if the form is submitted and valid, $onSuccess is called, usually this method saves the data and returns a 306 HTTP redirection
+     *
+     * @param callable(FormInterface): Response $onSuccess
+     * @param callable(FormInterface): Response $render
+     */
+    public function handleForm(Request $request, FormInterface $form, callable $onSuccess, callable $render): Response
     {
         $form->handleRequest($request);
 
         $submitted = $form->isSubmitted();
-        $valid = $submitted && $form->isValid();
-
-        if ($valid) {
-            if (null !== $onSuccess) {
-                return $onSuccess();
-            }
-
-            $data = $form->getData();
-            if ($manager = $this->getDoctrine()->getManagerForClass(get_class($data))) {
-                $manager->persist($data);
-                $manager->flush();
-            }
-
-            return $this->redirect($redirectUrl, Response::HTTP_SEE_OTHER);
+        if ($submitted && $form->isValid()) {
+            return $onSuccess($form);
         }
 
-        $response = $this->render($view, ['form' => $form->createView()] + $viewParameters, $response);
-        if ($submitted) {
+        $response = $render($form);
+        if ($submitted && ($code = $response->getStatusCode()) < 400 || $code >= 500) {
             $response->setStatusCode(Response::HTTP_UNPROCESSABLE_ENTITY);
         }
 
