It took me a while to figure out what the problem was why it was not working for me, but it appeared there is a selector in the javascript file with a reference using the @ sign which is no longer supported by jQuery 1.3.

Simply removing it is enough to get it working for both jQuery 1.2.6 (standard in drupal 6.x) and jQuery 1.3.

CommentFileSizeAuthor
#2 091002-remove-at-from-jquery-selector.patch377 bytesaloyr

Comments

aloyr’s picture

aloyr commented

Thanks TCRobbert! I will give this a try. I have had the same issue and the solution I had was very Rube Goldbergish! It involved creating hook_preprocess_page and overriding the jquery_update module to use the old one for certain urls... not pretty, but functional.

aloyr’s picture

aloyr commented
Status: Active » Needs review
StatusFileSize
new 091002-remove-at-from-jquery-selector.patch377 bytes

that worked out great! here is the [very simple] patch based on tcrobbert's solution.

this is my 2nd patch contribution and i am still trying to figure the process out, so forgive (and point out) any non-compliance...

stg11’s picture

stg11 commented

The patch worked for me. I am hoping this will be put in the next release.

rootwork’s picture

rootwork
he/him
Primary language English
Location 🇺🇸 US-Pacific 🌎 Chinook (Multnomah, Clackamas) & Cowlitz lands 🌹 Portland, OR
commented
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

This patch worked for me.

Can we get this patch applied and get a full release (or at least a beta release) posted? This module has NEVER had a full release, in more than four years of existence! It's time to move SOMETHING out of dev.

Happy to test if there are other bugs, but it seems like this is a pretty straightforward module that just includes the plugin. Let's get a real release posted.