Problem/Motivation

Before tagging beta1, without going as far as #3529494: Remove SlotSourceProxy, we can tidy slot_sources_proxy dependencies:

  display_builder.slot_sources_proxy:
    class: Drupal\display_builder\SlotSourceProxy
    arguments:
      - '@plugin.manager.ui_patterns_source'
      - '@entity.memory_cache'
      - '@plugin.manager.ui_patterns_source'
      - '@entity.memory_cache'

They have duplicate valeus (maybe a rebase error) and the service doesn't need memory_cache anymore:

  public function __construct(
    protected PluginManagerInterface $sourceManager,
  ) {}

Proposed resolution

Remove dependencies.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

pdureau created an issue. See original summary.

pdureau’s picture

Assigned: pdureau » mogtofu33
Status: Active » Needs review

Nice to have for beta1.

mogtofu33’s picture

Assigned: mogtofu33 » Unassigned
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • mogtofu33 committed a8fea932 on 1.0.x authored by pdureau
    chore: #3560331 Clean slot_sources_proxy dependencies
    
    By: pdureau
    

Status: Fixed » Closed (fixed)

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