- 🇺🇸United States smustgrave
Appears to be open thread in MR, proposed solution should be tweaked some too.
- 🇮🇳India shalini_jha
I Have addressed the feedback also checked the https://3v4l.org/UuKsj and re-ran the test and its is working as expected. pipeline is also green so moving this NR.
Kindly review. - First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇳🇱Netherlands eelkeblok Netherlands 🇳🇱
That last commit was me. I did it through the web interface, so that's probably the reason it doesn't properly show up.
From my limited testing, I noticed that only when the field is empty, the value will indeed be an array. I think the solution from the previous patch is than in fact not quite appropriate; it just takes one of the empty values and passes it through the test of the code. Instead, I changed it to treat it similarly to a NULL value; just bail out when it is an array.
Still needs a test, of course :)
- First commit to issue fork.