Add dispose logic for jQuery objects

Partial fix for #37
walmartlabs · Jul 10, 2014 · d4e4fcf · d4e4fcf
1 parent c9fb8d8
commit d4e4fcf
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/lib/jquery/ajax.js b/lib/jquery/ajax.js
@@ -59,6 +59,9 @@ module.exports = exports = function Ajax(window, exec, ajaxCache) {
 
     this.removeAllListeners();
   };
+  ajax.dispose = function() {
+    window = window.$.ajax = undefined;
+  };
 
   window.$.ajax = function(options) {
     var originalUrl = options.url,

diff --git a/lib/jquery/index.js b/lib/jquery/index.js
@@ -121,10 +121,17 @@ module.exports = exports = function jQuery(window, html, exec, ajaxCache) {
 
   window.jQuery = window.Zepto = window.$ = $;
 
+  var ajaxInstance = ajax(window, exec, ajaxCache);
   return {
     $: $,
     root: root,
-    ajax: ajax(window, exec, ajaxCache)
+    ajax: ajaxInstance,
+
+    dispose: function() {
+      ajaxInstance.dispose();
+
+      window = window.jQuery = window.Zepto = window.$ = undefined;
+    }
   };
 };
 

diff --git a/lib/page.js b/lib/page.js
@@ -234,6 +234,7 @@ module.exports = exports = function(options) {
 
   function loadPage(src) {
     $ = page.$ = jQuery(window, src, exec, ajaxCache);
+    toCleanup.push($);
 
     pending.push('beforeExec', 1);
     if (options.beforeExec) {
@@ -321,7 +322,7 @@ module.exports = exports = function(options) {
       options = callback =
           window = baseContext = context =
           toReset = toCleanup = location = $ =
-          window.FruitLoops =
+          window.FruitLoops = window.emit = window.onEmit = window.loadInContext =
           emitCallbacks = scripts =
           page.window = page.emit = page.$ = undefined;
     },