Cleanup plugin constructors

Created on 8 February 2023, over 1 year ago

Plugins should generally not use __construct() and should only use create() if they can. Adding constructors makes it hard for others to inherit the plugin base.
Every plugin needs the styleguide generator, so why not provide it in the base plugin class?
Plugins should use the provider annotation data instead of using the module handler. This will avoid loading plugins for modules that are not installed like comment, search, etc.
The Views mini pager styleguide item should be moved to its own plugin for Views. Generally we should be trying to reduce/remove moduleExists() checks inside the DefaultStyleguide class if we can.

📌 Task

Status

Fixed

Version

2.0

Component

Code

Created by

🇺🇸United States Dave Reid Nebraska 🇺🇸

Live updates comments and jobs are added and updated live.

Comments & Activities

Dave Reid → committed 6fc1a4ce on 2.x

Issue #3340357 by Dave Reid: Cleaned up plugin constructors to only use...

Status changed to Fixed over 1 year ago6:52pm 8 February 2023
Comment over 1 year ago →
🇺🇸United States Dave Reid Nebraska 🇺🇸
Merged to 2.x.
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.69.0 2024