Fix the issues reported by PHPCS

Created on 13 April 2023, about 1 year ago
Updated 27 April 2023, about 1 year ago

Problem/Motivation

FILE: ...web/modules/contrib/entity_reference_delete/entity_reference_delete.module
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
68 | WARNING | [ ] Line exceeds 80 characters; contains 109 characters
102 | WARNING | [ ] Line exceeds 80 characters; contains 106 characters
165 | WARNING | [ ] Unused variable $bundle.
194 | WARNING | [x] There must be no blank line following an inline comment
194 | WARNING | [ ] There must be no blank line following an inline comment
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ..._reference_delete/tests/src/Kernel/EntityReferenceDeleteBaseFieldsTest.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
5 | WARNING | [x] Unused use statement
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: .../entity_reference_delete/src/Plugin/QueueWorker/ParentQueryQueueWorker.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
17 | WARNING | Line exceeds 80 characters; contains 84 characters
--------------------------------------------------------------------------------

FILE: ...tion/drupal10/web/modules/contrib/entity_reference_delete/src/FieldMap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
--------------------------------------------------------------------------------
93 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
184 | WARNING | There must be no blank line following an inline comment
186 | WARNING | There must be no blank line following an inline comment
218 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
251 | WARNING | Unused variable $settings.
254 | WARNING | Unused variable $settings.
--------------------------------------------------------------------------------

Time: 993ms; Memory: 10MB

Steps to reproduce

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

Proposed resolution

Resolve all issues for Drupal and DrupalPractice coding standards

Remaining tasks

Patch Review and fix the remaining issues
FILE: ...tion/drupal10/web/modules/contrib/entity_reference_delete/src/FieldMap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
93 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
216 | WARNING | \Drupal calls should be avoided in classes, use dependency
| | injection instead
249 | WARNING | Unused variable $settings.
252 | WARNING | Unused variable $settings.
--------------------------------------------------------------------------------

Time: 804ms; Memory: 10MB

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇮🇳India urvashi_vora Madhya Pradesh, India

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Comments & Activities

Production build 0.69.0 2024