- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Did not test for issue FYI
But this will need a test case to show the issue and that the fix address it.
Thanks!
- 🇳🇿New Zealand quietone New Zealand
I tested this on Drupal 10.1.x and was able to reproduce the problem.
This is a duplicate of the earlier issue, but let's keep this one as it has more discussion and a patch. I have copied the Issue Summary from that issue because pameeela did a great job of summarizing the issue. This still needs tests.
- 🇮🇳India mohit_aghera Rajkot
Adding a test case and test only patch.
Note regarding new test-view added:
I tried to modify existing views by adding filter programmatically similar to `FilterBooleanOperatorStringTest`
However, it was not working as expected. I tried with different views from node_test_views module's test views.
It didn't work as expected.
So at last, I ended up creating seperate view for testing purposes. - Status changed to Needs review
about 1 year ago 3:46pm 13 April 2023 The last submitted patch, 19: test-only-3073518-19.patch, failed testing. View results →
- Status changed to RTBC
about 1 year ago 12:04am 16 April 2023 - 🇺🇸United States smustgrave
Tested following the issue summary steps and confirmed the issue.
Patch #19 addressed it for me.Red/green shows good test coverage.
Good job @mohit_aghera with the tests!
- last update
about 1 year ago 29,204 pass - Status changed to Fixed
about 1 year ago 7:54am 17 April 2023 - 🇬🇧United Kingdom catch
Committed/pushed to 10.1.x, cherry-picked to 10.0.x and 9.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
3 months ago 6:26am 22 March 2024