Fix the issues reported by phpcs

Created on 18 April 2023, about 1 year ago
Updated 28 February 2024, 4 months ago

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/optimized_assets_proxy.install
----------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
----------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/src/AmpCssCollectionRenderer.php
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AND 5 WARNINGS AFFECTING 9 LINES
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
9 | WARNING | [ ] Line exceeds 80 characters; contains 106 characters
23 | WARNING | [ ] Unused variable $key.
25 | WARNING | [ ] Unused variable $urls.
26 | WARNING | [ ] Unused variable $all_css.
28 | ERROR | [x] No space found before comment text; expected "// $css = file_get_contents(DRUPAL_ROOT . $url);" but found "//$css = file_get_contents(DRUPAL_ROOT .
| | $url);"
43 | ERROR | [x] list(...) is forbidden, use [...] instead.
43 | WARNING | [ ] Unused variable $query.
45 | ERROR | [x] No space found before comment text; expected "// $css = file_get_contents(DRUPAL_ROOT . $url);" but found "//$css = file_get_contents(DRUPAL_ROOT .
| | $url);"
54 | ERROR | [x] Expected 1 blank line after function; 0 found
55 | ERROR | [x] The closing brace for the class must have an empty line before it
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/src/AdvaggCssOptimizer.php
------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------
37 | ERROR | [x] Expected 1 blank line after function; 0 found
38 | ERROR | [x] The closing brace for the class must have an empty line before it
------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/src/OptimizedAssetsProxyServiceProvider.php
-----------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------------
7 | WARNING | [x] Unused use statement
36 | ERROR | [x] Expected 1 blank line after function; 0 found
37 | ERROR | [x] The closing brace for the class must have an empty line before it
-----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/src/EventSubscriber/MissingAsset.php
----------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AND 3 WARNINGS AFFECTING 12 LINES
----------------------------------------------------------------------------------------------------------------
15 | ERROR | [x] Missing class doc comment
45 | ERROR | [ ] Missing short description in doc comment
46 | ERROR | [ ] Missing parameter comment
47 | ERROR | [ ] Missing parameter comment
48 | ERROR | [ ] Missing parameter comment
49 | ERROR | [ ] Missing parameter comment
58 | ERROR | [x] Missing function doc comment
61 | WARNING | [ ] Line exceeds 80 characters; contains 91 characters
68 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
77 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
101 | ERROR | [x] Expected 1 blank line after function; 0 found
102 | ERROR | [x] The closing brace for the class must have an empty line before it
----------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/src/AdvaggJsOptimizer.php
-----------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------
37 | ERROR | [x] Expected 1 blank line after function; 0 found
38 | ERROR | [x] The closing brace for the class must have an empty line before it
-----------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/optimized_assets_proxy/optimized_assets_proxy.module
---------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------
1 | ERROR | [x] Missing file doc comment
27 | ERROR | [ ] Parameter tags must be grouped together in a doc comment
47 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
---------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------

๐Ÿ“Œ Task
Status

Needs work

Version

2.0

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡ณIndia himanshu_jhaloya Mandsaur

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

Merge Requests

Comments & Activities

Production build 0.69.0 2024