Please check and fix the following issues raised by code sniffer.

See https://www.drupal.org/docs/develop/standards for more info.

FILE: health_check_url/config/install/health_check_url.settings.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: health_check_url/health_check_url.module
--------------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 7 LINES
--------------------------------------------------------------------------------
  1 | ERROR | [x] Missing file doc comment
  3 | ERROR | [x] There must be one blank line after the last USE statement; 2
    |       |     found;
  9 | ERROR | [x] Expected 1 blank line before function; 2 found
 17 | ERROR | [x] Whitespace found at end of line
 19 | ERROR | [x] There should be no white space after an opening "["
 20 | ERROR | [x] Whitespace found at end of line
 28 | ERROR | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: health_check_url/health_check_url.permissions.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: health_check_url/health_check_url.info.yml
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
--------------------------------------------------------------------------------
 1 | WARNING | Remove "project" 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: health_check_url/README.txt
------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------
  3 | WARNING | [ ] Line exceeds 80 characters; contains 151 characters
  4 | WARNING | [ ] Line exceeds 80 characters; contains 205 characters
 11 | ERROR   | [x] Expected 1 newline at end of file; 3 found
------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------


FILE: health_check_url/src/Form/HealthCheckSettingsForm.php
--------------------------------------------------------------------------------
FOUND 56 ERRORS AND 11 WARNINGS AFFECTING 50 LINES
--------------------------------------------------------------------------------
   1 | ERROR   | [x] The PHP open tag must be followed by exactly one blank
     |         |     line
   2 | ERROR   | [x] Namespaced classes, interfaces and traits should not begin
     |         |     with a file doc comment
   4 | ERROR   | [x] Doc comment short description must end with a full stop
   6 | ERROR   | [x] There must be one blank line after the namespace
     |         |     declaration
   9 | WARNING | [x] Unused use statement
   9 | ERROR   | [x] There must be one blank line after the last USE statement;
     |         |     0 found;
  14 | ERROR   | [x] Whitespace found at end of line
  17 | ERROR   | [x] Expected 1 blank line before function; 0 found
  19 | ERROR   | [x] Expected 1 blank line after function; 0 found
  20 | ERROR   | [x] Whitespace found at end of line
  27 | ERROR   | [x] Expected 1 blank line after function; 0 found
  28 | ERROR   | [x] Whitespace found at end of line
  33 | ERROR   | [x] The first index in a multi-value array must be on a new
     |         |     line
  33 | ERROR   | [x] Whitespace found at end of line
  34 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
  34 | ERROR   | [x] Expected 1 space after "=>"; 2 found
  34 | ERROR   | [x] Whitespace found at end of line
  35 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
  35 | ERROR   | [x] Concat operator must be surrounded by a single space
  36 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
  36 | ERROR   | [x] Concat operator must be surrounded by a single space
  37 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
  37 | ERROR   | [x] Concat operator must be surrounded by a single space
  38 | ERROR   | [x] Array closing indentation error, expected 4 spaces but
     |         |     found 15
  39 | ERROR   | [x] Short array syntax must be used to define arrays
  40 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  40 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  41 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  42 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  43 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  46 | ERROR   | [x] Short array syntax must be used to define arrays
  47 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  47 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  48 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  49 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  49 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  50 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  52 | ERROR   | [x] Whitespace found at end of line
  53 | ERROR   | [x] Short array syntax must be used to define arrays
  54 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  54 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  55 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  56 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  56 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  56 | WARNING | [ ] Translatable strings must not begin or end with white
     |         |     spaces, use placeholders with t() for variables
  57 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  60 | ERROR   | [x] Short array syntax must be used to define arrays
  61 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  61 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  62 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  63 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  63 | WARNING | [ ] t() calls should be avoided in classes, use dependency
     |         |     injection and $this->t() instead
  63 | WARNING | [ ] Translatable strings must not begin or end with white
     |         |     spaces, use placeholders with t() for variables
  64 | ERROR   | [x] Array indentation error, expected 6 spaces but found 4
  67 | ERROR   | [x] Expected 1 blank line after function; 2 found
  70 | ERROR   | [x] Line indented incorrectly; expected 2 spaces, found 3
  71 | ERROR   | [x] Expected 4 space(s) before asterisk; 3 found
  72 | ERROR   | [x] Expected 4 space(s) before asterisk; 3 found
  81 | ERROR   | [x] Whitespace found at end of line
  87 | ERROR   | [x] Whitespace found at end of line
  93 | ERROR   | [x] Expected one space after the comma, 0 found
  94 | ERROR   | [x] Expected one space after the comma, 0 found
  95 | ERROR   | [x] Expected one space after the comma, 0 found
  95 | ERROR   | [x] Whitespace found at end of line
  97 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency
     |         |     injection instead
  99 | ERROR   | [x] Expected 1 blank line after function; 0 found
 100 | ERROR   | [x] The closing brace for the class must have an empty line
     |         |     before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 57 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: health_check_url/src/Controller/HealthCheckController.php
--------------------------------------------------------------------------------
FOUND 27 ERRORS AND 1 WARNING AFFECTING 21 LINES
--------------------------------------------------------------------------------
 23 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency
    |         |     injection instead
 26 | ERROR   | [x] Expected 1 blank line after function; 3 found
 28 | ERROR   | [x] Whitespace found at end of line
 31 | ERROR   | [x] Doc comment short description must start with a capital
    |         |     letter
 31 | ERROR   | [x] Doc comment short description must end with a full stop
 33 | ERROR   | [ ] Description for the @return value is missing
 36 | ERROR   | [x] The first index in a multi-value array must be on a new
    |         |     line
 36 | ERROR   | [x] Whitespace found at end of line
 37 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
 37 | ERROR   | [x] Expected 1 space after "=>"; 2 found
 37 | ERROR   | [x] Whitespace found at end of line
 38 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
 38 | ERROR   | [x] Concat operator must be surrounded by a single space
 39 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
 39 | ERROR   | [x] Concat operator must be surrounded by a single space
 40 | ERROR   | [x] Array indentation error, expected 6 spaces but found 16
 40 | ERROR   | [x] Concat operator must be surrounded by a single space
 41 | ERROR   | [x] Array closing indentation error, expected 4 spaces but
    |         |     found 15
 43 | ERROR   | [x] Expected 1 blank line after function; 2 found
 45 | ERROR   | [x] Whitespace found at end of line
 47 | ERROR   | [x] Doc comment short description must end with a full stop
 49 | ERROR   | [ ] Description for the @return value is missing
 52 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 5
 53 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 5
 54 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 5
 55 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 5
 56 | ERROR   | [x] Expected 1 blank line after function; 0 found
 57 | ERROR   | [x] The closing brace for the class must have an empty line
    |         |     before it
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 25 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------


FILE: health_check_url/src/Routing/RouteService.php
--------------------------------------------------------------------------------
FOUND 7 ERRORS AND 2 WARNINGS AFFECTING 9 LINES
--------------------------------------------------------------------------------
  9 | ERROR   | [x] Missing class doc comment
 18 | ERROR   | [ ] Missing short description in doc comment
 32 | ERROR   | [x] Expected one space after the comma, 0 found
 39 | WARNING | [x] A comma should follow the last multiline array item. Found:
    |         |     'TRUE'
 46 | ERROR   | [x] TRUE, FALSE and NULL must be uppercase; expected "TRUE" but
    |         |     found "true"
 47 | ERROR   | [x] Whitespace found at end of line
 52 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 3
 56 | WARNING | [x] A comma should follow the last multiline array item. Found:
    |         |     'Health Check URL settings'
 62 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 2
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Comments

leymannx created an issue. See original summary.

pranali.addweb’s picture

Assigned: Unassigned » pranali.addweb
Snehal Brahmbhatt’s picture

Status: Active » Needs review
StatusFileSize
new13.32 KB

@leymannx, Please find my patch whereas I resolved all the standard error mentioned above. Hope this helps you!..

Thanks

norman.lol’s picture

I'm not the maintainer of this module. The maintainer needs to check and fix that.

karthikeyan-manivasagam’s picture

Status: Needs review » Fixed

@snehal.addweb thanks for the patch. its fixed thanks to all

Status: Fixed » Closed (fixed)

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