Hi Quick suggestions to the module.

1. Implement hook_uninstall() to delete the variables when we uninstall the module, following are set when we submit the config form (admin/config/content/linkchecker_summary_mail) add the file linkchecker_summary_mail.install

/**
 * Implements hook_uninstall()
 */
function linkchecker_summary_mail_uninstall() {
  variable_del('linkchecker_summary_mail_address');
  variable_del('linkchecker_summary_mail_enable_global');
  variable_del('linkchecker_summary_mail_interval');
  variable_del('linkchecker_summary_mail_last_checked');
  variable_del('linkchecker_summary_mail_notify_node_author');
}

2. add configure link in linkchecker_summary_mail.info file

core = 7.x
package = Ny Media
configure = admin/config/content/linkchecker_summary_mail

Comments

vishwac9 created an issue. See original summary.

vishwac9’s picture

Status: Active » Needs review
StatusFileSize
new1.06 KB

please find attached patch , happy to help :)

eiriksm’s picture

Cool! Thanks!

This was just something that was open sourced quickly. Do you mind also removing the "package" part of the info file, and then adding the same thing that linkchecker has?

vishwac9’s picture

StatusFileSize
new1.06 KB

hi @eriksm, i have removed the package line from linkchecker_summary_mail.info file. please find attached the updated patch.

  • eiriksm committed 17b83cb on 7.x-1.x authored by vishwac9
    Issue #2861275 by vishwac9, eiriksm: LinkChecker : Quick suggestions
    
eiriksm’s picture

Status: Needs review » Fixed

Great, thanks!

Committed!

Status: Fixed » Closed (fixed)

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