Skip to content

Commit 1f2011e

Browse files
committed
Widget: Don't swallow errors in remove events
The try/catch was only there to support jQuery <1.6.3, which we no longer support. Ref jquery/jquery#3554
1 parent 278d1e1 commit 1f2011e

File tree

1 file changed

+5
-9
lines changed

1 file changed

+5
-9
lines changed

ui/widget.js

+5-9
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,12 @@ $.cleanData = ( function( orig ) {
3232
return function( elems ) {
3333
var events, elem, i;
3434
for ( i = 0; ( elem = elems[ i ] ) != null; i++ ) {
35-
try {
3635

37-
// Only trigger remove when necessary to save time
38-
events = $._data( elem, "events" );
39-
if ( events && events.remove ) {
40-
$( elem ).triggerHandler( "remove" );
41-
}
42-
43-
// Http://bugs.jquery.com/ticket/8235
44-
} catch ( e ) {}
36+
// Only trigger remove when necessary to save time
37+
events = $._data( elem, "events" );
38+
if ( events && events.remove ) {
39+
$( elem ).triggerHandler( "remove" );
40+
}
4541
}
4642
orig( elems );
4743
};

0 commit comments

Comments
 (0)