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
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | linkchecker_quick-2861275-4.patch | 1.06 KB | vishwac9 |
| #2 | linkchecker_quick-2861275-2.patch | 1.06 KB | vishwac9 |
Comments
Comment #2
vishwac9 commentedplease find attached patch , happy to help :)
Comment #3
eiriksmCool! 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?
Comment #4
vishwac9 commentedhi @eriksm, i have removed the package line from linkchecker_summary_mail.info file. please find attached the updated patch.
Comment #6
eiriksmGreat, thanks!
Committed!