FILE: .../web/modules/contrib/diff_moderate/src/Plugin/views/field/DiffModerate.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AND 3 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------------
21 | WARNING | [ ] t() calls should be avoided in classes, use
| | \Drupal\Core\StringTranslation\StringTranslationTrait and
| | $this->t() instead
31 | WARNING | [ ] Unused variable $rev_key.
33 | WARNING | [ ] Line exceeds 80 characters; contains 87 characters
43 | ERROR | [x] Missing function doc comment
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------
FILE: ...es/contrib/diff_moderate/src/Controller/NodeModerateRevisionController.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
40 | WARNING | Line exceeds 80 characters; contains 92 characters
--------------------------------------------------------------------------------
Time: 293ms; Memory: 10MB
Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,js,info,txt,md,yml,twig diff_moderate/
Resolve all the issues for Drupal and DrupalPractice coding standards
Patch review.
1.0
Code
It involves compliance with, or the content of coding standards. Requires broad community agreement.
Review patch and tested locally. Mention all phpcs issues are resolved. RTBC +
Hi Rassoni,
I also reviewed MR!1, applied against Moderate diff 1.0.x-dev with Drupal core version 10.1.7, and confirmed it fixes the issue reported by phpcs.
I added screenshots for reference.
Thank you.