- Issue created by @sidharth_soman
- Assigned to sidharth_soman
- Open on Drupal.org →Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8last update
almost 2 years ago Waiting for branch to pass - @sidharth_soman opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 10:35am 10 May 2023 - 🇩🇪Germany Anybody Porta Westfalica
@Grevil: Could you please review this?
- Status changed to Needs work
almost 2 years ago 8:14am 12 May 2023 - 🇩🇪Germany Grevil
We do not need to inject these services, the parent already has them injected.
- Open on Drupal.org →Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8last update
almost 2 years ago Not currently mergeable. - Open on Drupal.org →Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8last update
almost 2 years ago Not currently mergeable. - last update
almost 2 years ago 56 pass, 1 fail - last update
almost 2 years ago 82 pass - Status changed to Needs review
almost 2 years ago 8:48am 12 May 2023 - 🇩🇪Germany Grevil
OK, my apologies, the original patch was partially correct! We just don't need to inject the messenger service. :)
Please review!
- 🇩🇪Germany Anybody Porta Westfalica
Great work, thank you! RTBC and merged!
- last update
almost 2 years ago 82 pass -
Anybody →
committed 8aab97bb on 4.x authored by
sidharth_soman →
Issue #3359280 by Grevil, Anybody: \Drupal calls should be avoided in...
-
Anybody →
committed 8aab97bb on 4.x authored by
sidharth_soman →
- Status changed to Fixed
almost 2 years ago 9:00am 12 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.