Ok. We'll see if Adam can shed any light. I'd like to make it so
initstage is not clobbered in either case, if that is a safe change.
On 2007-09-10, at 14:15 EDT, Elliot Winard wrote:
When working with APIs and software systems, I want to things
should work. I don't think I'm being overly pedantic.
This is a gray area that can be (and was) worked around, but just
made me say "huh?" when I saw what was happening.
I haven't been blocked on OL development for the last two years
because of this. :P
-e
P T Withington wrote:
I'm concerned that:
a) There will now be a complaint that initstage=late changes to
early, or,
b) We are missing something as to why the initstage is changed in
completeInstantiation
What I'd like to know from the original reporter (cc-ed) is if
there is a problem with the initstage being changed, or if he is
just being pedantic.
I will ask Adam if he can elaborate on why he changed the
initstage (and cc you).
On 2007-09-10, at 12:38 EDT, Philip Romanik wrote:
Change 20070910-Philip-6 by [EMAIL PROTECTED] on 2007-09-10
12:06:18 EDT
in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/wafflecone
for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone
Summary: Prevent initstage=defer from changing to "early"
New Features:
Bugs Fixed: LPP-712
Technical Reviewer: ptw
QA Reviewer: (pending)
Doc Reviewer: (pending)
Documentation:
Release Notes:
Details:
initstage is changed to "early" in completeInstantiation().
However, I don't see why this change is necessary. Objects are
instantiated without problem if you leave initstage=defer. I did
not touch the other case where initstage is set to "early".
Tests:
Run spike-initstage.lzx test that is listed in LPP-712. This is
the app, with slightly enhanced debugging output. When you click
on the "finish" button, the last window will be inited.
<canvas>
<debug y="200" />
<class name="mywindow" extends="window" width="100"
height="100" >
<method event="oninit">
Debug.write("oninit", this.initstage, this, this.title);
if (global['deferedWindow']) Debug.write
("deferedWindow.initstage",
deferedWindow.initstage, deferedWindow.isinited);
</method>
</class>
<mywindow title="immediate" initstage="immediate" x="0" />
<mywindow title="early" initstage="early" x="100"/>
<mywindow title="normal" initstage="normal" x="200">
<button text="finish"
onclick="deferedWindow.completeInstantiation()" />
</mywindow>
<mywindow title="late" initstage="late" x="300"/>
<mywindow id="deferedWindow" title="defer" initstage="defer"
x="400"/>
<simplelayout/>
</canvas>
Files:
M WEB-INF/lps/lfc/core/LzNode.lzs
Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070910-
Philip-6.tar
