The link to download jquery 1.6 leads to an empty search (since 1.6 is no longer a featured download).
Attached patches contain an updated link (http://code.google.com/p/jquery-ui/downloads/list?can=1&q=1.6).

(I don't really know the procedure for creating proper patches, but I'm attaching diffs for affected files.)

Cheers!

CommentFileSizeAuthor
#4 879446_jquery_ui_link_v2.patch2.05 KBjolidog
#1 879446_jquery_ui_link.patch734 bytesjolidog
README-linkupdate.patch146 bytesitangalo
jquery_ui_install-linkupdate.patch926 bytesitangalo

Comments

jolidog’s picture

jolidog commented
StatusFileSize
new 879446_jquery_ui_link.patch734 bytes

Yap, I just made a patch myself, attaching it here, since it also corrects the link for version 1.7 witch will be affected in the future.
I'm going to mark #362509: New jQuery UI release system as a duplicate since this a is a clean issue dealing with this.

Thanks

jolidog’s picture

jolidog commented
Priority: Normal » Major
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Needs work

We should also fix the link in jquery_ui_requirements() since that also points to the old one.

jolidog’s picture

jolidog commented
Status: Needs work » Needs review
StatusFileSize
new 879446_jquery_ui_link_v2.patch2.05 KB

Yap! Here it is the patch that fixes that too.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Awesome! That basically looks good to me. The only question is, is there a reason not to link to http://jquery-ui.googlecode.com/files/jquery.ui-1.6.zip directly?

jolidog’s picture

jolidog commented

Hum... I guess for the 1.6 version that's fine, since it's in legacy mode, but the 1.7 version had some revisions, and I think it might still have, so even if we linked to http://jquery-ui.googlecode.com/files/jquery-ui-1.7.3.zip people might not get the latest revision.

So, should the links behave differently?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

Nope, that sounds like solid reasoning to me. :)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Reviewed & tested by the community » Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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

AlanAtLarge’s picture

AlanAtLarge commented

That worked, nice an simple. Thank you so much, this has been a headache for me since I started with Drupal. Sometimes I got it to work, sometimes not, which file with which version, what directory, etc...... whew.
Alan