- Issue created by @slucero
- Merge request !136[#3482299] Simplify Pattern Library Plugin Manager Implementation → (Merged) created by slucero
Automatically closed - issue fixed for 2 weeks with no activity.
The PatternLibraryPluginManager service was built with more robust functionality in mind, but in active use it doesn't need the definition filtering and categorization features it currently includes. There is also custom logic in place both within this class and in other uses calling the createInstance() method to support fallback to a less-specific plugin ID like "image.svg" to "image" if "image.svg" is unavailable. The standard solution for this in Drupal core, however, is to implement the \Drupal\Component\Plugin\FallbackPluginManagerInterface. This behavior should be consolidated into just the plugin manager class using this model.
PatternLibraryPluginDefinition classNone
PatternLibraryPluginDefinition class is removed\Drupal\patternkit\Attribute\PatternLibrary::get() and \Drupal\patternkit\Annotation\PatternLibrary::get() now return an array for the plugin definition instead of a PatternLibraryPluginDefinition instancePatternLibraryPluginManager no longer implements CategorizingPluginManagerInterfacePatternLibraryPluginManager no longer uses FilteredPluginManagerTrait or CategorizingPluginManagerTraitPatternLibraryPluginManager no longer provides getLayoutOptions() methodPatternLibraryPluginManager no longer provides getLayoutOptions() methodPatternLibraryPluginManager now implements \Drupal\Component\Plugin\FallbackPluginManagerInterfaceNone
Active
9.1
Module Core
Automatically closed - issue fixed for 2 weeks with no activity.