Closed (fixed)
Project:
jQuery CDN
Version:
7.x-1.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
23 Nov 2011 at 13:25 UTC
Updated:
3 Nov 2013 at 17:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
leandro713 commentedi have observed the problem and i guess this module sometimes doesnt work because Drupal not always (at least my D7 installation on my testing machine at work) loads jquery.ui
in order to be sure Drupal loads it, you can get working this module pasting the following code to the «jquery_cdn.module» file (maybe at the top of it)
adding this it works for me. i will try to commit this lines to the project afterwards.
-Leandro
Comment #2
leandro713 commented@creatile, if you dont want to edit code, i attach patch for your convenience :)
Comment #3
Anonymous (not verified) commentedGiven access to leandro713 to commit his recommendation.
Comment #4
Anonymous (not verified) commentedComment #6
An43 commentedI think that forcing ui library to load is not the best way to fix this. Drupal core doesn't load jquery.ui as default library and it's not always needed.
This problem is caused by modules/themes that load submodules of jquery.ui without loading jquery.ui core, assuming it will be loaded by the dependencies. So if we unset all ui submodules items, drupal_get_library() returns FALSE and no libraries are loaded.
This patch will cause loading jquery.ui as dependency only if it's needed.
Comment #7
An43 commentedSorry, i made a mistake while moving changes from my test server. Here is the right version of the patch.
Comment #8
leandro713 commentedAn43 patch applied
sorry An43 for the long, long, long delay