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

CommentFileSizeAuthor
#2 addthis-codecleanup-2580801-1.patch2.98 KBtusharbodke

Comments

tusharbodke created an issue. See original summary.

tusharbodke’s picture

StatusFileSize
new2.98 KB

Hi,

Here is clean code patch to remove above listed issue.

Thanks

tusharbodke’s picture

Status: Active » Needs review
tusharbodke’s picture

Issue tags: -code cleanup +Code standards issues fixed
nitesh sethia’s picture

Assigned: tusharbodke » Unassigned
Status: Needs review » Reviewed & tested by the community

Marking this issue as Reviewed and tested by community. This patch fixes all the Drupal coding issue.

  • gisle committed c53e51a on 7.x-4.x authored by tusharbodke
    Issue #2580801 by tusharbodke: Drupal conding standards issue found
    
matglas86’s picture

Status: Reviewed & tested by the community » Fixed

Commited by @gisle 2016-03-07.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.