[Laszlo-dev] UPDATE: For Review: Change 20081008-bargull-xEZ Summary: selection state did not update properly

[André Bargull]

Change 20081008-bargull-xEZ by bargull at dell--p4--2-53 on 2008-10-08 11:48:04
in /home/Admin/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: selection state did not update properly

New Features:

Bugs Fixed: LPP-2339

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

Documentation:

Release Notes:

Details:
The current selection-logic did not account a selection-change if the 
data-element didn't change,
which might happen for lazy replicated elements. Therefore I've moved 
the "setSelected()"-call before
the "if(pc)"-block.
Additionally moved setter for "datacontrolsvisibility" to property 
declaration

UPDATE:
Tucker asked me to take a more thorough look at clone-pool, especially 
when a clone gets recycled back to the pool.
This revealed the original cause for the selection-bug: when the clone 
was recycled, the clone's datapath wasn't reset to null.


Tests:
see bugreport

Files:
M WEB-INF/lps/lfc/data/LzDatapath.lzs
M WEB-INF/lps/lfc/data/LzReplicationManager.lzs

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20081008-bargull-xEZ.tar