FILE: /var/www/html/modules/contrib/extra_css_js/extra_css_js.module
------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 10 ERRORS AND 6 WARNINGS AFFECTING 10 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------------------
   5 | ERROR   | [x] Doc comment short description must end with a full stop
  12 | WARNING | [ ] All constants defined by a module must be prefixed with the module's name, expected "EXTRA_CSS_JS_EXTRA_CSS_PATH" but found "EXTRA_CSS_PATH"
  13 | WARNING | [ ] All constants defined by a module must be prefixed with the module's name, expected "EXTRA_CSS_JS_EXTRA_JS_PATH" but found "EXTRA_JS_PATH"
  37 | WARNING | [ ] Hook implementations should not duplicate @param documentation
  37 | ERROR   | [ ] Missing parameter comment
  37 | ERROR   | [ ] Missing parameter type
  38 | WARNING | [ ] Hook implementations should not duplicate @param documentation
  38 | ERROR   | [ ] Missing parameter comment
  75 | WARNING | [ ] Hook implementations should not duplicate @param documentation
  75 | ERROR   | [ ] Missing parameter comment
  75 | ERROR   | [ ] Missing parameter type
  76 | WARNING | [ ] Hook implementations should not duplicate @param documentation
  76 | ERROR   | [ ] Missing parameter comment
  97 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
  99 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
 100 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but found "true"
------------------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/modules/contrib/extra_css_js/extra_css_js.install
-------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-------------------------------------------------------------------------
 5 | ERROR | [x] Doc comment short description must end with a full stop
 8 | ERROR | [x] Missing function doc comment
-------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------


FILE: /var/www/html/modules/contrib/extra_css_js/src/Form/SettingsForm.php
------------------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 7 LINES
------------------------------------------------------------------------------------
 32 | ERROR | [x] Short array syntax must be used to define arrays
 36 | ERROR | [x] Space found before comma in argument list
 36 | ERROR | [x] Short array syntax must be used to define arrays
 38 | ERROR | [x] Short array syntax must be used to define arrays
 42 | ERROR | [x] Space found before comma in argument list
 42 | ERROR | [x] Short array syntax must be used to define arrays
 45 | ERROR | [x] Short array syntax must be used to define arrays
 71 | ERROR | [x] Expected 1 blank line after function; 0 found
 72 | ERROR | [x] The closing brace for the class must have an empty line before it
------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------

CommentFileSizeAuthor
#2 3155763-2.patch6.38 KBhimanshu_sindhwani

Comments

himanshu_sindhwani created an issue. See original summary.

himanshu_sindhwani’s picture

Status: Active » Needs review
StatusFileSize
new6.38 KB

Here is a patch.

sharma.amitt16’s picture

Status: Needs review » Reviewed & tested by the community

The patch fixing the coding standards reported in IS. Changing the status to RTBC.

himanshu_sindhwani’s picture

Thanks for the review. Committed

himanshu_sindhwani’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.