- Issue created by @berdir
- Merge request !7806Refactor getTitleAsStringable() to check both sources for an array β (Open) created by berdir
- Status changed to Needs review
8 months ago 9:54am 28 April 2024 - π¨πSwitzerland berdir Switzerland
Pushed a fix, not sure about tests here. This is a pretty severe regression from 10.2, completely breaks a module like moderation_sidebar and the new implementation is quite a bit simpler (and also a bit faster, since it avoids calling getTitle() twice.
- π¨πSwitzerland berdir Switzerland
Looking at the changes again, the previous implementation explicitly supported #title as NULL, I can't really imagine how that would be desirable and there don't seem to be any tests for that? But it would be easier to expand the ?? to an if/else again if we want to keep that.
- π¬π§United Kingdom catch
I think we should have test coverage for the bug, but also I think we might want to open a dedicated issue for that and fix the regression before 10.3.0 beta so that various contrib aren't stuck with an unresolvable blocker. Bumping to critical and tagging as beta target.
The other option would be rolling back the original issue, but I didn't review the original change there to see what that would look like, except that it needed a CR so might not be easy to get back into 10.3.x again.
- Status changed to RTBC
8 months ago 1:47pm 6 May 2024 - πΊπΈUnited States smustgrave
Opened π Add additional test coverage for DialogRenderer::getTitleAsStringable() Active for the test coverage to not hold up 10.3.
- Status changed to Fixed
8 months ago 8:14pm 6 May 2024 - π¬π§United Kingdom catch
Committed/pushed to 11.x and cherry-picked to 11.0.x, 10.4.x, 10.3.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.