From dc77812f02b05bc2017f4a9cc8d54673ddeb6ae1 Mon Sep 17 00:00:00 2001 From: Isao Yagi Date: Fri, 22 Jun 2012 18:46:34 -0700 Subject: [PATCH] minor clarity tweaks --- source/lib/app/autoload/mojit-proxy.client.js | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/source/lib/app/autoload/mojit-proxy.client.js b/source/lib/app/autoload/mojit-proxy.client.js index 33d7b5ee4..c508bec74 100644 --- a/source/lib/app/autoload/mojit-proxy.client.js +++ b/source/lib/app/autoload/mojit-proxy.client.js @@ -296,7 +296,10 @@ YUI.add('mojito-mojit-proxy', function(Y, NAME) { * the dom. */ destroyChild: function(id, retainNode) { - var slot, doomed, children = this.getChildren(), child = children[id]; + var slot, + doomed, // viewid/dom id + children = this.getChildren(), + child = children[id]; if (child) { doomed = child.viewId; @@ -306,6 +309,7 @@ YUI.add('mojito-mojit-proxy', function(Y, NAME) { for (slot in children) { if (children.hasOwnProperty(slot) && children[slot].viewId === id) { doomed = id; + break; } } } @@ -332,10 +336,12 @@ YUI.add('mojito-mojit-proxy', function(Y, NAME) { * the dom. */ destroyChildren: function(retainNode) { - var my = this; - Y.Object.each(this.getChildren(), function(child, childId) { - my.destroyChild(childId, retainNode); - }); + var children = this.getChildren(), child; + for(child in children) { + if(children.hasOwnProperty(child)) { + this.destroyChild(child, retainNode); + } + } },