[Laszlo-checkins] r10067 - openlaszlo/trunk/WEB-INF/lps/lfc/views

[pbr@openlaszlo.org]

Author: pbr
Date: 2008-06-27 10:34:49 -0700 (Fri, 27 Jun 2008)
New Revision: 10067

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/views/LzText.lzs
Log:
Change 20080626-Philip-0 by Philip at Philip-DC on 2008-06-26 17:28:08 EDT
     in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/trunk
     for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix LzText.getText() for inputtext

New Features:

Bugs Fixed: LPP-6518

Technical Reviewer: max
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
This problem was caused by the fix for LPP-5678. In LPP-5678,
LzInputText.setText() was removed. However, the code in
LzText.setText() looked at this.text and not this.getText() to see if the text has changed. I fixed this and now the test passes.

I created LPP-6547 for the next release to fix (this.text !=
this.getText()) for input text.

Tests:
  Amazon app in dhtml and swf
    - Add a new address (checkout tab)
    - Add some text to the boxes and hit cancel
    - Click add new address and the previous values should be gone.

Files:
M      WEB-INF/lps/lfc/views/LzText.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080626-Philip-0.tar


Modified: openlaszlo/trunk/WEB-INF/lps/lfc/views/LzText.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/views/LzText.lzs	2008-06-27 17:29:18 UTC (rev 10066)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/views/LzText.lzs	2008-06-27 17:34:49 UTC (rev 10067)
@@ -787,7 +787,7 @@
 function setText ( t, force = null){
     // force to a string
     t = '' + t;
-    if (force != true && t == this.text) return;
+    if (force != true && t == this.getText()) return;
     var tsprite:LzTextSprite = (this.sprite cast LzTextSprite);
     if (this.visible) tsprite.setVisible(this.visible);
     if (this.maxlength != null && t.length > this.maxlength){