Problem/Motivation

Drupal and DrupalPractice asked dependency injection on ContextConfigure

Proposed resolution

inject Drupal\Core\Entity\EntityTypeManagerInterface

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 ctools-drupal_dependency_injection-3013146.patch1.94 KBthalles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented
Status: Needs work » Needs review
FileSize
ctools-drupal_dependency_injection-3013146.patch1.94 KB

Follow the patch!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented
Parent issue: #3013138: Replace \Drupal::state() with injected Drupal\Core\State\State »
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented
Assigned: thalles » Unassigned

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Needs review » Fixed

Thanks @Thalles, committed to the latest dev release.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented

Thanks @joelpittet!

Status: Fixed » Closed (fixed)

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