Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The array declaration extends to column 101 (the limit is 80). The array content should be split up over multiple lines
(Drupal.Arrays.Array.LongLineDeclaration)
Closing parenthesis of array declaration must be on a new line
(Squiz.Arrays.ArrayDeclaration.CloseBraceNewLine)
Unused use statement
(Drupal.Classes.UnusedUseStatement.UnusedUse)
Missing file doc comment (Drupal.Commenting.FileComment.Missing)
Comment | File | Size | Author |
---|---|---|---|
#2 | email_change_verification-fix_drupal_coding_standard_and_drupal_practice-3229392-14197606-1.x.patch | 2.43 KB | omrmankar |
Comments
Comment #2
omrmankarHi,
I have created a patch that is working according to the phpcs standards and resolves all errors.
The patch is attached.
Comment #3
darvanenUnnecessary change.
Comment #4
omrmankar@darvanen it just sorts the namespace doesn't affect in performance using code. you can commit these changes.
Comment #6
daften CreditAttribution: daften at Dropsolid commentedFix incorporated and I've also fixed some direct t calls with $this->t in the ConfigForm class. The code now fully adheres to Drupal and DrupalPractice coding standards. Thanks omrmankar and darvanen!