[Laszlo-dev] For Review: Change 20081130-ptw-v Summary: Revert init arg behavior

P T Withington ptw at pobox.com
Mon Dec 1 08:55:29 PST 2008


Change 20081130-ptw-v by ptw at dueling-banjos.home on 2008-11-30  
12:01:16 EST
     in /Users/ptw/OpenLaszlo/trunk
     for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Revert init arg behavior

Bugs Fixed:
Lpp-7378 LzTextSprite.setPattern warning in Laszlo 4.1.1, not in 4.0.12
LPP-7386 Proposal to revert behavior of LzNode.construct to 4.1 behavior

Technical Reviewer: a.bargull at intensis.de (pending)
QA Reviewer: max (pending)

Details:
     Added a new internal API LzNode/__LZhasConstraint, which can be
     used in LzNode/construct to determine if an attribute will be
     constrained by LzNode/__LZapplyArgs.  This replaces testing for
     LzInitExpr's in construct's arglist.  LzNode/construct' initargs
     are now as they were in 4.1 -- only constant initial values appear
     in that list.  Currently only LzText (and subclasses) use the
     __LZhasConstraint API, because they (apparently) need to know of
     their width/height will be constrained at construct time.  No
     other LFC classes or component classes use this facility, so I
     have kept is as an LFC private API.

     LzTextSprite, LzInputTextSprite: These two sprite classes should
     not be using the __LZhasConstraint API, but they are probably
     going to be obsolete soon, so I just fixed them in the most
     expedient way.

     LzNode: Remove LzInitExprs from init args when calling construct,
     while preserving the ability of construct replacing/adding init
     args.

     LzText:  Use new __LZhasConstraint API.

     LzText, LzInputText, LaszloView: Simplify construct method, now
     that args only contains constant values.

     LzReplicationManager: Clarifying comment

     hintedlayout: Fix construct arg defaulting bug noted in passing
     (similar to LPP-7376)

Tests:
     smokecheck, ant test, test case from LPP-7386

Files:
M      WEB-INF/lps/lfc/kernel/swf/LzTextSprite.as
M      WEB-INF/lps/lfc/kernel/swf/LzInputTextSprite.as
M      WEB-INF/lps/lfc/kernel/swf9/LzTextSprite.as
M      WEB-INF/lps/lfc/core/LzNode.lzs
M      WEB-INF/lps/lfc/views/LzInputText.lzs
M      WEB-INF/lps/lfc/views/LzText.lzs
M      WEB-INF/lps/lfc/views/LaszloView.lzs
M      WEB-INF/lps/lfc/data/LzReplicationManager.lzs
M      demos/lzpixmobile/lib/hintedlayout.lzx

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20081130-ptw-v.tar


More information about the Laszlo-dev mailing list