Approved.
Change 20080411-hqm-R by
<mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED] on 2008-04-11 02:51:15 EDT
in /Users/hqm/openlaszlo/trunk5
for
<http://svn.openlaszlo.org/openlaszlo/trunk>http://svn.openlaszlo.org/openlaszlo/trunk
Summary: fix for XML RPC, and some other brain damage in data loader
New Features:
Bugs Fixed: LPP-5481
Technical Reviewer: max
QA Reviewer: pbr
Doc Reviewer: (pending)
Documentation:
Release Notes:
Details:
+ A while back we discovered that that data loading API would not let
you POST data to a url while also supporting an independent query
string in the URL.
So I had made some fixes for that, both in SOLO and proxied
mode. In proxied mode I moved the place where the POST payload was
stored out of the "url" arg.
That caused some trouble because there is older code in
XMLRPCDataSource.java which expects to be able to stick
"lzpostbody=DATA" into the url and call the
HTTPDataSource.getHTTPData() method, which no longer expects to see
that data there.
I fixed this by putting back the code to look for the "lzpostbody"
arg inside the URL as well as in the proxy request parameters.
Sorry this is so baroque. The main idea going forward is that we
have the LzHTTPLoader API which looks like the javascript XMLHTTPRequest.
Everything in LzHTTPDataProvider is geared to talk to that API; a
URL with query args, and an optional POST content string.
The SWF kernel loader code has to then unpack that and make it work
with the brain-dead Flash LoadVars HTTP API as much as possible, in
solo and proxied mode.
Tests:
test/lfc/data/alldata.lzx, in dhtml and swf demos/amazon XMLRPC test
case in bug report
Files:
M test/data/data-url.lzx
A test/lfc/data/testgetmethod.lzx
A test/lfc/data/testgetmethodsolo.lzx
A test/lfc/data/testpostmethod.lzx
M test/lfc/data/alldata.lzx
A test/lfc/data/testpostmethodsolo.lzx
M WEB-INF/lps/lfc/kernel/swf/LzLoader.lzs
M WEB-INF/lps/lfc/kernel/swf/LzLoadQueue.as
M WEB-INF/lps/lfc/kernel/swf/LzHTTPLoader.as
M WEB-INF/lps/lfc/data/LzHTTPDataProvider.lzs
M WEB-INF/lps/lfc/data/LzDataset.lzs
M WEB-INF/lps/server/src/org/openlaszlo/data/HTTPDataSource.java
M demos/vacation-survey/vacation-survey.lzx
Changeset:
<http://svn.openlaszlo.org/openlaszlo/patches/20080411-hqm-R.tar>http://svn.openlaszlo.org/openlaszlo/patches/20080411-hqm-R.tar
