- Issue created by @BenStallings
- Merge request !78change query->get to query->all to allow for non-scalar values → (Open) created by BenStallings
- Status changed to Needs review
10 months ago 5:18pm 12 January 2024 - 🇺🇸United States BenStallings
BenStallings → changed the visibility of the branch 3414404-badrequesthttpexception-input-value to hidden.
- 🇺🇸United States BenStallings
I'm not sure why I can't open a merge request, but here is the updated diff: https://git.drupalcode.org/issue/smart_date-3414404/-/commit/aa3d4f0711a...
- 🇨🇦Canada mandclu
@BenStallings it looks like you already opened MR !78 on this issue.
- 🇺🇸United States BenStallings
@mandclu yes, but it is botched. The MR contains nothing, and I can't figure out how to fix it, hence the patch.
- 🇨🇦Canada mandclu
I believe I was able to get the change back into the MR. Do you know when the change was made in Symfony? I'm just wondering if this change requires dropping support for versions of Drupal before 10.
- 🇺🇸United States BenStallings
@mandclu sorry, I have no idea when the change was made to Symfony.
- 🇨🇦Canada mandclu
According to 🐛 $request->get may no longer return array as of Symfony 6.0 Fixed this is related to new behaviour in Symfony 6, so the updated behaviour will only work in D10 or newer, which is consistent which what was originally reported in the Issue Summary. We'll need to update the requirements accordingly.