Problem/Motivation

FILE: /var/www/html/web/modules/onetimeloginlink/src/Form/OneTimeLoginLinkForm.php
---------------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AND 2 WARNINGS AFFECTING 6 LINES
---------------------------------------------------------------------------------------------------------------------------------------------
14 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
24 | ERROR | [ ] Missing short description in doc comment
41 | ERROR | [x] Expected 1 blank line after function; 3 found
44 | ERROR | [ ] More than 2 empty lines are not allowed
60 | WARNING | [ ] t() calls should be avoided in classes, use \Drupal\Core\StringTranslation\StringTranslationTrait and $this->t() instead
82 | ERROR | [x] Expected 1 space after IF keyword; 0 found
82 | ERROR | [x] Expected 1 space after closing parenthesis; found 5

Steps to reproduce

./vendor/bin/phpcs --standard="Drupal,DrupalPractice" --extensions="php,module,inc,install,test,profile,theme" web/modules/

Proposed resolution

./vendor/bin/phpcbf --standard="Drupal,DrupalPractice" --extensions="php,module,inc,install,test,profile,theme" web/modules/
After that fix the remaining issue manualy

Command icon 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

Nishant created an issue. See original summary.

nishant’s picture

Attached Patch for the above issue, Please verify

nishant’s picture

Assigned: nishant » Unassigned
Status: Needs work » Needs review
nishant’s picture

StatusFileSize
new2.96 KB

Attached the Latest patch#4,Please review.

Rajan Kumar made their first commit to this issue’s fork.

rajan kumar’s picture

Status: Needs review » Fixed

I have merged above patch in latest version.

rajan kumar’s picture

Status: Fixed » Closed (fixed)