Skip to content

Commit c7c5b6b

Browse files
committed
Widget: Wrap the remove event trigerring in a try/catch. Fixes #7510 - jQuery.data throws a script error in certain circumstances.
1 parent 1b5af10 commit c7c5b6b

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

ui/jquery.ui.widget.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,10 @@ if ( $.cleanData ) {
1414
var _cleanData = $.cleanData;
1515
$.cleanData = function( elems ) {
1616
for ( var i = 0, elem; (elem = elems[i]) != null; i++ ) {
17-
$( elem ).triggerHandler( "remove" );
17+
try {
18+
$( elem ).triggerHandler( "remove" );
19+
// http://bugs.jquery.com/ticket/8235
20+
} catch( e ) {}
1821
}
1922
_cleanData( elems );
2023
};
@@ -25,7 +28,10 @@ if ( $.cleanData ) {
2528
if ( !keepData ) {
2629
if ( !selector || $.filter( selector, [ this ] ).length ) {
2730
$( "*", this ).add( [ this ] ).each(function() {
28-
$( this ).triggerHandler( "remove" );
31+
try {
32+
$( this ).triggerHandler( "remove" );
33+
// http://bugs.jquery.com/ticket/8235
34+
} catch( e ) {}
2935
});
3036
}
3137
}

0 commit comments

Comments
 (0)