LINKCHECKER_DEFAULT_FILTER_BLACKLIST is defined in .module but not in .install.
Notice: Use of undefined constant LINKCHECKER_DEFAULT_FILTER_BLACKLIST - assumed 'LINKCHECKER_DEFAULT_FILTER_BLACKLIST' in linkchecker_update_7007() (line 381 of /var/www/example.com/public_html/sites/all/modules/linkchecker/linkchecker.install).

You could load the .module file at the start of linkchecker_update_7007 to use the same definition.

CommentFileSizeAuthor
#2 linkchecker.patch580 bytespeterx

Comments

peterx’s picture

peterx commented

update.php reported the following with no mention of the error:
Update #7007
Updated the blacklisted internal filter names from Drupal 6 to Drupal 7. The update has only updated the blacklisted filters linkchecker is aware of. Please review the "Filters disabled for link extraction" settings, if all filters with references to other content are still disabled.

peterx’s picture

peterx commented
StatusFileSize
new linkchecker.patch580 bytes

Try this patch.

hass’s picture

hass commented

Thank you very much for reporting this! I remember something like this from any documentation, but have not seen the Notice myself and cannot remember the details. How can i repro this notice?

peterx’s picture

peterx commented
Status: Active » Needs review

@hass, I update a dev version from December to the Jan 3 dev version. The Jan 3 dev version included update 7007 and produced the message.

I applied the patch (manually) to another site and updated to 7007. There was no error message. I marked this issue as needs review because I applied the patch by hand instead of the patch file.

Status: Needs review » Needs work

The last submitted patch, linkchecker.patch, failed testing.

hass’s picture

hass commented

I'm highly confused... See http://drupal.org/update/modules/6/7#module_file_during_install please.

hass’s picture

hass commented
Title: LINKCHECKER_DEFAULT_FILTER_BLACKLIST » Notice: Use of undefined constant LINKCHECKER_DEFAULT_FILTER_BLACKLIST
Status: Needs work » Fixed
peterx’s picture

peterx commented

If hook_requirements($phase) is working, perhaps you need files[] = linkchecker.module or the requirements list might be wrong.

hass’s picture

hass commented

Maybe... This is a "closed book" for me... No clue how this works.

peterx’s picture

peterx commented

See Media module. They load .module for every function except hook_install. The automated load is for hook_install, not hook_update_*. The same for uc_credit and others. Many modules load .module for uninstall.

Status: Fixed » Closed (fixed)

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