Fix PHPCS Errors and warnings

Created on 5 January 2023, over 2 years ago
Updated 19 February 2024, over 1 year ago

PHPCS reported the following errors and warnings :


FILE: /fsk_frontend/css/base/variables.css
----------------------------------------------------------------------------------------------
FOUND 59 ERRORS AFFECTING 49 LINES
----------------------------------------------------------------------------------------------
  14 | ERROR | [x] Blank lines are not allowed in class definitions
  23 | ERROR | [x] Blank lines are not allowed in class definitions
  30 | ERROR | [x] Blank lines are not allowed in class definitions
  32 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
  33 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
  35 | ERROR | [ ] Style definitions must end with a semicolon
  36 | ERROR | [ ] Style definitions must end with a semicolon
  37 | ERROR | [ ] Style definitions must end with a semicolon
  38 | ERROR | [ ] Style definitions must end with a semicolon
  39 | ERROR | [ ] Style definitions must end with a semicolon
  40 | ERROR | [x] Blank lines are not allowed in class definitions
  57 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  57 | ERROR | [ ] No style definition found on line; check for missing colon
  58 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
  58 | ERROR | [ ] No style definition found on line; check for missing colon
  69 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  69 | ERROR | [ ] No style definition found on line; check for missing colon
  70 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
  70 | ERROR | [ ] No style definition found on line; check for missing colon
  72 | ERROR | [x] Blank lines are not allowed in class definitions
  77 | ERROR | [x] Blank lines are not allowed in class definitions
  93 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  93 | ERROR | [ ] No style definition found on line; check for missing colon
  97 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  97 | ERROR | [ ] No style definition found on line; check for missing colon
 103 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 103 | ERROR | [ ] No style definition found on line; check for missing colon
 104 | ERROR | [x] Blank lines are not allowed in class definitions
 111 | ERROR | [x] Blank lines are not allowed in class definitions
 117 | ERROR | [x] Blank lines are not allowed in class definitions
 126 | ERROR | [x] Blank lines are not allowed in class definitions
 147 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 147 | ERROR | [ ] No style definition found on line; check for missing colon
 148 | ERROR | [x] Blank lines are not allowed in class definitions
 150 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 151 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 165 | ERROR | [x] Blank lines are not allowed in class definitions
 167 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 168 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 175 | ERROR | [x] Blank lines are not allowed in class definitions
 177 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 178 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 196 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
 196 | ERROR | [ ] No style definition found on line; check for missing colon
 197 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 198 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 198 | ERROR | [ ] No style definition found on line; check for missing colon
 200 | ERROR | [x] Blank lines are not allowed in class definitions
 202 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 203 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 217 | ERROR | [x] Blank lines are not allowed in class definitions
 219 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 220 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 229 | ERROR | [x] Blank lines are not allowed in class definitions
 231 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 232 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 237 | ERROR | [x] Blank lines are not allowed in class definitions
 239 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
 240 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
----------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 44 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------


FILE: /fsk_frontend/templates/form/input/input--checkbox/input--checkbox.css
--------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------
 30 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 30 | ERROR | [ ] No style definition found on line; check for missing colon
--------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------


FILE: /fsk_frontend/templates/form/input/input--radio/input--radio.css
--------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------
 31 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 31 | ERROR | [ ] No style definition found on line; check for missing colon
--------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------


FILE: /fsk_frontend/templates/form/vertical-tabs/vertical-tabs.css
----------------------------------------------------------------------------------------------------------------------
FOUND 8 ERRORS AFFECTING 7 LINES
----------------------------------------------------------------------------------------------------------------------
  84 | ERROR | [x] Selectors must be on a single line
  85 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
  97 | ERROR | [x] Selectors must be on a single line
  98 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
 195 | ERROR | [x] Selectors must be on a single line
 196 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
 196 | ERROR | [x] Selectors must be on a single line
 197 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------


FILE:/fsk_frontend/templates/form/dropbutton-wrapper/dropbutton-wrapper.css
--------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------
 56 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 56 | ERROR | [ ] No style definition found on line; check for missing colon
--------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------
๐Ÿ“Œ Task
Status

Needs review

Version

1.0

Component

Code

Created by

๐Ÿ‡ฉ๐Ÿ‡ชGermany solimanharkas Hamburg

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

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Hi, Reviewed the patch #2, applies cleanly and fixes all the errors reported.

  • Status changed to RTBC 10 months ago
  • ๐Ÿ‡ต๐Ÿ‡ญPhilippines cleavinjosh

    Hi@_pratik_,

    I applied patch #2, it was applied smoothly and fixed all of the phpcs issues.

    โžœ  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml fsk_frontend
    
    FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/fsk_frontend/css/base/variables.css
    --------------------------------------------------------------------------------------------------------
    FOUND 59 ERRORS AFFECTING 49 LINES
    --------------------------------------------------------------------------------------------------------
      14 | ERROR | [x] Blank lines are not allowed in class definitions
      23 | ERROR | [x] Blank lines are not allowed in class definitions
      30 | ERROR | [x] Blank lines are not allowed in class definitions
      32 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
      33 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
      35 | ERROR | [ ] Style definitions must end with a semicolon
      36 | ERROR | [ ] Style definitions must end with a semicolon
      37 | ERROR | [ ] Style definitions must end with a semicolon
      38 | ERROR | [ ] Style definitions must end with a semicolon
      39 | ERROR | [ ] Style definitions must end with a semicolon
      40 | ERROR | [x] Blank lines are not allowed in class definitions
      57 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
      57 | ERROR | [ ] No style definition found on line; check for missing colon
      58 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
      58 | ERROR | [ ] No style definition found on line; check for missing colon
      69 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
      69 | ERROR | [ ] No style definition found on line; check for missing colon
      70 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
      70 | ERROR | [ ] No style definition found on line; check for missing colon
      72 | ERROR | [x] Blank lines are not allowed in class definitions
      77 | ERROR | [x] Blank lines are not allowed in class definitions
      93 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
      93 | ERROR | [ ] No style definition found on line; check for missing colon
      97 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
      97 | ERROR | [ ] No style definition found on line; check for missing colon
     103 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     103 | ERROR | [ ] No style definition found on line; check for missing colon
     104 | ERROR | [x] Blank lines are not allowed in class definitions
     111 | ERROR | [x] Blank lines are not allowed in class definitions
     117 | ERROR | [x] Blank lines are not allowed in class definitions
     126 | ERROR | [x] Blank lines are not allowed in class definitions
     147 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     147 | ERROR | [ ] No style definition found on line; check for missing colon
     148 | ERROR | [x] Blank lines are not allowed in class definitions
     150 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     151 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     165 | ERROR | [x] Blank lines are not allowed in class definitions
     167 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     168 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     175 | ERROR | [x] Blank lines are not allowed in class definitions
     177 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     178 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     196 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 6
     196 | ERROR | [ ] No style definition found on line; check for missing colon
     197 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     198 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     198 | ERROR | [ ] No style definition found on line; check for missing colon
     200 | ERROR | [x] Blank lines are not allowed in class definitions
     202 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     203 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     217 | ERROR | [x] Blank lines are not allowed in class definitions
     219 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     220 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     229 | ERROR | [x] Blank lines are not allowed in class definitions
     231 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     232 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     237 | ERROR | [x] Blank lines are not allowed in class definitions
     239 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
     240 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 0
    --------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 44 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    --------------------------------------------------------------------------------------------------------
    
    
    FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/fsk_frontend/templates/form/input/input--checkbox/input--checkbox.css
    ------------------------------------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------------------------------------------------
     30 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     30 | ERROR | [ ] No style definition found on line; check for missing colon
    ------------------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    ------------------------------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/fsk_frontend/templates/form/input/input--radio/input--radio.css
    ------------------------------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------------------------------------------
     31 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     31 | ERROR | [ ] No style definition found on line; check for missing colon
    ------------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    ------------------------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/fsk_frontend/templates/form/vertical-tabs/vertical-tabs.css
    --------------------------------------------------------------------------------------------------------------------------------
    FOUND 8 ERRORS AFFECTING 7 LINES
    --------------------------------------------------------------------------------------------------------------------------------
      84 | ERROR | [x] Selectors must be on a single line
      85 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
      97 | ERROR | [x] Selectors must be on a single line
      98 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
     195 | ERROR | [x] Selectors must be on a single line
     196 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
     196 | ERROR | [x] Selectors must be on a single line
     197 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
    --------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    --------------------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/fsk_frontend/templates/form/dropbutton-wrapper/dropbutton-wrapper.css
    ------------------------------------------------------------------------------------------------------------------------------------------
    FOUND 2 ERRORS AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------------------------------------------------
     56 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
     56 | ERROR | [ ] No style definition found on line; check for missing colon
    ------------------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    ------------------------------------------------------------------------------------------------------------------------------------------
    
    Time: 301ms; Memory: 12MB
    
    โžœ  contrib cd fsk_frontend
    โžœ  fsk_frontend git:(1.0.x) curl https://www.drupal.org/files/issues/2023-01-05/3331249-2.patch | patch -p1
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100 10747  100 10747    0     0  24999      0 --:--:-- --:--:-- --:--:-- 25051
    patching file css/base/variables.css
    patching file templates/form/dropbutton-wrapper/dropbutton-wrapper.css
    patching file templates/form/input/input--checkbox/input--checkbox.css
    patching file templates/form/input/input--radio/input--radio.css
    patching file templates/form/vertical-tabs/vertical-tabs.css
    โžœ  fsk_frontend git:(1.0.x) โœ— ..
    โžœ  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml fsk_frontend
    โžœ  contrib

    Thank you.

  • ๐Ÿ‡ฎ๐Ÿ‡นItaly apaderno Brescia, ๐Ÿ‡ฎ๐Ÿ‡น

    avpaderno โ†’ made their first commit to this issueโ€™s fork.

  • ๐Ÿ‡ฎ๐Ÿ‡นItaly apaderno Brescia, ๐Ÿ‡ฎ๐Ÿ‡น

    Since patches are no longer tested, a merge request must be provided.

  • Pipeline finished with Failed
    20 days ago
    Total: 112s
    #495265
Production build 0.71.5 2024