[Laszlo-checkins] r7902 - openlaszlo/branches/devildog/WEB-INF/lps/lfc/views

[dda@openlaszlo.org]

Author: dda
Date: 2008-01-27 14:49:01 -0800 (Sun, 27 Jan 2008)
New Revision: 7902

Modified:
   openlaszlo/branches/devildog/WEB-INF/lps/lfc/views/LaszloView.lzs
Log:
This fixes lzpix (and undoubtedly much more).  Approved by promanik and ptw via email:

On 2008-01-26, at 19:10 EST, Donald Anderson wrote:

Phil, Henry,

In core/LzNode.lzs, in devildog branch, this line is commented:

 //prototype.getProp = getAttribute;

Not so in trunk.  This appears to be part of r7519.
Could be that getProp is now gone (it was private, so
no doc issue) and that getAttribute could be used instead.

Indeed, I changed LaszloView.lzs, where the reference to
getProp is made, and got lzpix to work to DHTML again.

Henry, is this an appropriate fix?

Tucker:

Yes - let's make that change.  If getProp was private, I see no reason to keep it.

This line, and those below, will be ignored--

M    WEB-INF/lps/lfc/views/LaszloView.lzs


Modified: openlaszlo/branches/devildog/WEB-INF/lps/lfc/views/LaszloView.lzs
===================================================================
--- openlaszlo/branches/devildog/WEB-INF/lps/lfc/views/LaszloView.lzs	2008-01-27 22:36:43 UTC (rev 7901)
+++ openlaszlo/branches/devildog/WEB-INF/lps/lfc/views/LaszloView.lzs	2008-01-27 22:49:01 UTC (rev 7902)
@@ -2079,11 +2079,11 @@
 
     if (prop == "x" || prop =="y" ){
         tLink.update ( prop );
-        return tLink.offset[prop] + tLink.scale[prop] * this.getProp( prop );
+        return tLink.offset[prop] + tLink.scale[prop] * this.getAttribute( prop );
     }else if ( prop == "width" || prop == "height" ){
         var axis = prop == "width" ? "x" : "y"
         tLink.update ( axis )
-        return tLink.scale[axis] * this.getProp( prop );
+        return tLink.scale[axis] * this.getAttribute( prop );
 
     }else{
         //not yet implemented: rotation, alpha...