8000 [Form] Missing Data Handling (checkbox edge cases) by filiplikavcan · Pull Request #45081 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Form] Missing Data Handling (checkbox edge cases) #45081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: 7.4
Choose a base branch
from
Open

[Form] Missing Data Handling (checkbox edge cases) #45081

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
d47a57f
[Form] Missing Data Handling (checkbox)
filiplikavcan Jan 19, 2022
753382d
Fix failing tests (missingDataHandler was not used in HttpFoundationR…
filiplikavcan Aug 2, 2022
9da6d1c
Remove unnecessary 'use'
filiplikavcan Aug 2, 2022
82535ab
Force push
filiplikavcan Aug 2, 2022
4eb6dad
Force push
filiplikavcan Aug 2, 2022
663984e
Add original logic (has/all) back
filiplikavcan Aug 2, 2022
591b4b9
CS
filiplikavcan Aug 2, 2022
d1853fd
CS
filiplikavcan Aug 2, 2022
5a5fe0a
Handle uninitialized FormConfigBuilder::$type
filiplikavcan Aug 2, 2022
474ac30
CS
filiplikavcan Aug 2, 2022
ab3c101
Force restart tests
filiplikavcan Aug 2, 2022
5ef1388
[WIP] Rebase to 6.2
filiplikavcan Aug 3, 2022
4dcd5a4
[WIP] Code clarifications
filiplikavcan Oct 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add original logic (has/all) back
  • Loading branch information
@filiplikavcan
filiplikavcan committed Oct 4, 2022
commit 663984ed061d7d0611f2cea006bc8417ec47b61c
9 changes: 8 additions & 1 deletion src/Symfony/Component/Form/Extension/HttpFoundation/HttpFoundationRequestHandler.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,14 @@ public function handleRequest(FormInterface $form, mixed $request = null)
} else {
$missingData = $this->missingDataHandler->missingData;

if ($missingData === $data = $this->missingDataHandler->handle($form, $request->query->get($name) ?? $missingData)) {
if ($request->query->has($name)) {
$data = $request->query->all()[$name];
}
else {
$data = $missingData;
}

if ($missingData === $data = $this->missingDataHandler->handle($form, $data)) {
// Don't submit GET requests if the form's name does not exist
// in the request
return;
Expand Down
0