- Issue created by @longwave
- First commit to issue fork.
- Merge request !7200Resolve #3436243: DisplayRouterInterface documents an unused 'uses_routes' property → (Closed) created by jackwrfuller
- Status changed to Needs review
12 months ago 12:39am 27 March 2024 - Status changed to RTBC
12 months ago 11:53am 27 March 2024 - Status changed to Needs work
12 months ago 1:11pm 27 March 2024 - 🇬🇧United Kingdom longwave UK
@alexpott ah did not spot that at all - do you think we should deprecate these Views flags and
Views::getApplicableViews()
and replace with interfaces? - 🇬🇧United Kingdom alexpott 🇪🇺🌍
@longwave well we'd need to create instances of the plugins for that - I guess it will work. Having both the interface and the flag is fun and v drupal.
- 🇬🇧United Kingdom longwave UK
@alexpott no we can determine interfaces from the class name alone with
is_a()
. - First commit to issue fork.
- 🇧🇷Brazil charlliequadros
Hi @alexpott, @longwave,
I re-added the comment, fixed the typo, and submitted it for review again. - 🇮🇳India adwivedi008
Hello @charlliequadros
I have checked the MR and it seems that the suggested changes have been implementedMoving the issue to RTBC and let's wait for some more feedback from @alexpott or @longwave,
- 🇫🇷France nod_ Lille
There is a phpcs failure, needs to be fixed before I can commit