- Issue created by @joehuggans
-
timohuisman β
committed a6a3939e on 8.x-2.x authored by
Joe Huggans β
Issue #3357040 by Joe Huggans: It would be useful to see what the...
-
timohuisman β
committed a6a3939e on 8.x-2.x authored by
Joe Huggans β
- Status changed to Fixed
about 1 year ago 1:30pm 22 November 2023 - Status changed to Needs work
about 1 year ago 11:20am 23 November 2023 - πΊπ¦Ukraine AstonVictor
It's better to pass
\Drupal::entityTypeManager()
as an argument to the service and use dependency injection. - Open on Drupal.org βCore: 9.5.x + Environment: PHP 7.4 & MySQL 5.7last update
12 months ago Waiting for branch to pass - Status changed to Needs review
12 months ago 8:06am 28 November 2023 - Open on Drupal.org βCore: 9.5.x + Environment: PHP 7.4 & MySQL 5.7last update
12 months ago Waiting for branch to pass - πΊπ¦Ukraine AstonVictor
Created a new MR.
Added a new argument to the service.
- First commit to issue fork.
-
bbrala β
committed e7405e12 on 3.x authored by
AstonVictor β
Issue #3357040 by AstonVictor, Joe Huggans, timohuisman: It would be...
-
bbrala β
committed e7405e12 on 3.x authored by
AstonVictor β
- Status changed to Fixed
11 months ago 1:52pm 3 January 2024 - π³π±Netherlands bbrala Netherlands
Great work, thank you, dependency injection is a good idea. It could've been a new issue, but this is fine. :)
Automatically closed - issue fixed for 2 weeks with no activity.