Henry Minsky | 2 Sep 2008 16:36

Re: For Review: Change 20080901-bargull-M9C Summary: dhtml http-loader

approved!

On Mon, Sep 1, 2008 at 5:30 PM, André Bargull <andre.bargull <at> udo.edu> wrote:
Change 20080901-bargull-M9C by bargull <at> dell--p4--2-53 on 2008-09-01 22:32:11
in /home/Admin/src/svn/openlaszlo/trunk
for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: dhtml http-loader

New Features:

Bugs Fixed: LPP-6912

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

Documentation:

Release Notes:

Details:
LzXMLParser:
- changed to a less verbose form
LzXMLTranslator:
- changed to a less verbose form
- removed slashPat, it wasn't referenced anywhere
- use now "localName" (resp. "baseName") instead of manually splitting the nsprefix
- removed debug-statements
LzHTTPLoader:
- moved all "static" properties to the beginning
- added local variable in translateXML (LPP-6912)
- fixed/added implementation of getResponseStatus(), getResponseHeaders(), getResponseHeader(), getElapsedTime()
- clear response-variables in "open" before doing a new request
- and some other trivial changes (e.g. renamed variables and moved functions for consistency)


Tests:
all-data passes
weather (uses nsprefix)

Files:
M WEB-INF/lps/lfc/kernel/dhtml/LzHTTPLoader.js
M WEB-INF/lps/lfc/kernel/dhtml/LzXMLTranslator.js
M WEB-INF/lps/lfc/kernel/dhtml/LzXMLParser.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20080901-bargull-M9C.tar





--
Henry Minsky
Software Architect
hminsky <at> laszlosystems.com



Gmane