I have the latest jquery_update-7.x-2.x-dev installed and use jquery 1.7 for the front-end theme and jquery 1.5 for the backend theme. On the frontend theme, I am getting the following error in all browsers:

Uncaught TypeError: Cannot set property 'events' of undefined jquery.ui.dialog.patch.js:24

I fixed this by adding an extra check to make sure that $.ui.dialog.overlay is defined in jquery.ui.dialog.patch.js:24.

CommentFileSizeAuthor
#4 views-check_ui_dialog_overlay-1995892-4.patch888 byteshswong3i
cannot-set-property-events-as-undefined.patch650 bytesjlyon

Comments

Status: Needs review » Needs work

The last submitted patch, cannot-set-property-events-as-undefined.patch, failed testing.

jason.fisher’s picture

jason.fisher commented

Patch doesn't apply cleanly but the fix works for jQuery 1.7/1.8.

alayham’s picture

alayham commented

This patch solved my problem, although it does not apply properly.

hswong3i’s picture

hswong3i commented
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new views-check_ui_dialog_overlay-1995892-4.patch888 bytes

Patch reroll via GIT 7.x-3.x, and now applied to drustack_core-7.x-24.x.

BrockBoland’s picture

BrockBoland commented
Status: Needs review » Reviewed & tested by the community

#4 looks good and applies cleanly.

hadsie’s picture

hadsie commented

#4 works for me too. thanks!

aprohl5’s picture

aprohl5 commented

This patch applied cleanly for me as well and seems to fix the problem on 7.x-3.8. It would be great to see this patch make it into the views module.

  • dawehner committed 0607d83 on authored by hswong3i 
    Issue #1995892 by hswong3i, jlyon: Fixed JS error: Cannot set property '...
dawehner’s picture

dawehner
Primary language German
commented
Status: Reviewed & tested by the community » Fixed

Seems legit, thank you for the patch!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.