[Laszlo-checkins] r12026 - openlaszlo/trunk/WEB-INF/lps/lfc/core

[bargull@openlaszlo.org]

Author: bargull
Date: 2008-12-09 08:39:35 -0800 (Tue, 09 Dec 2008)
New Revision: 12026

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/core/LzNode.lzs
Log:
Change 20081208-bargull-Vta by bargull at dell--p4--2-53 on 2008-12-08 22:03:24
    in /home/Admin/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: add type check in releaseConstraint

New Features:

Bugs Fixed: LPP-7292

Technical Reviewer: ptw
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
"__LZdelegates" consists of LzDelegates _and_ LzDataAttrBinds, so a type check needs to be added.
    

Tests:
testcase from bugreport



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/core/LzNode.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/core/LzNode.lzs	2008-12-09 16:38:00 UTC (rev 12025)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/core/LzNode.lzs	2008-12-09 16:39:35 UTC (rev 12026)
@@ -1621,12 +1621,13 @@
    * implementation was nonesense.
    */
   function releaseConstraintMethod (constraintMethodName) {
-    var result = false;
-    var dels = this.__LZdelegates;
+    var result:Boolean = false;
+    var dels:Array = this.__LZdelegates;
     if (dels) {
-      for (var i = 0; i < dels.length; ) {
-        var del = dels[i];
-        if (del.c === this && del.m === this[constraintMethodName]) {
+      for (var i:int = 0; i < dels.length; ) {
+        var del:* = dels[i];
+        if (del is LzDelegate && del.c === this &&
+                del.m === this[constraintMethodName]) {
           del.unregisterAll();
           dels.splice(i, 1);
           result = true;