- Merge request !63270171: Fixed an issue with headers variable in MultiresponseJsonNormalizer → (Open) created by quadrexdev
- last update
over 1 year ago 8 pass - last update
over 1 year ago 5 pass, 4 fail - last update
about 1 year ago 8 pass - 🇺🇸United States mglaman WI, USA
How could
sub-content-type
be empty? It should be populated by\Drupal\subrequests\Blueprint\BlueprintManager::negotiateSubContentType
.The bug seems to be in
\Drupal\subrequests\Normalizer\MultiresponseJsonNormalizer::supportsNormalization
normalizing things it shouldn't be. That's why the context doesn't match. - 🇺🇸United States mglaman WI, USA
Closed 🐛 Multiresponse normalizer invalid headers "sub-content-type" Needs review as a duplicate, I can't copy over an issue credits but it'd be nice to incorporate those folks into this ticket.
I think the root fix is still in
supportsNormalization
. Notnormalize
, since the$context
is maldformed. - last update
12 months ago 9 pass - Status changed to Needs review
12 months ago 9:09pm 18 December 2023 - 🇺🇸United States mglaman WI, USA
I've opened https://git.drupalcode.org/project/subrequests/-/merge_requests/22
Based on the related issues, the problem seems to occur when there is an empty array being normalized.
if (count($responses) !== count($data)) { return FALSE; }
- Status changed to RTBC
12 months ago 9:46pm 18 December 2023 - 🇺🇸United States wylbur Minneapolis, Minnesota, USA
- e0ipso Can Picafort
This looks good!
I am adding issue credit as suggested above.
- last update
12 months ago 9 pass - Status changed to Fixed
12 months ago 9:48pm 18 December 2023 Automatically closed - issue fixed for 2 weeks with no activity.