IMHO, is better use Dependency Injection.

CommentFileSizeAuthor
DI.patch2.48 KBrpayanm

Comments

rpayanm created an issue. See original summary.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Active » Needs review
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs review » Reviewed & tested by the community

Tested your patch and it's working as expected.Moving to RTBC.

BarisW’s picture

BarisW commented

Thanks for the patch and the review!

  • BarisW committed 1c05a6a on 8.x-1.x authored by rpayanm 
    Issue #2900155 by rpayanm, nkoporec, BarisW: Dependency injection on...
BarisW’s picture

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

Status: Fixed » Closed (fixed)

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