Commit3db80ca

committed

pass $data as second parameter

1 parent4b7271a commit3db80caCopy full SHA for 3db80ca

File tree

-6

lines changed

src/Symfony/Bundle/FrameworkBundle
- Controller
  - AbstractController.php
- Tests/Controller
  - AbstractControllerTest.php

-6

lines changed

Lines changed: 6 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -297,19 +297,21 @@ protected function stream(string $view, array $parameters = [], StreamedResponse`
`297`	`297`	`* * if the form is submitted but invalid, $render is called and a 422 HTTP status code is set if the current status hasn't been customized`
`298`	`298`	`* * if the form is submitted and valid, $onSuccess is called, usually this method saves the data and returns a 303 HTTP redirection`
`299`	`299`	`*`
`300`		`- * @param callable(FormInterface): Response $onSuccess`
`301`		`- * @param callable(FormInterface): Response $render`
	`300`	`+ * @param callable(FormInterface, mixed): Response $onSuccess`
	`301`	`+ * @param callable(FormInterface, mixed): Response $render`
`302`	`302`	`*/`
`303`	`303`	`publicfunctionhandleForm(FormInterface$form,Request$request,callable$onSuccess,callable$render):Response`
`304`	`304`	`{`
`305`	`305`	`$form->handleRequest($request);`
`306`	`306`
`307`	`307`	`$submitted =$form->isSubmitted();`
	`308`	`+`
	`309`	`+$data =$form->getData();`
`308`	`310`	`if ($submitted &&$form->isValid()) {`
`309`		`-return$onSuccess($form);`
	`311`	`+return$onSuccess($form,$data);`
`310`	`312`	`}`
`311`	`313`
`312`		`-$response =$render($form);`
	`314`	`+$response =$render($form,$data);`
`313`	`315`	`if ($submitted &&200 ===$response->getStatusCode()) {`
`314`	`316`	`$response->setStatusCode(Response::HTTP_UNPROCESSABLE_ENTITY);`
`315`	`317`	`}`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -433,10 +433,10 @@ public function testHandleFormNotSubmitted()`
`433`	`433`	`$response =$controller->handleForm(`
`434`	`434`	`$form,`
`435`	`435`	`Request::create('https://example.com'),`
`436`		`-function (FormInterface$form):Response {`
	`436`	`+function (FormInterface$form,$data):Response {`
`437`	`437`	`returnnewRedirectResponse('https://example.com/redir', Response::HTTP_SEE_OTHER);`
`438`	`438`	`},`
`439`		`-function (FormInterface$form):Response {`
	`439`	`+function (FormInterface$form,$data):Response {`
`440`	`440`	`returnnewResponse('rendered');`
`441`	`441`	`}`
`442`	`442`	`);`

Comments

(0)