Skip to content

Commit

Permalink
Widget: Don't swallow errors in remove events
Browse files Browse the repository at this point in the history
The try/catch was only there to support jQuery <1.6.3, which we no
longer support.

Ref jquery/jquery#3554
  • Loading branch information
scottgonzalez committed Feb 28, 2017
1 parent 278d1e1 commit 1f2011e
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions ui/widget.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,16 +32,12 @@ $.cleanData = ( function( orig ) {
return function( elems ) {
var events, elem, i;
for ( i = 0; ( elem = elems[ i ] ) != null; i++ ) {
try {

// Only trigger remove when necessary to save time
events = $._data( elem, "events" );
if ( events && events.remove ) {
$( elem ).triggerHandler( "remove" );
}

// Http://bugs.jquery.com/ticket/8235
} catch ( e ) {}
// Only trigger remove when necessary to save time
events = $._data( elem, "events" );
if ( events && events.remove ) {
$( elem ).triggerHandler( "remove" );
}
}
orig( elems );
};
Expand Down

0 comments on commit 1f2011e

Please sign in to comment.