- Issue created by @b_sharpe
- 🇦🇹Austria klausi 🇦🇹 Vienna
Closing this as duplicate - let's continue in 🐛 GlobalDrupalSniff's use of $baseClasses is insufficient and fails to detect relevant cases Needs review . Thanks!
Currently the following must be true to catch \Drupal:: calls in DI classes:
Class doesn't extend anything OR Doesn't extend one of those defined as base classes
Is not a service itself
Doesn't implement ContainerInjectionInterface
So this omits a lot of legit cases of plugins as they commonly will use ContainerFactoryPluginInterface
Have better detection of container interfaces
Active
8.3
Coder Sniffer
Closing this as duplicate - let's continue in 🐛 GlobalDrupalSniff's use of $baseClasses is insufficient and fails to detect relevant cases Needs review . Thanks!