This might be bug report or a feature request depending on your point of view :D
And this may already reported, but I'd didn't spot it on a quick search.
Conflicts exist between devel_themer in the Devel module(which appears to includes a few versioned jquery ui files) and the Jquery_ui module, which allows a site to use specific (often newer) versions of the jquery_ui.
I propose that devel_themer ads some small logic on the order of:
if (module_exists('jquery_ui'))
{
jquery_ui_add('ui.core');
jquery_ui_add('ui.mouse');
jquery_ui_add('ui.draggable');
}
else
{
drual_add_js('...'); // what currently is: ui.mouse.js, and ui.draggable.js
}
I have been developing my theme and expecting my jquery ui stuff to fail for as long as I have the devel_themer turned on. Which is obviously not optimal, but also not critical.
Thanks for your consideration.
Comment | File | Size | Author |
---|---|---|---|
#19 | devel_themer-440854.patch | 849 bytes | JeremyFrench |
#5 | devel_440854.patch | 482 bytes | drewish |
#2 | devel-themer-440854.patch | 1.14 KB | jjeff |
Comments
Comment #1
jjeff CreditAttribution: jjeff commentedYes. Great idea. It was a bit of a hack that we just grabbed those files and stuck them in there, but since Drupal doesn't have a Javascript plugin management system, this was the only thing to do. Now that jQuery UI module exists, your idea is probably the way to go.
Comment #2
jjeff CreditAttribution: jjeff commentedComment #3
moshe weitzman CreditAttribution: moshe weitzman commentedcommitted. thanks.
Comment #5
drewish CreditAttribution: drewish commentedI'm re-opening this since it's still an issue though more minor than before, see: #578402
Basically we're got really old JQueryUI code and ui.mouse.js no longer exists.
Comment #6
moshe weitzman CreditAttribution: moshe weitzman commentedComment #7
Anonymous (not verified) CreditAttribution: Anonymous commentedThis didn't work for me. The error persists even on a brand new site with a totally different theme. It's quite crippling...
Comment #8
neoglez CreditAttribution: neoglez commentedSubscribing
Comment #9
a5342346 CreditAttribution: a5342346 commentedSubscribing
Comment #10
BBCSubscribing
Comment #11
a5342346 CreditAttribution: a5342346 commentedCan someone clarify if we can get around this problem yet, and how? Is patching still required, or just the right combination of module versions?
Atm, e.g., @
admin/build/panels/style
I see,
and, in
i.e., no "ui.mouse.min.js" to be found ...
atm i've,
Thanks,
Ben
Comment #12
druplicate CreditAttribution: druplicate commentedSee this thread for additional solutions: http://drupal.org/node/578402
I just ignore the warnings since devel isn't used on a live site.
Comment #13
butler360 CreditAttribution: butler360 commentedNow that the thread linked to above is marked as a duplicate of this thread, what's the status here?
Comment #14
achtonSubscribing. I believe this is still an issue.
Comment #15
gfxguru CreditAttribution: gfxguru commentedsubscribing. Yes still...
Comment #16
lucascaro CreditAttribution: lucascaro commentedsubscribing + bump
Comment #17
scarvajal CreditAttribution: scarvajal commentedsubscribing
Comment #18
YK85 CreditAttribution: YK85 commentedsubscribing - any further development in this area?
i see the error in the logs every time =(
Comment #19
JeremyFrench CreditAttribution: JeremyFrench commentedPatch attached. Is basically the patch in #5 by drewish but checking the jquery ui version to make sure the file is not ommited if version is less than 1.5.
Any takers for a review?
Comment #20
HongPong CreditAttribution: HongPong commentedsubscribe -thanx
Comment #21
JeremyFrench CreditAttribution: JeremyFrench commentedCommited here
Comment #23
spgd01 CreditAttribution: spgd01 commentedHas this patch been reviewed? I am still getting the error. #21 dosen't help without direct link to the proper page.