You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xap-commits@incubator.apache.org by jm...@apache.org on 2007/09/19 01:55:46 UTC
svn commit: r577127 -
/incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js
Author: jmargaris
Date: Tue Sep 18 18:55:45 2007
New Revision: 577127
URL: http://svn.apache.org/viewvc?rev=577127&view=rev
Log:
be more careful of iterators
Modified:
incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js
Modified: incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js?rev=577127&r1=577126&r2=577127&view=diff
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js (original)
+++ incubator/xap/trunk/codebase/src/xap/bridges/xap/ListBoxBridge.js Tue Sep 18 18:55:45 2007
@@ -134,6 +134,9 @@
var elementListItem = childHandler.getElement();
var childPeer = childHandler.getPeer();
+ if (!childPeer){
+ return;
+ }
var peer = this.getPeer() ;
peer.addChild(childPeer);
}
@@ -153,6 +156,10 @@
}
xap.bridges.xap.ListBoxBridge.prototype.removeChild = function( childHandler ){
+ var childPeer = childHandler.getPeer();
+ if (!childPeer){
+ return;
+ }
var peer = this.getPeer() ;
peer.removeChild(childHandler.getPeer());
}