Controller
\npublic function updateBalanceSettings(\n #[MapRequestPayload] UpdateBalanceSettingsInput $input\n): Response {\n dump($input);\n exit;\n}DTOs
\nfinal readonly class UpdateBalanceSettingsInput\n{\n public function __construct(\n /** @var BalanceSettingInput[] */\n public array $settings,\n ) {}\n}\n\nclass BalanceSettingInput\n{\n public function __construct(\n public int $id,\n public bool $active,\n ) {}\n}Dump (Symfony 7.2)
\nApp\\UI\\Web\\DTO\\Cabinet\\Balance\\UpdateBalanceSettingsInput {#1961\n +settings: array:1 [\n 0 => App\\UI\\Web\\DTO\\Cabinet\\Balance\\BalanceSettingInput {#1964\n +id: 2\n +active: true\n }\n ]\n}Dump (Symfony 7.3)
\nApp\\UI\\Web\\DTO\\Cabinet\\Balance\\UpdateBalanceSettingsInput {#1613\n +settings: array:1 [\n 0 => array:2 [\n \"id\" => 2\n \"active\" => true\n ]\n ]\n}Expected Behavior
\nThe settings property should still contain an array of BalanceSettingInput objects,
\nbut since Symfony 7.3 it now returns an array of raw arrays instead.
Same problem is here https://www.reddit.com/r/symfony/comments/1nebngt/mapquerybuilder_in_73_parsing_problem/
","upvoteCount":1,"answerCount":1,"acceptedAnswer":{"@type":"Answer","text":"Ah this may be the same issue than #60795; could you try setting framework.property_info.with_constructor_extractor to false?
-
|
Hello everyone Problem Summary Input Payload {
"settings": [
{
"id": 2,
"active": false
}
]
}Controller public function updateBalanceSettings(
#[MapRequestPayload] UpdateBalanceSettingsInput $input
): Response {
dump($input);
exit;
}DTOs final readonly class UpdateBalanceSettingsInput
{
public function __construct(
/** @var BalanceSettingInput[] */
public array $settings,
) {}
}
class BalanceSettingInput
{
public function __construct(
public int $id,
public bool $active,
) {}
}Dump (Symfony 7.2) App\UI\Web\DTO\Cabinet\Balance\UpdateBalanceSettingsInput {#1961
+settings: array:1 [
0 => App\UI\Web\DTO\Cabinet\Balance\BalanceSettingInput {#1964
+id: 2
+active: true
}
]
}Dump (Symfony 7.3) App\UI\Web\DTO\Cabinet\Balance\UpdateBalanceSettingsInput {#1613
+settings: array:1 [
0 => array:2 [
"id" => 2
"active" => true
]
]
}Expected Behavior Same problem is here https://www.reddit.com/r/symfony/comments/1nebngt/mapquerybuilder_in_73_parsing_problem/ |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 7 replies
-
|
Hello, could you share the corresponding request payload? Note that since Symfony 7.1 you can map an array using |
Beta Was this translation helpful? Give feedback.
-
|
I updated the description of the problem to make it clear that I am using the MapRequestPayload option |
Beta Was this translation helpful? Give feedback.
-
|
Not sure if you saw but my message mentioned Anyways I can reproduce your issue so I’ll have a look. |
Beta Was this translation helpful? Give feedback.
-
|
Thx for your attention, and yes, I understand what you mean. I can't do without a root object because this input has other fields—not arrays. I just didn't add them so the example would be straightforward. So, I need a root object, and the type option won't help. The main point is that the framework version is different, because everything worked before. |
Beta Was this translation helpful? Give feedback.
-
|
Ah this may be the same issue than #60795; could you try setting |
Beta Was this translation helpful? Give feedback.
-
|
Damn, that really does sound like this problem! Setting the option didn't help me, but it does help if I add the fix in the issue you mentioned! Thanks so much for your time! final readonly class UpdateBalanceSettingsInput
{
/**
* @param BalanceSettingInput[] $settings
*/
public function __construct(
/** @var BalanceSettingInput[] */
public array $settings,
) {}
} |
Beta Was this translation helpful? Give feedback.
-
|
Nice 👌 If you can provide a reproducer for #60795 it would certainly help solving it! |
Beta Was this translation helpful? Give feedback.
-
|
Hi there, I pushed the PR #62385 regarding this issue, would you mind trying if it solves the use case with MapRequestPayload ? |
Beta Was this translation helpful? Give feedback.
Ah this may be the same issue than #60795; could you try setting
framework.property_info.with_constructor_extractortofalse?