Problem/Motivation

Each plugin gets an instance with \Drupal::database(). It would be good to inject this into the plugin base for ease of use for future plugin types as well.

Blocked behind #2684771: Update LinkGeneratorInterface to enforce what link generators need to provide..

Proposed resolution

Inject it.

Remaining tasks

Patch.

User interface changes

None.

API changes

Added protected properties for future link providers.

Data model changes

None.

CommentFileSizeAuthor
#3 2684781-injected-database-3.patch3.92 KBSam152
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152 created an issue. See original summary.

gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented

Hey Sam, I can tell you have a plan. ;)
I fixed that blocking issue so you can have a go at this.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Active » Needs review
FileSize
2684781-injected-database-3.patch3.92 KB
gbyte’s picture

gbyte CreditAttribution: gbyte as a volunteer and commented
Status: Needs review » Closed (fixed)

Pushed to 2.x unchanged. Thanks!