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

Comments

Thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review
StatusFileSize
new ctools-drupal_dependency_injection-3013146.patch1.94 KB

Follow the patch!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
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
commented
Assigned: thalles » Unassigned

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
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
commented

Thanks @joelpittet!

Status: Fixed » Closed (fixed)

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