Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
$_POST['ajax_page_state']['jquery_version'] fixed jquery_version but not fixed jQuery UI version
jquery_update_jquery_replace($javascript, $cdn, $path, $min, $version); // $version are fixed inside
if (version_compare($version, '1.6', '>=')) { // non fixed $version
jquery_update_jqueryui_replace($javascript, $cdn, $path, $min);
}
Comment | File | Size | Author |
---|---|---|---|
#8 | jquery_update-2056413-08.patch | 1.29 KB | sylus |
#1 | jquery_update-2056413.patch | 1.22 KB | Punk_UnDeaD |
Comments
Comment #1
Punk_UnDeaDComment #2
Punk_UnDeaDComment #3
gooddesignusa CreditAttribution: gooddesignusa commentedThank you so much. Patch installed cleanly and fixed my issues.
I installed the latest dev of jquery_update the other night to take advantage of the new option to set a different jquery version for admin pages. Without this patch the file widget 'add another' for a media field would not work when editing a node. It threw jquery errors in the console b/c of the wrong jquery.ui version.
Comment #4
Tomefa CreditAttribution: Tomefa commentedI just pass the last 2 hours to find a fix for this issue
And i was on my way to post it before seeing your post :)
That's a major fix for jquery_update because it makes all the ajax request working correctly.
It should be had on the next dev version.
Thanks Punk_UnDeaD
Comment #5
samtny CreditAttribution: samtny commentedI made the same change by hand - works well for me. In my case I had jquery version "1.5" set on Admin pages, but AJAX requests were adding jquery ui 10.2 back to the page (erroneously), as the module could not determine from the ajax request that i was on an admin path.
Comment #6
samtny CreditAttribution: samtny commentedComment #7
yareckon CreditAttribution: yareckon commentedThis is an important patch that we needed in production, please do add it to the dev branch as soon as possible.
Comment #8
sylus CreditAttribution: sylus commentedUpdating patch to latest dev
Comment #9
chr.fritschWorks fine for me
Comment #10
Robin_K CreditAttribution: Robin_K commentedMaybe its worth noting that this patch is also part of the patch in #1548028: Make the default jQuery version (1.4.4) for D7 an option
Comment #11
Punk_UnDeaDComment #12
Punk_UnDeaDComment #13
markhalliwellThis was fixed by #1548028: Make the default jQuery version (1.4.4) for D7 an option.