Hi,
Here is list of problem listed reagrding Drupal coding standards.
FILE: addthis/addthis.module
----------------------------------------------------------------------
FOUND 7 ERRORS AND 2 WARNINGS AFFECTING 8 LINES
----------------------------------------------------------------------
5 | WARNING | Line exceeds 80 characters; contains 93 characters
5 | ERROR | Doc comment short description must be on a single
| | line, further text should be a separate paragraph
71 | ERROR | Do not use t() in hook_menu()
75 | ERROR | Do not use t() in hook_menu()
166 | WARNING | Format should be "* Implements hook_foo().", "*
| | Implements hook_foo_BAR_ID_bar() for xyz_bar().",,
| | "* Implements hook_foo_BAR_ID_bar() for
| | xyz-bar.html.twig.", or "* Implements
| | hook_foo_BAR_ID_bar() for xyz-bar.tpl.php.".
175 | ERROR | Doc comment is empty
183 | ERROR | Doc comment is empty
311 | ERROR | Doc comment is empty
318 | ERROR | Doc comment is empty
----------------------------------------------------------------------
Thanks
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | addthis-codecleanup-2580801-1.patch | 2.98 KB | tusharbodke |
Comments
Comment #2
tusharbodke commentedHi,
Here is clean code patch to remove above listed issue.
Thanks
Comment #3
tusharbodke commentedComment #4
tusharbodke commentedComment #5
nitesh sethia commentedMarking this issue as Reviewed and tested by community. This patch fixes all the Drupal coding issue.
Comment #7
matglas86 commentedCommited by @gisle 2016-03-07.