Validate that the term ID value is an array in taxonomy_index_tid views filter

Should this be reopened based on previous comments? Anyone able to replicate on D10?

I've tried to reproduce using the info in the IS on 10.2 and no luck, not seeing any warnings with this config. The steps in #12 are not clear enough to test them. If this is still occurring could someone please update the issue summary with specific steps to reproduce it?

Discussed in Slack, will create an MR with a modified fix.

Created an MR to add a check that $values is an array. There is no test here because it would be very difficult to come up with one, so hoping it qualifies for an exemption given both the simplicity of the fix and the fact that it is only resolving a warning.

pameeela → credited chx → .

Crediting @chx for Slack participation.

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Hiding patches

Wondering if we the summary could be updated with proposed solution from the slack channel.

The MR is pretty straightforward but I added a line about the proposed resolution. Not sure if that is what you mean @smustgrave?

Yup just because not everyone is in slack so any discussion there isn’t known

Meant to mark this a few days ago.

I read the issue summary, comments and the MR. I see there are no tests, but that is explained in #19. I have not evaluated that with the testing heuristics. A reminder to us all to summary Slack discussions in a comment. There is no indication here why $value is not an array or if that is really a contrib problem.

Also, trying for a better title. Leaving at RTBC.

Better title, I hope.