Hi, congratulations for the module.

Problem/Motivation

Drupal and DrupalPractice asked for dependency injection

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 invite-dependency_injection-3017758.patch1.95 KBthalles

Comments

thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
StatusFileSize
new invite-dependency_injection-3017758.patch1.95 KB
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review
kishan.thakur’s picture

kishan.thakur commented

Use Dependency Injection for using any core service for this you have to inject service as argument and also see https://www.drupal.org/docs/8/api/services-and-dependency-injection/serv...

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

But, this is not a service!

renatog’s picture

renatog commented
Status: Needs review » Reviewed & tested by the community

The patch works good

  • RenatoG committed 440ecb3 on 8.x-1.x authored by thalles 
    Issue #3017758 by thalles, dev.krishna, RenatoG: Drupal dependency...
renatog’s picture

renatog commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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