Problem/Motivation
Getting following warnings.
FILE: /app/modules/contrib/anonymous_login/src/EventSubscriber/AnonymousLoginSubscriber.php
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------
24 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
-----------------------------------------------------------------------------------------------------------------
FILE: /app/modules/contrib/anonymous_login/src/Form/AnonymousLoginSettings.php
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------
13 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
-----------------------------------------------------------------------------------------------------------------
Time: 916ms; Memory: 12MB
Steps to reproduce
Run following command
phpcs --standard="Drupal,DrupalPractice" --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml modules/contrib/anonymous_login/
Proposed resolution
Above warnings need to be fixed.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | Screenshot 2023-02-15 at 12.14.27 PM.png | 48.5 KB | arunkumark |
| #2 | 3329751-2.patch | 968 bytes | samitk |
Issue fork anonymous_login-3329751
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
samitk commentedError/warnings are fixed.
Comment #5
charchil khandelwal commentedCreated MR for this issue, Please review.
Thank You
Comment #6
arunkumarkThe patch was applied cleanly and there is no other Coding standard issue.
Moving this issue to RTBC.
Comment #7
japerry