Cleanup plugin constructors

Created on 8 February 2023, about 2 years 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 USA

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @dave reid
@dave-reid opened merge request.
@dave-reid opened merge request.
Comment about 2 years ago →
System Message
Dave Reid → closed merge request !10
Comment about 2 years ago →
System Message

Dave Reid → committed 6fc1a4ce on 2.x
Issue #3340357 by Dave Reid: Cleaned up plugin constructors to only use...
Status changed to Fixed about 2 years ago6:52pm 8 February 2023
Comment about 2 years ago →
🇺🇸United States dave reid Nebraska USA
Merged to 2.x.
Comment about 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024