Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.\Drupal calls should be avoided in classes, use dependency injection instead in core/modules/user/src/AccountForm.php class, core/modules/user/src/Access/RegisterAccessCheck.php class and core/modules/user/src/Plugin/Block/UserLoginBlock.php
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 3106228-7.patch | 6.91 KB | hardik_patel_12 |
Comments
Comment #2
hardik_patel_12 commentedKindly review a patch.
Comment #3
hardik_patel_12 commentedComment #4
hardik_patel_12 commentedComment #5
hardik_patel_12 commentedcombining 2 issues #3106241 : \Drupal calls should be avoided in classes, use dependency injection instead in AccountForm.php and #3106232 : \Drupal calls should be avoided in classes, use dependency injection instead in RegisterAccessCheck.php file
Comment #6
hardik_patel_12 commentedComment #7
hardik_patel_12 commentedKindly review a new patch.
Comment #8
hardik_patel_12 commentedComment #9
hardik_patel_12 commentedComment #10
xjmThank you for filing this and for your contributions so far.
I am closing this as a duplicate of #2729597: [meta] Replace \Drupal with injected services where appropriate in core. You can use the work in the patches from this and similar issues as starting points for new issues scoped by concept.