Fix the issues reported by phpcs

Created on 13 January 2023, almost 2 years ago
Updated 28 June 2023, over 1 year ago

Problem/Motivation

FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/ckeditor.module
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------------------------------
 27 | ERROR | The array declaration extends to column 730 (the limit is 80). The array content should be split up over multiple lines
 31 | ERROR | The array declaration extends to column 588 (the limit is 80). The array content should be split up over multiple lines
 41 | ERROR | The array declaration extends to column 413 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/ckeditor.api.php
-------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------
 21 | ERROR | Missing parameter type
 48 | ERROR | Missing parameter type
-------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/modules/ckeditor_test.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: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/modules/src/Form/AjaxCssForm.php
-----------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------
 36 | ERROR | Missing short description in doc comment
-----------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/modules/src/Plugin/CKEditorPlugin/LlamaButton.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 8 | WARNING | Line exceeds 80 characters; contains 82 characters
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: ...goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/modules/src/Plugin/CKEditorPlugin/LlamaContextualAndButton.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 13 | ERROR | Doc comment short description must be on a single line, further text should be a separate paragraph
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/Unit/CKEditorPluginManagerTest.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------
 70 | ERROR | [x] Array closing indentation error, expected 10 spaces but found 8
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/FunctionalJavascript/MediaLibraryTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------------
 289 | ERROR | [x] Array indentation error, expected 10 spaces but found 8
 290 | ERROR | [x] Array indentation error, expected 10 spaces but found 8
 298 | ERROR | [x] Array closing indentation error, expected 8 spaces but found 6
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/FunctionalJavascript/MediaTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 9 LINES
------------------------------------------------------------------------------------------------------------------------------------------
 676 | ERROR | [ ] The array declaration extends to column 83 (the limit is 80). The array content should be split up over multiple lines
 679 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 760 | ERROR | [ ] The array declaration extends to column 85 (the limit is 80). The array content should be split up over multiple lines
 806 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 824 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 827 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 833 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 834 | ERROR | [ ] The array declaration extends to column 83 (the limit is 80). The array content should be split up over multiple lines
 837 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------


FILE: ...Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/FunctionalJavascript/BigPipeRegressionTest.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------------------------------------------------------------------------------
 55 | ERROR | Public method name "BigPipeRegressionTest::testCommentForm_2698811" is not in lowerCamel format
 56 | ERROR | The array declaration extends to column 107 (the limit is 80). The array content should be split up over multiple lines
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/Functional/CKEditorAdminTest.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------
 37 | ERROR | [x] Missing function doc comment
 86 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
---------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/Functional/CKEditorLoadingTest.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 2 WARNINGS AFFECTING 5 LINES
-----------------------------------------------------------------------------------------------------------------------------------
  42 | ERROR   | [x] Missing function doc comment
  93 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters
 187 | ERROR   | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 208 | WARNING | [ ] Line exceeds 80 characters; contains 83 characters
 219 | ERROR   | [x] Missing function doc comment
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/Kernel/CKEditorPluginManagerTest.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AND 1 WARNING AFFECTING 8 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
  36 | ERROR   | [x] Missing function doc comment
  39 | WARNING | [x] There must be no blank line following an inline comment
  66 | ERROR   | [ ] The array declaration extends to column 112 (the limit is 80). The array content should be split up over multiple lines
  84 | ERROR   | [ ] The array declaration extends to column 201 (the limit is 80). The array content should be split up over multiple lines
 114 | ERROR   | [ ] The array declaration extends to column 111 (the limit is 80). The array content should be split up over multiple lines
 121 | ERROR   | [ ] The array declaration extends to column 115 (the limit is 80). The array content should be split up over multiple lines
 127 | ERROR   | [ ] The array declaration extends to column 145 (the limit is 80). The array content should be split up over multiple lines
 133 | ERROR   | [ ] The array declaration extends to column 174 (the limit is 80). The array content should be split up over multiple lines
---------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/tests/src/Kernel/CKEditorTest.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 21 ERRORS AND 2 WARNINGS AFFECTING 23 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
  52 | ERROR   | [x] Missing function doc comment
  56 | WARNING | [x] There must be no blank line following an inline comment
  86 | ERROR   | [ ] Public method name "CKEditorTest::testGetJSSettings" is not in lowerCamel format
 137 | ERROR   | [ ] The array declaration extends to column 111 (the limit is 80). The array content should be split up over multiple lines
 138 | ERROR   | [ ] The array declaration extends to column 109 (the limit is 80). The array content should be split up over multiple lines
 139 | ERROR   | [ ] The array declaration extends to column 118 (the limit is 80). The array content should be split up over multiple lines
 140 | ERROR   | [ ] The array declaration extends to column 122 (the limit is 80). The array content should be split up over multiple lines
 170 | ERROR   | [ ] The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple lines
 228 | ERROR   | [ ] Public method name "CKEditorTest::testBuildToolbarJSSetting" is not in lowerCamel format
 259 | ERROR   | [ ] Public method name "CKEditorTest::testBuildContentsCssJSSetting" is not in lowerCamel format
 267 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters
 349 | ERROR   | [ ] The array declaration extends to column 82 (the limit is 80). The array content should be split up over multiple lines
 350 | ERROR   | [ ] The array declaration extends to column 93 (the limit is 80). The array content should be split up over multiple lines
 418 | ERROR   | [ ] Public method name "CKEditorTest::testJSTranslation" is not in lowerCamel format
 426 | ERROR   | [ ] The array declaration extends to column 83 (the limit is 80). The array content should be split up over multiple lines
 481 | ERROR   | [ ] Protected method name "CKEditorTest::assertCKEditorLanguage" is not in lowerCamel format
 497 | ERROR   | [x] Missing function doc comment
 502 | ERROR   | [ ] The array declaration extends to column 108 (the limit is 80). The array content should be split up over multiple lines
 508 | ERROR   | [x] Missing function doc comment
 518 | ERROR   | [ ] The array declaration extends to column 85 (the limit is 80). The array content should be split up over multiple lines
 523 | ERROR   | [x] Missing function doc comment
 529 | ERROR   | [x] Missing function doc comment
 555 | ERROR   | [x] Missing function doc comment
---------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/ckeditor.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: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/ckeditor.admin.inc
---------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------
 26 | ERROR | Type hint "array" missing for $variables
---------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/LICENSE.md
--------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 21 WARNINGS AFFECTING 21 LINES
--------------------------------------------------------------------------------------------------------
   38 | WARNING | Line exceeds 80 characters; contains 88 characters
   40 | WARNING | Line exceeds 80 characters; contains 109 characters
   42 | WARNING | Line exceeds 80 characters; contains 111 characters
   43 | WARNING | Line exceeds 80 characters; contains 88 characters
   44 | WARNING | Line exceeds 80 characters; contains 129 characters
   46 | WARNING | Line exceeds 80 characters; contains 113 characters
   48 | WARNING | Line exceeds 80 characters; contains 112 characters
   50 | WARNING | Line exceeds 80 characters; contains 111 characters
   52 | WARNING | Line exceeds 80 characters; contains 89 characters
   54 | WARNING | Line exceeds 80 characters; contains 90 characters
   56 | WARNING | Line exceeds 80 characters; contains 90 characters
   57 | WARNING | Line exceeds 80 characters; contains 82 characters
 1412 | WARNING | Line exceeds 80 characters; contains 142 characters
 1414 | WARNING | Line exceeds 80 characters; contains 127 characters
 1416 | WARNING | Line exceeds 80 characters; contains 205 characters
 1418 | WARNING | Line exceeds 80 characters; contains 190 characters
 1420 | WARNING | Line exceeds 80 characters; contains 755 characters
 1428 | WARNING | Line exceeds 80 characters; contains 142 characters
 1430 | WARNING | Line exceeds 80 characters; contains 127 characters
 1431 | WARNING | Line exceeds 80 characters; contains 205 characters
 1433 | WARNING | Line exceeds 80 characters; contains 755 characters
--------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/plugins/dialog/styles/dialog.css
------------------------------------------------------------------------------------------------------------------------------
FOUND 20 ERRORS AFFECTING 10 LINES
------------------------------------------------------------------------------------------------------------------------------
  2 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
  2 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
  6 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
  6 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
  7 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
  7 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
  8 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
  8 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
  9 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
  9 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 10 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
 10 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 11 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
 11 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 12 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
 12 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 13 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
 13 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
 17 | ERROR | [x] Spaces must be used to indent lines; tabs are not allowed
 17 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 1
------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 20 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/skins/moono-lisa/dialog.css
-------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
-------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/skins/moono-lisa/dialog_ie8.css
-----------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
-----------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/CHANGES.md
--------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 47 WARNINGS AFFECTING 47 LINES
--------------------------------------------------------------------------------------------------------
   12 | WARNING | Line exceeds 80 characters; contains 154 characters
   67 | WARNING | Line exceeds 80 characters; contains 243 characters
   69 | WARNING | Line exceeds 80 characters; contains 188 characters
  194 | WARNING | Line exceeds 80 characters; contains 210 characters
  198 | WARNING | Line exceeds 80 characters; contains 161 characters
  313 | WARNING | Line exceeds 80 characters; contains 398 characters
  317 | WARNING | Line exceeds 80 characters; contains 346 characters
  497 | WARNING | Line exceeds 80 characters; contains 85 characters
  498 | WARNING | Line exceeds 80 characters; contains 86 characters
  499 | WARNING | Line exceeds 80 characters; contains 114 characters
  500 | WARNING | Line exceeds 80 characters; contains 96 characters
  504 | WARNING | Line exceeds 80 characters; contains 96 characters
  506 | WARNING | Line exceeds 80 characters; contains 92 characters
  560 | WARNING | Line exceeds 80 characters; contains 280 characters
  684 | WARNING | Line exceeds 80 characters; contains 113 characters
  685 | WARNING | Line exceeds 80 characters; contains 162 characters
  690 | WARNING | Line exceeds 80 characters; contains 108 characters
  691 | WARNING | Line exceeds 80 characters; contains 133 characters
  695 | WARNING | Line exceeds 80 characters; contains 191 characters
  696 | WARNING | Line exceeds 80 characters; contains 104 characters
  698 | WARNING | Line exceeds 80 characters; contains 95 characters
  706 | WARNING | Line exceeds 80 characters; contains 275 characters
  764 | WARNING | Line exceeds 80 characters; contains 138 characters
  766 | WARNING | Line exceeds 80 characters; contains 98 characters
  767 | WARNING | Line exceeds 80 characters; contains 88 characters
  768 | WARNING | Line exceeds 80 characters; contains 81 characters
 1011 | WARNING | Line exceeds 80 characters; contains 117 characters
 1012 | WARNING | Line exceeds 80 characters; contains 124 characters
 1073 | WARNING | Line exceeds 80 characters; contains 88 characters
 1075 | WARNING | Line exceeds 80 characters; contains 345 characters
 1354 | WARNING | Line exceeds 80 characters; contains 188 characters
 1359 | WARNING | Line exceeds 80 characters; contains 335 characters
 1360 | WARNING | Line exceeds 80 characters; contains 233 characters
 1374 | WARNING | Line exceeds 80 characters; contains 117 characters
 1375 | WARNING | Line exceeds 80 characters; contains 100 characters
 1376 | WARNING | Line exceeds 80 characters; contains 275 characters
 1377 | WARNING | Line exceeds 80 characters; contains 282 characters
 1378 | WARNING | Line exceeds 80 characters; contains 157 characters
 1382 | WARNING | Line exceeds 80 characters; contains 164 characters
 1431 | WARNING | Line exceeds 80 characters; contains 280 characters
 1487 | WARNING | Line exceeds 80 characters; contains 280 characters
 1692 | WARNING | Line exceeds 80 characters; contains 102 characters
 1693 | WARNING | Line exceeds 80 characters; contains 113 characters
 2076 | WARNING | Line exceeds 80 characters; contains 117 characters
 2079 | WARNING | Line exceeds 80 characters; contains 117 characters
 2128 | WARNING | Line exceeds 80 characters; contains 102 characters
 2182 | WARNING | Line exceeds 80 characters; contains 88 characters
--------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/vendor/SECURITY.md
---------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------
  3 | WARNING | Line exceeds 80 characters; contains 105 characters
  8 | WARNING | Line exceeds 80 characters; contains 104 characters
 10 | WARNING | Line exceeds 80 characters; contains 173 characters
---------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/CKEditorPluginContextualInterface.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------
 38 | ERROR | Description for the @return value is missing
--------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Plugin/CKEditorPlugin/DrupalMedia.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------
 48 | ERROR | Type hint "array" missing for $plugin_definition
--------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Plugin/CKEditorPlugin/DrupalImage.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------------------------------
 69 | ERROR | The array declaration extends to column 89 (the limit is 80). The array content should be split up over multiple lines
 74 | ERROR | Doc comment short description must start with a capital letter
 83 | ERROR | The array declaration extends to column 103 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Plugin/CKEditorPlugin/Internal.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 108 | ERROR   | The array declaration extends to column 108 (the limit is 80). The array content should be split up over multiple lines
 344 | ERROR   | Parameter tags must be defined first in a doc comment
 369 | WARNING | Line exceeds 80 characters; contains 81 characters
 399 | ERROR   | Parameter tags must be defined first in a doc comment
 409 | ERROR   | Protected method name "Internal::generateACFSettings" is not in lowerCamel format
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Plugin/CKEditorPlugin/StylesCombo.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------------------
  96 | ERROR | Doc comment short description must start with a capital letter
 118 | ERROR | Parameter tags must be defined first in a doc comment
--------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Plugin/Editor/CKEditor.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AFFECTING 7 LINES
---------------------------------------------------------------------------------------------------------------------------------------
 291 | ERROR | The array declaration extends to column 109 (the limit is 80). The array content should be split up over multiple lines
 321 | ERROR | Public method name "CKEditor::getJSSettings" is not in lowerCamel format
 361 | ERROR | The array declaration extends to column 86 (the limit is 80). The array content should be split up over multiple lines
 447 | ERROR | Parameter tags must be defined first in a doc comment
 453 | ERROR | Public method name "CKEditor::buildToolbarJSSetting" is not in lowerCamel format
 471 | ERROR | Parameter tags must be defined first in a doc comment
 477 | ERROR | Public method name "CKEditor::buildContentsCssJSSetting" is not in lowerCamel format
---------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/Annotation/CKEditorPlugin.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
 10 | ERROR | [x] Doc comment long description must end with a full stop
 40 | ERROR | [ ] The @var tag must be the first tag in a member variable comment
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------


FILE: /Users/sahil.goyal/Documents/MySites/contrib10/drupal10/modules/contrib/ckeditor/src/CKEditorPluginInterface.php
----------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------
 39 | ERROR | [ ] Description for the @return value is missing
 74 | ERROR | [x] There must be exactly one blank line before the tags in a doc comment
 88 | ERROR | [x] Doc comment long description must end with a full stop
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------

Steps to reproduce

phpcs --standard=โ€œDrupal,DrupalPracticeโ€ --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml,css,js .

๐Ÿ“Œ Task
Status

Needs work

Version

1.0

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡ณIndia sahil.goyal

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

Comments & Activities

Not all content is available!

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

  • Status changed to RTBC almost 2 years ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia Ranjit1032002

    I had Reviewed the issue and it's looks good mentioned in merge request comment #2

  • Status changed to Needs work over 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡นItaly apaderno Brescia, ๐Ÿ‡ฎ๐Ÿ‡น

    The issue summary should always describe what the issue is trying to fix and, in the case of coding standards issues, show which command and arguments have been used and which report that command shown. In this way, project maintainers can verify the patch/MR fixes all the warnings/errors.

  • Status changed to Needs review over 1 year ago
  • Open in Jenkins โ†’ Open on Drupal.org โ†’
    Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7
    last update over 1 year ago
    54 pass
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia chaitanyadessai Goa

    Fixed fixable errors, please review patch. Used following command, vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,install,inc,theme web/modules/ckeditor

  • Status changed to Needs work over 1 year ago
  • ๐Ÿ‡ฎ๐Ÿ‡นItaly apaderno Brescia, ๐Ÿ‡ฎ๐Ÿ‡น

    The issue summary must still be updated, since it does not show the command nor the used arguments.

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

    Furthermore, the report shows errors/warnings for 30 files, but the provided patches change less files.

  • ๐Ÿ‡จ๐Ÿ‡ฆCanada imustakim Canada

    Issue summary updated.

Production build 0.71.5 2024