Fix the issues reported by phpcs

Created on 5 April 2023, over 1 year ago
Updated 20 April 2023, over 1 year ago

Problem/Motivation

Getting following error/warnings.

FILE: /var/www/html/modules/contrib/cookie_consent_notice_by_cookieyes/cookieyes_scripts.module
-------------------------------------------------------------------------------------------------------------------------------
FOUND 8 ERRORS AND 5 WARNINGS AFFECTING 8 LINES
-------------------------------------------------------------------------------------------------------------------------------
28 | WARNING | [ ] Line exceeds 80 characters; contains 93 characters
34 | WARNING | [x] There must be no blank line following an inline comment
34 | WARNING | [ ] There must be no blank line following an inline comment
34 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
36 | ERROR | [x] Expected 1 space after IF keyword; 0 found
36 | ERROR | [x] Expected 1 space after closing parenthesis; found 0
37 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 6
39 | ERROR | [x] Whitespace found at end of line
40 | WARNING | [x] There must be no blank line following an inline comment
40 | WARNING | [ ] There must be no blank line following an inline comment
40 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
45 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 5
106 | ERROR | [x] Expected 1 newline at end of file; 2 found
-------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 10 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/modules/contrib/cookie_consent_notice_by_cookieyes/cookieyes_scripts.links.task.yml
-------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------
4 | ERROR | [x] Expected 1 newline at end of file; 0 found
-------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------

FILE: /var/www/html/modules/contrib/cookie_consent_notice_by_cookieyes/cookieyes_scripts.routing.yml
----------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
7 | ERROR | [x] Expected 1 newline at end of file; 0 found
----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------

FILE: /var/www/html/modules/contrib/cookie_consent_notice_by_cookieyes/cookieyes_scripts.install
------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------
25 | ERROR | [x] Equals sign not aligned correctly; expected 1 space but found 3 spaces
36 | ERROR | [x] Functions must not contain multiple empty lines in a row; found 2 empty lines
------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------

FILE: /var/www/html/modules/contrib/cookie_consent_notice_by_cookieyes/src/Form/BodyForm.php
---------------------------------------------------------------------------------------------
FOUND 5 ERRORS AND 1 WARNING AFFECTING 4 LINES
---------------------------------------------------------------------------------------------
7 | ERROR | [x] There must be one blank line after the last USE statement; 2 found;
10 | ERROR | [x] Missing class doc comment
41 | ERROR | [x] Use null coalesce operator instead of ternary operator.
41 | WARNING | [x] A comma should follow the last multiline array item. Found: true
41 | ERROR | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
47 | ERROR | [x] Use null coalesce operator instead of ternary operator.
---------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------

Time: 827ms; Memory: 6MB

Steps to reproduce

Run following command

phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml modules/contrib/cookie_consent_notice_by_cookieyes/

Proposed resolution

Above error/warnings need to be fixed.

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇮🇳India samit.310@gmail.com

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.71.5 2024