Fix the issues reported by PHP_CodeSniffer

Created on 21 September 2024, 2 months ago

Problem/Motivation


FILE: lb_related_articles_blocks\assets\css\lb-related-articles-block.css
------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
------------------------------------------------------------------------------------------------------------

FILE: lb_related_articles_blocks\assets\css\lb-related-articles-node.css
-----------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
-----------------------------------------------------------------------------------------------------------

FILE: lb_related_articles_blocks\lb_related_articles_blocks.info.yml
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically
1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically
-------------------------------------------------------------------------------------------------------------

FILE: lb_related_articles_blocks\lb_related_articles_blocks.install
------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
------------------------------------------------------------------------------------------------------
5 | ERROR | [x] Function comment short description must start with exactly one space
9 | ERROR | [x] There must be one blank line after the last USE statement; 2 found;
15 | ERROR | [x] Expected 1 blank line before function; 2 found
33 | WARNING | [x] A comma should follow the last multiline array item. Found: '0'
69 | ERROR | [x] Doc comment short description must end with a full stop
------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------

FILE: lb_related_articles_blocks\lb_related_articles_blocks.module
-----------------------------------------------------------------------------------------------------
FOUND 18 ERRORS AFFECTING 18 LINES
-----------------------------------------------------------------------------------------------------
174 | ERROR | [x] The first index in a multi-value array must be on a new line
176 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
177 | ERROR | [x] The first index in a multi-value array must be on a new line
179 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
191 | ERROR | [x] The first index in a multi-value array must be on a new line
193 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
194 | ERROR | [x] The first index in a multi-value array must be on a new line
196 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
200 | ERROR | [x] Expected 1 blank line after function; 0 found
207 | ERROR | [x] The first index in a multi-value array must be on a new line
209 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
210 | ERROR | [x] The first index in a multi-value array must be on a new line
212 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
217 | ERROR | [x] Expected 1 blank line after function; 0 found
224 | ERROR | [x] The first index in a multi-value array must be on a new line
226 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
227 | ERROR | [x] The first index in a multi-value array must be on a new line
229 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
-----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 18 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------

Steps to reproduce

Run phpcs command to get the drupal coding standard issues.

Proposed resolution

Many of the issues can be fixed by running the phpcbf command. For the remaining ones, we will need to manually address and then commit these changes before pushing them.

πŸ“Œ Task
Status

Needs review

Version

1.2

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States sunil jolapara 3265 Summitrun Drive, Independence, KY, 41051

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

Merge Requests

Comments & Activities

Production build 0.71.5 2024