- Status changed to Needs work
over 1 year ago 12:35pm 2 March 2023 - 🇦🇺Australia VladimirAus Brisbane, Australia
Thanks for contributions. 🥂
Switching to MRs, please commit to gitlab.phpcs returns
FILE: /Users/tesboss/_PROJECTS/drupal/taxonomy_manager/src/Form/MoveTermsForm.php ------------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ------------------------------------------------------------------------------------------------------------------------------------- 63 | ERROR | The array declaration extends to column 90 (the limit is 80). The array content should be split up over multiple lines ------------------------------------------------------------------------------------------------------------------------------------- FILE: /Users/tesboss/_PROJECTS/drupal/taxonomy_manager/src/Form/ExportTermsMiniForm.php --------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES --------------------------------------------------------------------------------------- 60 | WARNING | Unused variable $desccsv. 78 | WARNING | Possible useless method overriding detected --------------------------------------------------------------------------------------- FILE: /Users/tesboss/_PROJECTS/drupal/taxonomy_manager/src/Form/ExportTermsForm.php ---------------------------------------------------------------------------------------------- FOUND 3 ERRORS AND 4 WARNINGS AFFECTING 7 LINES ---------------------------------------------------------------------------------------------- 94 | WARNING | Line exceeds 80 characters; contains 98 characters 121 | ERROR | Comment indentation error, expected only 1 spaces 122 | ERROR | Comment indentation error, expected only 8 spaces 133 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 141 | WARNING | Possible useless method overriding detected 170 | ERROR | Comment indentation error, expected only 1 spaces 182 | WARNING | Line exceeds 80 characters; contains 86 characters ---------------------------------------------------------------------------------------------- FILE: /Users/tesboss/_PROJECTS/drupal/taxonomy_manager/src/Form/TaxonomyManagerForm.php ------------------------------------------------------------------------------------------------------------------------------------------ FOUND 3 ERRORS AND 5 WARNINGS AFFECTING 7 LINES ------------------------------------------------------------------------------------------------------------------------------------------ 77 | ERROR | Parameter $entity_form_builder is not described in comment 222 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 227 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 265 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 265 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 268 | WARNING | t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead 317 | ERROR | Public method name "TaxonomyManagerForm::taxonomy_term_submit_handler" is not in lowerCamel format 459 | ERROR | Doc comment is empty ------------------------------------------------------------------------------------------------------------------------------------------ FILE: /Users/tesboss/_PROJECTS/drupal/taxonomy_manager/src/TaxonomyManagerHelper.php ---------------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES ---------------------------------------------------------------------------------------------------------------------------------------- 155 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead 259 | ERROR | The array declaration extends to column 97 (the limit is 80). The array content should be split up over multiple lines ---------------------------------------------------------------------------------------------------------------------------------------- Time: 827ms; Memory: 12MB
- @vladimiraus opened merge request.
- Status changed to Needs review
over 1 year ago 1:13pm 2 March 2023