I still don't think this is the right approach, because a $path 
constraint may hide a view even if the view's datapath is still valid. 
Or consider the case when multiple $path constraints are used by a view, 
with the current patch, a single $path constraint may hide/show the view 
even if other $path constraints are evaluated to non-null resp. null. 
This may lead to indeterministic behaviour since constraints are not 
applied in a well determined order.


On 10/7/2009 10:32 AM, Max Carlson wrote:
> Change 20091007-maxcarlson-O by [email protected] on 2009-10-07 
> 01:22:48 PDT
>     in /Users/maxcarlson/openlaszlo/trunk-clean
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: Prevent $path expressions from hiding views
> 
> Bugs Fixed: LPP-290 - $path attributes can hide the instance/view they 
> belong to, LPP-8229 - No text displayed for check box
> 
> Technical Reviewer: [email protected]
> QA Reviewer: hminsky
> 
> Details: Follow the recipe in LzDatapath.__LZApplyData() to update 
> $path-constrained view's visibility.
> 
> Tests: Testcases from LPP-290, LPP-8229 and LPP-8529 all pass.
> 
> Files:
> M       WEB-INF/lps/lfc/data/LzDataAttrBind.lzs
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20091007-maxcarlson-O.tar
> 
_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to