Modernize services: Add autowiring aliases, use autoconfigure, etc

We can also fix up https://drupal.org/node/3395436 for autoconfiguration, and remove use of ContainerInjectionInterface.

This also moves use of config out of constructor. Its by convention a config isnt stored from the constructor, its accessed from configfactory on demand as its state can change during the lifecycle of a request.

Left some minor remarks

Feedback and questions for maintainers. Nothing actionable by non-maintainers.

Thanks for helping modernize the module's codebase. I don't feel strongly about the readonly property and/or docblocks, so I'd say let's get this in and iterate over in 📌 Fix the issues reported by phpcs Needs review , where we can fix these and other CS violations that have bubbled up lately.

Thanks again!

Thanks @marcoscano, sounds reasonable to me.

Automatically closed - issue fixed for 2 weeks with no activity.