Problem/Motivation

MetatagToken has several calls to \Drupal which means it has a dependency on a bootstrapped Drupal container.

Proposed resolution

Inject dependencies.

Remaining tasks

Review

User interface changes

None

API changes

Change to constructor, but that's not part of the API.

Data model changes

None

CommentFileSizeAuthor
#2 metatag-token-inject-2593153.1.patch2.83 KBlarowlan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

larowlan created an issue. See original summary.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext commented
Status: Active » Needs review
FileSize
metatag-token-inject-2593153.1.patch2.83 KB
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2563605: Plan for Metatag 8.x-1.0-beta1 release

Lets get this into 1.0-beta1.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed. Thanks.

Status: Fixed » Closed (fixed)

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