[Laszlo-checkins] r11998 - openlaszlo/trunk/WEB-INF/lps/lfc/data

[bargull@openlaszlo.org]

Author: bargull
Date: 2008-12-06 13:46:24 -0800 (Sat, 06 Dec 2008)
New Revision: 11998

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/data/LzDataset.lzs
Log:
Change 20081206-bargull-aqE by bargull at dell--p4--2-53 on 2008-12-06 20:52:57
    in /home/Admin/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: LzDataset#errorstring not nulled

New Features:

Bugs Fixed: LPP-7410

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

Documentation:

Release Notes:

Details:
Reset "errorstring" after each response (and not before each request like in LPS3), this seems to be more consistent (e.g. see rawdata).
    

Tests:
see bugreport



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/data/LzDataset.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/data/LzDataset.lzs	2008-12-06 20:03:57 UTC (rev 11997)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/data/LzDataset.lzs	2008-12-06 21:46:24 UTC (rev 11998)
@@ -820,6 +820,9 @@
         if (dreq.queryparams) {
             // a "lzpostbody" arg overrides postbody, for back compatibility
             lzpostbody = dreq.queryparams.getValue('lzpostbody');
+            if (lzpostbody != null) {
+                dreq.queryparams.remove('lzpostbody');
+            }
         }
         // If there is a lzpostbody query arg, we are going to remove
         // it from the query args, and set it as the dreq.postbody
@@ -829,11 +832,8 @@
         // The Flash swf8 native API has no way to support POST of
         // arbitrary raw data in SOLO mode; the best we can do is to
         // send a list of url-encoded key=value pairs.
-        if (lzpostbody != null) { 
+        if (lzpostbody != null) {
             dreq.postbody = lzpostbody;
-            if (dreq.queryparams && dreq.queryparams.getValue('lzpostbody')) {
-                dreq.queryparams.remove('lzpostbody');
-            }
         }
     }
 
@@ -854,7 +854,7 @@
 
     // NB: You had better set the onstatus event handler *before* issuing request
     if (this.dsloadDel == null) {
-        this.dsloadDel = new LzDelegate(this , "handleDataResponse" , dreq, "onstatus");
+        this.dsloadDel = new LzDelegate(this, "handleDataResponse", dreq, "onstatus");
     } else {
         this.dsloadDel.register(dreq, "onstatus");
     }
@@ -872,6 +872,7 @@
     }
 
     this.rawdata = datareq.rawdata;
+    this.errorstring = null;
 
     if (datareq.status == LzDataRequest.SUCCESS) {
         if (this.responseheaders != null) {