[Laszlo-dev] fix for LzTrack in truck

Sarah Allen sallen at laszlosystems.com
Tue Nov 21 18:25:13 PST 2006


Good catch!  The change looks good.  Please check in the test file in 
/test/lps/lztrack/...  ideally with some text on the canvas that says 
what to look for.

Sarah

On Tue, Nov 21, 2006 at  6:08 PM, Adam Wolff wrote:

> Summary: Here's a quick fix for LzTrack in trunk. The old code 
> referred to view's
> "depth" property, which has been renamed to __LZdepth. The enclosed
> testcase shows the bug and is corrected by the fix.
>
> New Features:
>
> Bugs Fixed:
> (No bug number -- see enclosed testcase)
>
> Technical Reviewer: sarah allen(pending)
> QA Reviewer: (pending)
> Doc Reviewer: (pending)
>
> Documentation:
>
> Release Notes:
>
> Details:
> Code in LzTrack had rotted
>
> Tests:
> brokenTrack.lzx (not for checkin)
>
> Files:
> WEB-INF/lps/lfc/services/LzTrack.as
>
> svn diff services/LzTrack.as
> Index: services/LzTrack.as
> ===================================================================
> --- services/LzTrack.as (revision 2409)
> +++ services/LzTrack.as (working copy)
> @@ -127,7 +127,7 @@
>          btemp = btemp.immediateparent;
>      }
>      // a and b are siblings, check depth
> -    if (atemp.depth > btemp.depth) return a;
> +    if (atemp.__LZdepth > btemp.__LZdepth) return a;
>      else return b;
>  }



More information about the Laszlo-dev mailing list