[Laszlo-checkins] r5720 - openlaszlo/branches/legals/WEB-INF/lps/lfc/controllers

[pbr@openlaszlo.org]

Author: pbr
Date: 2007-07-19 15:59:43 -0700 (Thu, 19 Jul 2007)
New Revision: 5720

Modified:
   openlaszlo/branches/legals/WEB-INF/lps/lfc/controllers/LzAnimatorGroup.lzs
Log:
Change 20070619-Philip-2 by Philip at Philip-DC on 2007-06-19 09:21:16 EST
   in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/legals
   for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: Animatorgroup doesn't initialize animators correctly

New Features:

Bugs Fixed: LPP-2700

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

Documentation:

Release Notes:

Details:
LzAnimatorGroup: Animators controlled by a group are not being enabled before ru
n. This causes some variables not to become initialized. This isn't a problem if
 you don't change the values once animation starts. The solution is to set isact
ive=true when the group starts running.

lztest-animatorgroup.lzx. There were a few copy/paste errors that I fixed.


Tests:
Test case in LPP-2700 (change the window size during animation. The final value
is now correct)

/test/lztest/lztest-animator.lzx
/test/lztest/lztest-animatorgroup.lzx
 (Any unexpected errors you see are from small errors in measuring time, or fin
al values.)

Files:
M      test/lztest/lztest-animatorgroup.lzx
M      WEB-INF/lps/lfc/controllers/LzAnimatorGroup.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070619-Philip-2.tar


Modified: openlaszlo/branches/legals/WEB-INF/lps/lfc/controllers/LzAnimatorGroup.lzs
===================================================================
--- openlaszlo/branches/legals/WEB-INF/lps/lfc/controllers/LzAnimatorGroup.lzs	2007-07-19 22:23:27 UTC (rev 5719)
+++ openlaszlo/branches/legals/WEB-INF/lps/lfc/controllers/LzAnimatorGroup.lzs	2007-07-19 22:59:43 UTC (rev 5720)
@@ -366,6 +366,7 @@
         var a = this.actAnim[i];
         //if ( this.process == "simultaneous" ) Debug.write( "call " + a );
         if (a.notstarted) {
+            a.isactive = true;
             a.prepareStart();
             a.notstarted = false;
         } else if (a.needsrestart) {