Problem/Motivation
Follow up from #2852463: Create a trait and base class to implement \Drupal\Component\Plugin\ConfigurableInterface. from @benjifisher around core/modules/image/src/ImageEffectBase.php
eliminate the constructor and set the $logger property in the create() method?
One comment following was @megachriz
I guess we can, but why would this be needed? And wouldn't that cause issues for classes that extend ImageEffectBase and override the constructor now? In other words: I think removing the $logger property would be a BC break.
Steps to reproduce
N/A
Proposed resolution
Determine if anything can be done
Remaining tasks
Research
If can be removed do it
Else close this out
User interface changes
N/A
Introduced terminology
N/A
API changes
TBD
Data model changes
N/A
Release notes snippet
N/A
Comments