After enabling this module I need to check readme.txt for detils, so if it is better if you implement hook_help() that will give help option in module listing.

CommentFileSizeAuthor
#7 disable_messages-implement_hook_help_for_drupal_7-2722023-7.patch5.03 KBnitinsp
#4 disable_messages-implement_hook_help_for_drupal_7-2722023-4.patch1.01 KBnitinsp

Comments

NitinSP created an issue. See original summary.

nitinsp’s picture

nitinsp commented
Assigned: Unassigned » nitinsp
nitinsp’s picture

nitinsp commented
Issue summary: View changes
nitinsp’s picture

nitinsp commented
Assigned: nitinsp » Unassigned
Status: Active » Needs review
StatusFileSize
new disable_messages-implement_hook_help_for_drupal_7-2722023-4.patch1.01 KB

As per requirement I have implemented hook_help() for drupal 7 version of this module and created patch file of that. Please apply and review patch.

anish_zyxware’s picture

anish_zyxware commented
Related issues: +#2721583: Implement hook_help

This needs more data. Can you do a backport of the issue that's added as related issue?

Ie, just use the content from the 8.x module.

nitinsp’s picture

nitinsp commented
Assigned: Unassigned » nitinsp
nitinsp’s picture

nitinsp commented
Assigned: nitinsp » Unassigned
StatusFileSize
new disable_messages-implement_hook_help_for_drupal_7-2722023-7.patch5.03 KB

@anish_zyxware as per suggestion I have updated hook_help(), please apply & review the patch.

  • anish.a committed 73ba93f on 7.x-1.x authored by NitinSP 
    Issue #2722023 by NitinSP: Implement hook_help for drupal 7
anish_zyxware’s picture

anish_zyxware commented
Status: Needs review » Fixed

Merged. Thanks for the patch.

Status: Fixed » Closed (fixed)

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