- 🇺🇸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.
If still a valid bug in D10 it will need a test case to show it please
Thanks!
- 🇬🇧United Kingdom catch
I think we can skip tests here, there's not really a bug as such, we're just normalizing something to lowercase. To write test coverage we'd need to know PECL is always including upper case characters when it generates a UUID, but it might not every time so the test could easily have false negatives.
- First commit to issue fork.
- Status changed to Needs review
5 months ago 4:15am 3 August 2024 - Status changed to RTBC
5 months ago 12:36am 4 August 2024 - 🇺🇸United States smustgrave
Seems straight forward and believe small enough to that doesn't warrant test coverage.
- Status changed to Fixed
5 months ago 9:28am 9 August 2024 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.