- ๐บ๐ธUnited States smustgrave
Should this be reopened based on previous comments? Anyone able to replicate on D10?
- ๐ฆ๐บAustralia pameeela
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?
- Issue was unassigned.
- Status changed to Active
2 months ago 9:09pm 9 July 2024 - ๐ฆ๐บAustralia pameeela
Discussed in Slack, will create an MR with a modified fix.
- Status changed to Needs review
2 months ago 9:13pm 9 July 2024 - ๐ฆ๐บAustralia pameeela
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.
- Status changed to Needs work
2 months ago 9:25pm 9 July 2024 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.)
- Status changed to Needs review
2 months ago 11:54pm 9 July 2024 - Status changed to Needs work
about 2 months ago 10:35pm 10 July 2024 - ๐บ๐ธUnited States smustgrave
Wondering if we the summary could be updated with proposed solution from the slack channel.
- Status changed to Needs review
about 2 months ago 8:53pm 11 July 2024 - ๐ฆ๐บAustralia pameeela
The MR is pretty straightforward but I added a line about the proposed resolution. Not sure if that is what you mean @smustgrave?
- ๐บ๐ธUnited States smustgrave
Yup just because not everyone is in slack so any discussion there isnโt known
- Status changed to RTBC
about 2 months ago 1:10pm 15 July 2024 - ๐ณ๐ฟNew Zealand quietone New Zealand
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.