\Drupal calls should be avoided in classes, use dependency injection instead

I am working on this and will provide an MR.

Hi sidharth_soman

I applied your MR to the “Sitemap Status” module and confirmed that the phpcs issue mentioned is resolved in that file, but the error appears also in SettingsForm.php maybe fix it also.

FILE: /Users/studenttrainees/Project10/drupalorg10/docroot/modules/contrib/sitemap_status/src/Form/SettingsForm.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 7 WARNINGS AFFECTING 8 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
12 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
70 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
168 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
248 | WARNING | [x] There must be no blank line following an inline comment
248 | WARNING | [ ] There must be no blank line following an inline comment
252 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
286 | ERROR | [ ] Missing parameter comment
287 | ERROR | [ ] Missing parameter comment
293 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY

Please see the screenshots attached.

For your review.
Thank you.

Ashutosh Ahirwal → made their first commit to this issue’s fork.

Created MR5 with phpcs issue fixes.