The service decorator Drupal\ui_patterns\ComponentPluginManager is incorrectly implemented.
It extends Drupal\Core\Theme\ComponentPluginManager and calls parent:: methods instead of using the decorated service.
This bypasses Drupal’s service decoration mechanism, so additional decorators or alternative implementations of component_plugin_manager are ignored.
The ComponentPluginManager in ui_patterns should call the decorated service, not the parent class.
// Wrong (current implementation):
parent::processDefinition($definition, $plugin_id);
// Correct (expected behavior):
$this->decorated->processDefinition($definition, $plugin_id);
component_plugin_manager service via the constructor.parent:: calls with calls to $this->decorated.<?php
use Drupal\Core\Theme\ComponentPluginManagerInterface;
class ComponentPluginManager implements ComponentPluginManagerInterface {
  protected ComponentPluginManagerInterface $decorated;
  public function __construct(ComponentPluginManagerInterface $decorated) {
    $this->decorated = $decorated;
  }
  public function processDefinition(&$definition, $plugin_id) {
    // Delegate to the decorated service.
    $this->decorated->processDefinition($definition, $plugin_id);
    // Add ui_patterns-specific logic here if needed.
  }
  // Delegate other methods likewise...
}
component_plugin_manager.
Drupal\ui_patterns\ComponentPluginManager should delegate to the decorated service, not the parent class, to correctly follow Drupal’s service decoration pattern and maintain extensibility.
Needs review
2.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
No activities found.