-
|
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?