From 269fb43b3667c74b35980798fd54937d65b7b7de Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Sat, 24 Nov 2012 01:51:19 +0100 Subject: fix(jqLite): fire $destroy event via triggerHandler in jQuery 1.8.x the data() data structure is changed and events are not accessible via data().events. Since all we need is to trigger all event handlers, we can do so via triggerHandler() api instead of mocking with the internal jQuery data structures. This fix was originally proposed by PeteAppleton via PR #1512. Closes #1512 --- src/jqLite.js | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) (limited to 'src') diff --git a/src/jqLite.js b/src/jqLite.js index 90a363f3..26be5977 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -130,12 +130,7 @@ function JQLitePatchJQueryRemove(name, dispatchThis) { for(setIndex = 0, setLength = set.length; setIndex < setLength; setIndex++) { element = jqLite(set[setIndex]); if (fireEvent) { - events = element.data('events'); - if ( (fns = events && events.$destroy) ) { - forEach(fns, function(fn){ - fn.handler(); - }); - } + element.triggerHandler('$destroy'); } else { fireEvent = !fireEvent; } -- cgit v1.2.3