diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[FrameworkBundle] fix AbstractController::handleForm()

Browse Source
This commit is contained in:
Nicolas Grekas 2021-05-12 09:43:11 +02:00
parent b14d76921f
commit 777c3c25b2
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Bundle/FrameworkBundle/Controller/AbstractController.php
View File

@ -314,7 +314,7 @@ abstract class AbstractController implements ServiceSubscriberInterface
} else { } else {
$response = $render($form, $data, $request); $response = $render($form, $data, $request);
if ($submitted && 200 === $response->getStatusCode()) { if ($response instanceof Response && $submitted && 200 === $response->getStatusCode()) {
$response->setStatusCode(Response::HTTP_UNPROCESSABLE_ENTITY); $response->setStatusCode(Response::HTTP_UNPROCESSABLE_ENTITY);
} }
} }

4
src/Symfony/Bundle/FrameworkBundle/Tests/Controller/AbstractControllerTest.php
View File

@ -494,12 +494,12 @@ class AbstractControllerTest extends TestCase
{ {
$form = $this->createMock(FormInterface::class); $form = $this->createMock(FormInterface::class);
$form->expects($this->once())->method('isSubmitted')->willReturn(true); $form->expects($this->once())->method('isSubmitted')->willReturn(true);
$form->expects($this->once())->method('isValid')->willReturn(true); $form->expects($this->once())->method('isValid')->willReturn(false);
$controller = $this->createController(); $controller = $this->createController();
$this->expectException(\TypeError::class); $this->expectException(\TypeError::class);
$this->expectExceptionMessage('The "$onSuccess" callable passed to "Symfony\Bundle\FrameworkBundle\Tests\Controller\TestAbstractController::handleForm()" must return a Response, "string" returned.'); $this->expectExceptionMessage('The "$render" callable passed to "Symfony\Bundle\FrameworkBundle\Tests\Controller\TestAbstractController::handleForm()" must return a Response, "string" returned.');
$response = $controller->handleForm( $response = $controller->handleForm(
$form, $form,