- Issue created by @jurgenhaas
- Status changed to Needs review
about 1 year ago 6:35am 23 October 2023 - last update
about 1 year ago 297 pass - 🇮🇳India sarwan_verma
Hi @jurgenhaas,
I have fixed this issue "ECA Form: Invalid array key type array in \Drupal\eca_render\Plugin\Action\RenderElementActionBase::execute" and also attached patch ,
please review and verify. - Status changed to Needs work
about 1 year ago 10:43am 27 October 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
@sarwan_verma sorry for the late reply. In what way do we know that simply encoding the array into a string would actually be the correct solution? Do we have a sample where this could be reproduced and demonstrated that this resolves the issue correctly?
Also, we're working with merge requests, not with patches. So, if we continue with this, let's work with an issue fork. Soon, that will be the only possible way when the GitLab migration will be completed.
- Issue was unassigned.
- Status changed to Needs review
12 months ago 4:17pm 28 November 2023 - Status changed to RTBC
9 months ago 3:11pm 22 February 2024 - 🇩🇪Germany danielspeicher Steisslingen
See my comment in https://www.drupal.org/project/eca/issues/3423162 📌 Restructure tests in render module Active . If these tests are ok, we can fix this one here, too.
- 🇩🇪Germany jurgenhaas Gottmadingen
As the tests in 📌 Restructure tests in render module Active already contain the merge tests, it looks to me as if this is already covered, right?
But we should back port this to 1.1.x as well.
-
jurgenhaas →
committed aa57acc4 on 1.1.x
Issue #3395591 by jurgenhaas: ECA Form: Invalid array key type array in...
-
jurgenhaas →
committed aa57acc4 on 1.1.x
- Status changed to Fixed
9 months ago 3:43pm 22 February 2024 - 🇩🇪Germany jurgenhaas Gottmadingen
Has been back ported and works in 1.1.x tests as well.
Automatically closed - issue fixed for 2 weeks with no activity.