D6 ships with jquery 1.2.x so the dependency on the jquery_update module is not needed.

CommentFileSizeAuthor
navigate.info_.patch503 bytesjaydub

Comments

chrisshattuck’s picture

chrisshattuck commented

Did you happen to test it out without jquery_update installed? When I did, some of the jquery calls didn't work properly. I'll go ahead and test this out again, maybe it was a caching problem. It would sure be nice to get rid of the dependency.

Thanks,
Chris

jaydub’s picture

jaydub commented

well the jquery_update module says:

"Stock" Drupal 6.x: jQuery 1.2.x (Drupal 6.2 and below: 1.2.3, Drupal 6.3 and above: 1.2.6)
jQuery Update 6.x-1.x: jQuery 1.2.x (currently 1.2.6)

so the jquery that is in core should be no different than that in jquery_update especially if you have D6 core 6.3 or higher which with the rash of security updates lately most people should have now :)

I only tested in a cursory manner so don't take my word for it but core jquery should be fine.

redndahead’s picture

redndahead commented
Status: Needs review » Reviewed & tested by the community

I can confirm that jquery_update is not necessary at least in D6.6 I'm guessing all of D6 since it uses the jQuery 1.2.x codeline. Patch works

chrisshattuck’s picture

chrisshattuck commented
Status: Reviewed & tested by the community » Fixed

The dependency has been removed and the fix will be included in BETA3. Thanks!

Status: Fixed » Closed (fixed)

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