Merge branch 'destroy-all' of git://github.com/lsmith/yui3 into lsmit…

…h-destroy-all

src/node/HISTORY.md

@@ -1,6 +1,11 @@
 Node Change History
 ===================
 
+@VERSION@
+-----
+
+* Fix node.all() to return an empty NodeList if the node was destroyed - Fixes #580 (hat tip Dallas Wheeler)
+
 3.9.1
 -----
 

src/node/js/node-core.js

@@ -634,10 +634,15 @@ Y.mix(Y_Node.prototype, {
      * @return {NodeList} A NodeList instance for the matching HTMLCollection/Array.
      */
     all: function(selector) {
-        var nodelist = Y.all(Y.Selector.query(selector, this._node));
-        nodelist._query = selector;
-        nodelist._queryRoot = this._node;
-        return nodelist;
+        var nodelist;
+
+        if (this._node) {
+            nodelist = Y.all(Y.Selector.query(selector, this._node));
+            nodelist._query = selector;
+            nodelist._queryRoot = this._node;
+        }
+
+        return nodelist || Y.all([]);
     },
 
     // TODO: allow fn test

src/node/tests/unit/node.html

@@ -1105,6 +1105,16 @@ <h2>test</h2>
             Assert.isTrue(Y.DOM.isWindow(Y.all(window)._nodes[0], "Y.all(window)"));
         },
 
+        test_all_after_destroy: function () {
+            var node = Y.one('#test-form');
+
+            Assert.isTrue((node.all('*').size() > 0), "node.all('*').size() should be > 0");
+
+            node.destroy();
+
+            Assert.areSame(0, node.all('*').size(), "destroyed node.all('*').size() should be 0");
+        },
+
         test_isEmpty: function() {
             Assert.isFalse(Y.all('input').isEmpty());
             Assert.isTrue(Y.all('.nomatch').isEmpty());