Active
Project:
Theme developer
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Apr 2011 at 18:30 UTC
Updated:
15 Apr 2011 at 21:20 UTC
Hello there, just wanted to let you know that this module needs to load it's JS files sooner in the chain. Right now they load somewhere in the middle/end of the JS load order, which means that other modules that are dependent on it, like devel_themer, throw an error. To reproduce: install Jquery_update, Jquery_ui and devel_themer. Devel_themer will complain about an unknown function: dragable. To fix:
I disabled your module, and then manually call the jquery_ui file from my theme instead and load it as drupal_add_js (/filename/jquery_ui.min.js, "core"); and voila, problem resolved -- however I had to bypass your module to resolve it... probably not the intention of the module.
Thanks!
Comments
Comment #1
datarazor commentedoh and I should add I also tried to change module weights, but this had zero effect.
Comment #2
corey.aufang commentedThis is actually a bug in the devel_themer module.
I looked through the module and it seems that it doesn't have very flexible code into how it tries to call jquery_ui module, or how to account for the dependencies that change between revisions of jQuery UI library.
jQuery UI draggable dependencies:
1.6.x - ui.core.js
1.7.x - ui.core.js
1.8.x - jquery.ui.core.js, jquery.ui.mouse.js, jquery.ui.widget.js
devel_themer module checks to see if the version of jQuery UI is less than 1.5 (really old) and tries to make sure that ui.mouse is there (only required at version 1.8.x+ of jQuery UI).
At no point does it try to load ui.widget, which is required for ver 1.8.x of jQuery UI
Your solution bypasses the individual files and loads the entire jQuery UI library, and hence doesn't have to deal with dependencies.