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

Created on 18 July 2023, over 1 year ago
Updated 1 August 2023, over 1 year ago

Problem/Motivation

FILE: /home/lenovo/d7to9/web/modules/contrib/safedelete/src/Form/SafedeleteSettingsForm.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 11 WARNINGS AFFECTING 11 LINES
------------------------------------------------------------------------------------------------------------------------------------
  41 | WARNING | [ ] NodeType::loadMultiple calls should be avoided in classes, use dependency injection instead
 173 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 174 | WARNING | [ ] NodeType::loadMultiple calls should be avoided in classes, use dependency injection instead
 185 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 190 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 195 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 201 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 222 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 224 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 228 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 232 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------------------------

FILE: /home/lenovo/d7to9/web/modules/contrib/safedelete/src/Controller/SearchingOrphanedPageController.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 10 WARNINGS AFFECTING 10 LINES
------------------------------------------------------------------------------------------------------------------------------------
  48 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  49 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  50 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  57 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  62 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  81 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  83 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  87 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
  96 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 127 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------------------------

Steps to reproduce

Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig safedelete/

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇮🇳India roshni27

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024