[Laszlo-checkins] r10886 - openlaszlo/trunk/WEB-INF/lps/lfc/helpers

[ptw@openlaszlo.org]

Author: ptw
Date: 2008-09-05 02:16:38 -0700 (Fri, 05 Sep 2008)
New Revision: 10886

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/helpers/LzState.lzs
Log:
Change 20080905-ptw-p by ptw at dueling-banjos.home on 2008-09-05 03:42:21 EDT
    in /Users/ptw/OpenLaszlo/krispy-kreme
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Add missing state event

Bugs Fixed:
LPP-6663: 'onapplied' Event For State Not Firing

Technical Reviewer: a.bargull at intensis.de (Message-ID: <48C0EE9A.5090904 at udo.edu>)
QA Reviewer: jcrowley (Message-ID: <48C0F478.2070103 at laszlosystems.com>)

Details:
    Add missing event

Tests:
    Test case from bug report



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/helpers/LzState.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/helpers/LzState.lzs	2008-09-05 08:30:09 UTC (rev 10885)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/helpers/LzState.lzs	2008-09-05 09:16:38 UTC (rev 10886)
@@ -110,8 +110,9 @@
   static var attributes = new LzInheritedHash(LzNode.attributes);
 
   /**
-   * Sent after a state is applied.  The event argument is the state
-   * that is being applied.
+   * Backward compatible version of `onapplied`
+   * @deprecated in 4.2
+   * @access private
    * @lzxtype event
    */
   var onapply:LzDeclaredEventClass = LzDeclaredEvent;
@@ -183,8 +184,11 @@
 
   /** @access private */
   static var props = { apply : true }; // unused?
-  /** @access private */
-  static var events = { onremove : true , onapply : true };
+  /**
+   * Remove onapply after 4.2
+   * @access private
+   */
+  static var events = { onremove : true , onapply : true, onapplied : true };
   prototype.$isstate = true; // Defined in LzNode
   /** @access private */
   var asyncnew = false;
@@ -332,6 +336,7 @@
 
     parent.__LZdelegates = od;
 
+    // Remove onapply after 4.2
     if (this.onapply.ready) this.onapply.sendEvent( this );
     if (this.onapplied.ready) this.onapplied.sendEvent( true );
   }