FILE: /home/addweb/Drupal8-vagrant/web/web/entityform_block/tests/src/Functional/EntityFormBlockTest.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AND 5 WARNINGS AFFECTING 12 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
24 | ERROR | [x] Short array syntax must be used to define arrays
30 | WARNING | [x] A comma should follow the last multiline array item. Found: 'contact'
38 | ERROR | [x] Short array syntax must be used to define arrays
40 | ERROR | [x] Short array syntax must be used to define arrays
59 | ERROR | [x] Short array syntax must be used to define arrays
62 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
73 | ERROR | [x] Short array syntax must be used to define arrays
78 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
83 | ERROR | [x] Short array syntax must be used to define arrays
86 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
96 | ERROR | [x] Short array syntax must be used to define arrays
99 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
---------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 8 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------------------------------
FILE: /home/addweb/Drupal8-vagrant/web/web/entityform_block/tests/src/Functional/LocaleTest.php
-----------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 3 LINES
-----------------------------------------------------------------------------------------------
15 | ERROR | [ ] Missing member variable doc comment
27 | ERROR | [x] Expected 1 space after "="; 2 found
29 | ERROR | [x] Short array syntax must be used to define arrays
-----------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------------
FILE: /home/addweb/Drupal8-vagrant/web/web/entityform_block/src/Plugin/Block/EntityEditFormBlock.php
----------------------------------------------------------------------------------------------------
FOUND 7 ERRORS AND 2 WARNINGS AFFECTING 9 LINES
----------------------------------------------------------------------------------------------------
76 | ERROR | [x] Short array syntax must be used to define arrays
105 | ERROR | [x] Short array syntax must be used to define arrays
123 | ERROR | [x] Short array syntax must be used to define arrays
148 | ERROR | [x] Short array syntax must be used to define arrays
152 | ERROR | [x] Short array syntax must be used to define arrays
160 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
166 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
167 | ERROR | [x] Expected 1 blank line after function; 0 found
168 | ERROR | [x] The closing brace for the class must have an empty line before it
----------------------------------------------------------------------------------------------------
| Comment | File | Size | Author |
|---|---|---|---|
| resolve_phpcs_issues.patch | 9.79 KB | silvi.addweb |
Comments
Comment #2
clarkssquared commentedHi
I applied the patch and I confirmed that it fixes the PHPCS issues.
RTBC +1 from me. retaining the status to needs review for others to verify the patch