Use dependency injection

Issue created by @Liam Morland
Assigned to atul_ghate
Comment 9 months ago →
🇮🇳India atul_ghate
I will work on this issue.
Issue was unassigned.
Status changed to Needs review 9 months ago5:45pm 6 January 2024
Comment 9 months ago →
🇮🇳India atul_ghate
please review the patch.
Comment 9 months ago →
🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Thanks for the patch. Please use constructor property promotion to shorten the code.
Assigned to atul_ghate
Comment 9 months ago →
🇮🇳India atul_ghate
okay will update updated patch.
Issue was unassigned.
Comment 9 months ago →
🇮🇳India atul_ghate
please review updated patch.
Merge request !5Resolve #3412586 "Use dependency injection" → (Merged) created by Liam Morland
Pipeline finished with Success
5 months ago
Total: 168s
#171975
Status changed to RTBC 5 months ago2:14pm 14 May 2024
Comment 5 months ago →
🇨🇦Canada smulvih2 Canada 🍁
Looks good to me, merged into 1.1.x

Liam Morland → committed 1de4cf23 on 1.1.x

Issue #3412586 by Liam Morland: Replace calls to \Drupal with dependency...

Comment 5 months ago →
System Message

Liam Morland → committed 036ea9f5 on 1.1.x
Issue #3412586 by Liam Morland: Replace calls to \Drupal with calls to...
Status changed to Fixed 5 months ago2:37pm 14 May 2024
Comment 5 months ago →
🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Comment 5 months ago →
🇨🇦Canada smulvih2 Canada 🍁
Changes are now available in 1.1.4-rc1
Comment 4 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Merge Requests