Contrib.social

Fix the issues reported by phpcs

Open on Drupal.org ā†’
Created on 2 May 2023, over 1 year ago
Updated 21 August 2024, 4 months ago

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/README.md
----------------------------------------------------------------------------------------
FOUND 1 ERROR AND 11 WARNINGS AFFECTING 12 LINES
----------------------------------------------------------------------------------------
9 | WARNING | [ ] Line exceeds 80 characters; contains 200 characters
26 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
44 | WARNING | [ ] Line exceeds 80 characters; contains 163 characters
45 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters
46 | WARNING | [ ] Line exceeds 80 characters; contains 86 characters
47 | WARNING | [ ] Line exceeds 80 characters; contains 93 characters
48 | WARNING | [ ] Line exceeds 80 characters; contains 102 characters
63 | WARNING | [ ] Line exceeds 80 characters; contains 239 characters
70 | WARNING | [ ] Line exceeds 80 characters; contains 105 characters
71 | WARNING | [ ] Line exceeds 80 characters; contains 173 characters
72 | WARNING | [ ] Line exceeds 80 characters; contains 136 characters
73 | ERROR | [x] Expected 1 newline at end of file; 2 found
----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/appsearch.module
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------------------------
49 | WARNING | [x] A comma should follow the last multiline array item. Found: NULL
59 | WARNING | [x] A comma should follow the last multiline array item. Found: NULL
66 | WARNING | [x] A comma should follow the last multiline array item. Found: NULL
-----------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/css/appsearch.css
------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------
1 | WARNING | File appears to be minified and cannot be processed
------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/css/general-fix.css
--------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------
70 | ERROR | Class definition is empty
--------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/appsearch.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: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/Database.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 1 WARNING AFFECTING 3 LINES
-------------------------------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
77 | ERROR | [x] Expected 1 space before ":"; 0 found
77 | ERROR | [x] Expected 1 space after ":"; 0 found
139 | ERROR | [x] Inline comments must start with a capital letter
-------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/BatchHelper.php
-------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 2 WARNINGS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
96 | ERROR | [ ] Public method name "BatchHelper::wipe_all" is not in lowerCamel format
103 | WARNING | [ ] Unused variable $page_count.
114 | ERROR | [x] Expected 1 space after FOREACH keyword; 0 found
131 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
177 | ERROR | [x] Expected newline after closing brace
-------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Plugin/Block/SearchUIBlock.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 19 ERRORS AND 5 WARNINGS AFFECTING 22 LINES
-----------------------------------------------------------------------------------------------------------------------------------
7 | WARNING | [x] Unused use statement
20 | ERROR | [ ] Class name doesn't match filename; expected "class SearchUIBlock"
36 | ERROR | [ ] Parameter $entityTypeManager is not described in comment
45 | ERROR | [ ] Doc comment for parameter $currentUser does not match actual variable name $entityTypeManager
68 | ERROR | [x] Expected 1 blank line after function; 0 found
166 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
170 | ERROR | [ ] Comment indentation error, expected only 1 spaces
173 | WARNING | [ ] Unused variable $cache.
173 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
176 | ERROR | [ ] Comment indentation error, expected only 1 spaces
178 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
179 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
180 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
181 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
182 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
183 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
184 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
184 | ERROR | [ ] Comment indentation error, expected only 1 spaces
185 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
186 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
187 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
188 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
213 | ERROR | [x] Closing parenthesis of array declaration must be on a new line
241 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 15 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Controller/ServerController.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
7 | WARNING | [x] Unused use statement
8 | WARNING | [x] Unused use statement
9 | WARNING | [x] Unused use statement
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Tracker/Tracker.php
------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 6 WARNINGS AFFECTING 8 LINES
------------------------------------------------------------------------------------------------------------------------
9 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
62 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
72 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
177 | ERROR | [x] list(...) is forbidden, use [...] instead.
323 | WARNING | [ ] Unused variable $res.
376 | WARNING | [ ] Unused variable $count.
410 | ERROR | [x] list(...) is forbidden, use [...] instead.
434 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Tracker/TrackingManager.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------------------
9 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
68 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
--------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/AppSearchClient.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 4 WARNINGS AFFECTING 5 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------
5 | WARNING | [x] Unused use statement
6 | WARNING | [x] Unused use statement
16 | ERROR | [ ] Missing short description in doc comment
34 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
59 | WARNING | [x] '@todo: Validate if this is enough to check if the connection is valid.' should match the format '@todo Fix problem X here.'
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Entity/Engine.php
----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------
195 | WARNING | [x] '@todo: Add support for other entity types.' should match the format '@todo Fix problem X here.'
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Entity/Server.php
----------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 2 WARNINGS AFFECTING 4 LINES
----------------------------------------------------------------------------------------------------------------------
159 | ERROR | [x] Inline comments must start with a capital letter
160 | ERROR | [ ] Comment indentation error, expected only 1 spaces
161 | WARNING | [x] There must be no blank line following an inline comment
161 | WARNING | [ ] There must be no blank line following an inline comment
164 | ERROR | [ ] Comment indentation error, expected only 1 spaces
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineForm.php
------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 2 WARNINGS AFFECTING 5 LINES
------------------------------------------------------------------------------------------------------------------------
107 | WARNING | [x] A comma should follow the last multiline array item. Found: 'width: 50em;'
122 | ERROR | [x] Missing function doc comment
124 | ERROR | [x] Expected 1 blank line after function; 0 found
129 | WARNING | [x] '@todo: Add support for other entity types.' should match the format '@todo Fix problem X here.'
145 | ERROR | [x] Expected 1 space between double arrow and "$this"; 0 found
145 | ERROR | [x] Expected 1 space after "=>"; 0 found
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 6 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineSchema.php
--------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------------
51 | WARNING | [ ] Unused variable $key.
52 | WARNING | [x] '@TODO: Validate fields.' should match the format '@todo Fix problem X here.'
--------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/SyncEngineConfirmForm.php
-----------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AND 2 WARNINGS AFFECTING 4 LINES
-----------------------------------------------------------------------------------------------------------------------------------
17 | ERROR | [x] Inline comments must start with a capital letter
18 | ERROR | [ ] Comment indentation error, expected only 1 spaces
19 | WARNING | [ ] There must be no blank line following an inline comment
19 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
42 | WARNING | [ ] Line exceeds 80 characters; contains 104 characters
-----------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineStatusForm.php
------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------
53 | WARNING | Line exceeds 80 characters; contains 104 characters
------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/FieldSchemaForm.php
-----------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------
171 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
-----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/SearchUiForm.php
--------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
--------------------------------------------------------------------------------------------------------------------------
91 | WARNING | [x] A comma should follow the last multiline array item. Found: 'width: 50em;'
113 | WARNING | [x] A comma should follow the last multiline array item. Found: '

'
125 | WARNING | [x] A comma should follow the last multiline array item. Found: '

'
151 | WARNING | [x] A comma should follow the last multiline array item. Found: '

'
239 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
--------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/AppsearchClientInterface.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------
7 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
8 | ERROR | Interface name doesn't match filename; expected "interface AppsearchClientInterface"
---------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/src/index.css
------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------
4 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
5 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
5 | ERROR | [ ] No style definition found on line; check for missing colon
12 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
12 | ERROR | [ ] No style definition found on line; check for missing colon
------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/README.md
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------------------------------------------
25 | WARNING | Line exceeds 80 characters; contains 95 characters
36 | WARNING | Line exceeds 80 characters; contains 171 characters
38 | WARNING | Line exceeds 80 characters; contains 322 characters
40 | WARNING | Line exceeds 80 characters; contains 271 characters
--------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/appsearch.theme.inc
--------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 10 WARNINGS AFFECTING 11 LINES
--------------------------------------------------------------------------------------------------------------------
82 | WARNING | [x] A comma should follow the last multiline array item. Found: )
87 | WARNING | [x] A comma should follow the last multiline array item. Found: "Disabled"
97 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
102 | WARNING | [x] A comma should follow the last multiline array item. Found: )
107 | WARNING | [x] A comma should follow the last multiline array item. Found: )
112 | WARNING | [x] A comma should follow the last multiline array item. Found: )
121 | WARNING | [x] A comma should follow the last multiline array item. Found: )
161 | ERROR | [x] Namespaced classes/interfaces/traits should be referenced with use statements
202 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
224 | WARNING | [x] A comma should follow the last multiline array item. Found: )
225 | WARNING | [x] A comma should follow the last multiline array item. Found: ]
--------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 11 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/contribution/web/modules/contrib/elastic_enterprise_search/modules/appsearch/tests/Controller/EngineControllerTest.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------------
6 | WARNING | [x] Unused use statement
7 | WARNING | [x] Unused use statement
8 | WARNING | [x] Unused use statement
50 | WARNING | [ ] Possible useless method overriding detected
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------

šŸ“Œ Task
Status

Needs work

Version

1.0

Component

Code

Created by

šŸ‡®šŸ‡³India himanshu_jhaloya Indore

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.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @himanshu_jhaloya
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India msankhala Bikaner, Rajasthan

    This will be fixed in the stable version. This is still in beta release. I would appreciate the real bug fix instead of just coding standard fix.

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India himanshu_jhaloya Indore

    @msankhala Ok thanks for the update. can I add the patch it will be helpful

  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Comment over 1 year ago ā†’
    šŸ‡®šŸ‡³India msankhala Bikaner, Rajasthan

    Fixed in 1.1.x dev branch. This will reflect in the next beta release. There are a couple of warnings that can be ignored as of now.

  • Status changed to Needs work 4 months ago
  • Comment 4 months ago ā†’
    šŸ‡µšŸ‡­Philippines cleavinjosh

    Hi @msankhala,

    I ran phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml on 1.1.x branch and there are still some errors I encountered:

    āžœ  contrib cd elastic_enterprise_search
āžœ  elastic_enterprise_search git:(1.1.x) ..
āžœ  contrib phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig elastic_enterprise_search

FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/README.md
--------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 19 WARNINGS AFFECTING 19 LINES
--------------------------------------------------------------------------------------------------------
   9 | WARNING | Line exceeds 80 characters; contains 200 characters
  26 | WARNING | Line exceeds 80 characters; contains 86 characters
  42 | WARNING | Line exceeds 80 characters; contains 133 characters
  43 | WARNING | Line exceeds 80 characters; contains 146 characters
  45 | WARNING | Line exceeds 80 characters; contains 90 characters
  54 | WARNING | Line exceeds 80 characters; contains 82 characters
  55 | WARNING | Line exceeds 80 characters; contains 86 characters
  56 | WARNING | Line exceeds 80 characters; contains 93 characters
  57 | WARNING | Line exceeds 80 characters; contains 102 characters
  72 | WARNING | Line exceeds 80 characters; contains 307 characters
  76 | WARNING | Line exceeds 80 characters; contains 355 characters
  79 | WARNING | Line exceeds 80 characters; contains 397 characters
  81 | WARNING | Line exceeds 80 characters; contains 107 characters
  82 | WARNING | Line exceeds 80 characters; contains 189 characters
  89 | WARNING | Line exceeds 80 characters; contains 173 characters
  90 | WARNING | Line exceeds 80 characters; contains 123 characters
  95 | WARNING | Line exceeds 80 characters; contains 133 characters
  97 | WARNING | Line exceeds 80 characters; contains 308 characters
 110 | WARNING | Line exceeds 80 characters; contains 101 characters
--------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/docker-compose/README.md
-----------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------
 3 | WARNING | Line exceeds 80 characters; contains 102 characters
-----------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/css/appsearch.css
----------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
----------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/tests/Controller/EngineControllerTest.php
----------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------------
 47 | WARNING | Possible useless method overriding detected
----------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/templates/server.html.twig
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------
 17 | ERROR | [x] Expected 1 newline at end of file; 0 found
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/templates/engine.html.twig
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------
 29 | ERROR | [x] Expected 1 newline at end of file; 0 found
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/appsearch.theme.inc
------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Render\Element.
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/README.md
------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------
 25 | WARNING | Line exceeds 80 characters; contains 95 characters
 36 | WARNING | Line exceeds 80 characters; contains 171 characters
 38 | WARNING | Line exceeds 80 characters; contains 322 characters
 40 | WARNING | Line exceeds 80 characters; contains 271 characters
------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/build/static/css/main.css
----------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------
 1 | WARNING | File appears to be minified and cannot be processed
----------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/build/static/js/main.js.LICENSE.txt
--------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------------------------------
 7 | WARNING | Line exceeds 80 characters; contains 148 characters
--------------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/search-ui/src/index.css
----------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------
  4 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  5 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
  5 | ERROR | [ ] No style definition found on line; check for missing colon
 12 | ERROR | [x] Line indented incorrectly; expected 2 spaces, found 4
 12 | ERROR | [ ] No style definition found on line; check for missing colon
----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/ServerListBuilder.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\Entity\ServerInterface.
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineWipeAllConfirmForm.php
------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------------------
 9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Logger\LoggerChannelTrait.
------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineForm.php
----------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------
  7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\AppSearchClientInterface.
 38 | ERROR | [x] The first parameter of a multi-line function declaration must be on the line after the opening bracket
 39 | ERROR | [x] Multi-line function declaration not indented correctly; expected 4 spaces but found 2
 39 | ERROR | [x] Multi-line function declarations must have a trailing comma after the last parameter
 39 | ERROR | [x] The closing parenthesis of a multi-line function declaration must be on a new line
----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineSchema.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------
 51 | WARNING | Possible useless method overriding detected
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/FieldSchemaForm.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
  9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityTypeManagerInterface.
 29 | ERROR | [x] Multi-line function declarations must have a trailing comma after the last parameter
 29 | ERROR | [x] The closing parenthesis of a multi-line function declaration must be on a new line
---------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/EngineStatusForm.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\Entity\EngineInterface.
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Form/SyncEngineConfirmForm.php
---------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Form\FormStateInterface.
---------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Entity/Engine.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\Utility\Database.
--------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Entity/Server.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Config\Entity\ConfigEntityBase.
--------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Plugin/Block/SearchUIBlock.php
---------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------------------------------------------------
  9 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityTypeManagerInterface.
 19 | ERROR | [ ] Class name doesn't match filename; expected "class SearchUIBlock"
 51 | ERROR | [x] Multi-line function declarations must have a trailing comma after the last parameter
---------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Tracker/TrackingManager.php
------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------------
 68 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Tracker/Tracker.php
----------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------------------------------------------------------------------------
  62 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  72 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 434 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
----------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/AppSearchClient.php
----------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Elastic\EnterpriseSearch\AppSearch\Request\GetEngine.
----------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Controller/EngineController.php
----------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\Entity\EngineInterface.
----------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Controller/ServerController.php
----------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\appsearch\Entity\ServerInterface.
----------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/EngineListBuilder.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\EntityInterface.
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/Database.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------------------------------
 6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\media\Entity\Media.
-----------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /Users/interns/Demo-site/drupal-org-issues/web/modules/contrib/elastic_enterprise_search/modules/appsearch/src/Utility/BatchHelper.php
--------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------------------------------------
 92 | ERROR   | Public method name "BatchHelper::wipe_all" is not in lowerCamel format
 99 | WARNING | Unused variable $page_count.
--------------------------------------------------------------------------------------------------------------------------------------------

Time: 1.06 secs; Memory: 12MB

āžœ  contrib

    Please check and advise.
    Thank you.

  • Merge request !8Issue #3357579: Fix the issues reported by phpcs ā†’ (Open) created by apaderno
  • Pipeline finished with Failed
    4 months ago
    Total: 49s
    #260225
  • Comment 4 months ago ā†’
    šŸ‡®šŸ‡¹Italy apaderno Brescia, šŸ‡®šŸ‡¹

    The issue summary still does not report which command has been used nor which arguments has been passed to that command.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024