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

smustgrave created an issue. See original summary.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.