From 91671b558fea0f751b36a4ae6fb796751af0ca4a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rapha=C3=ABl=20Gertz?= Date: Mon, 28 Dec 2020 08:17:29 +0100 Subject: [PATCH] Remove useless post check Add isSubmitted check Change title scheme format --- Controller/ApplicationController.php | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/Controller/ApplicationController.php b/Controller/ApplicationController.php index 7cb9d97..45538ff 100644 --- a/Controller/ApplicationController.php +++ b/Controller/ApplicationController.php @@ -28,11 +28,6 @@ class ApplicationController extends DefaultController { //Prevent non-guest to access here $this->denyAccessUnlessGranted('ROLE_GUEST', null, $this->translator->trans('Unable to access this page without role %role%!', ['%role%' => $this->translator->trans('Guest')])); - //Reject non post requests - if (!$request->isMethod('POST')) { - throw new \RuntimeException('Request method MUST be POST'); - } - //Create ApplicationType form $form = $this->createForm('Rapsys\AirBundle\Form\ApplicationType', null, [ //Set the action @@ -52,12 +47,12 @@ class ApplicationController extends DefaultController { $form->handleRequest($request); //Handle invalid form - if (!$form->isValid()) { + if (!$form->isSubmitted() || !$form->isValid()) { //Set section $section = $this->translator->trans('Application add'); //Set title - $title = $section.' - '.$this->translator->trans($this->config['site']['title']); + $title = $this->translator->trans($this->config['site']['title']).' - '.$section; //Render the view return $this->render('@RapsysAir/application/add.html.twig', ['title' => $title, 'section' => $section, 'form' => $form->createView()]+$this->context); @@ -290,7 +285,7 @@ class ApplicationController extends DefaultController { $section = $this->translator->trans('Application add'); //Set title - $title = $section.' - '.$this->translator->trans($this->config['site']['title']); + $title = $this->translator->trans($this->config['site']['title']).' - '.$section; //Render the view return $this->render('@RapsysAir/application/add.html.twig', ['title' => $title, 'section' => $section, 'form' => $form->createView()]+$this->context); @@ -305,7 +300,7 @@ class ApplicationController extends DefaultController { $section = $this->translator->trans('Application add'); //Set title - $title = $section.' - '.$this->translator->trans($this->config['site']['title']); + $title = $this->translator->trans($this->config['site']['title']).' - '.$section; //Render the view return $this->render('@RapsysAir/application/add.html.twig', ['title' => $title, 'section' => $section, 'form' => $form->createView()]+$this->context); -- 2.41.1