It seems that jqmulti does nothing if jquery_update is installed. Looking at the code it seems that it is using a jquery_update api hook which only exists in the dev branch but not the release branch of jquery_update. Perhaps the dependency check should be updated to look for the jquery_update version as well.

CommentFileSizeAuthor
#3 check_jquery_update_hook-1567680.patch678 bytesgoron

Comments

goron’s picture

Title: Incompatible with jquery_update » Check for existence of jquery_update hook and warn user if not present
Category: bug » feature
Priority: Major » Normal

This isn't a bug. The module is compatible with jquery_update, though it does require the dev version. This is stated in bold on the module's project page.

I do agree still that it would be nice to do some kind of check for this. It can't be done with a dependency because the module is not dependent on jquery_update, it's optional, and also dependency checking for dev versions doesn't work well.

Given all that, I think it would be nice to add a check to hook_requirements() which, if jquery_update is enabled, checks for the existence of the hook and warns if it doesn't exist that a newer version of jquery_update is required for compatibility.

This is a low priority for me, so a patch would speed things up.

goron’s picture

Status: Active » Patch (to be ported)

Fixed in 6.x version. Now checks for the hook when jqmulti is enabled (still doesn't warn if jquery_update is enabled after jqmulti, but oh well).

Fix in D7.

goron’s picture

StatusFileSize
new678 bytes

patch