Closed (fixed)
Project:
IMCE
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Aug 2007 at 03:12 UTC
Updated:
29 Mar 2008 at 02:33 UTC
Jump to comment: Most recent file
I added jquery_update and imce started failing. Firebug had a failure message in imce_set_tinymce.js saying that "unclick" was unknown. I found this bug report against Panels dealing with the same issue: http://drupal.org/node/163509
From that I did the following change and now imce is now working. The important thing is to call unbind('click') rather than unclick()
Index: imce_set_inline.js
===================================================================
--- imce_set_inline.js (revision 137)
+++ imce_set_inline.js (working copy)
@@ -7,7 +7,8 @@
function imceInitiateInline() {
$('a.imce-insert-inline').each( function () {
$(this.parentNode).css('display', 'block');
- $(this).unclick().click(function() {
+ $(this).unbind('click')
+ $(this).click(function() {
imceActiveTextarea = $('#'+this.name.split('-IMCE-')[0]).get(0);
imceActiveType = this.name.split('-IMCE-')[1];
window.open(this.href, '_imce_', 'width=640, height=480, resizable=1');
Hopefully the diff/patch format is appropriate for y'all to digest.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | imce-unmouse.patch | 504 bytes | noelbush |
Comments
Comment #1
ajayg commentedI reviewed the code and confirmed this works in 5.x
+1 for getting this patch in.
Comment #2
ajayg commentedComment #3
noelbush commentedSimilarly, imce_extras.js has a line that needs to be corrected like so (also attached):
Comment #4
ufku commentedcommited. thanks
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.