You may want to create two if-statements here, so the block won't be 
included in the non-debug LFC. Otherwise approved!

> +            if ($debug && xmlDom == null) {
> +                Debug.error('Could not instantiate a XML DOM ActiveXObject');
> +            }



On 9/1/2009 8:21 PM, Henry Minsky wrote:
> Change 20090901-hqm-D by [email protected] on 2009-09-01 14:19:37 EDT
>     in /Users/hqm/openlaszlo/trunk-clean
>     for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: use newer XML DOM object for ActiveX
> 
> New Features:
> 
> Bugs Fixed: LPP-8420
> 
> Technical Reviewer: andre
> QA Reviewer: max
> Doc Reviewer: (pending)
> 
> Documentation:
> 
> Release Notes:
> 
> Details:
> 
> + Use the recommended version of XML DOM object for ActiveX
> 
> Tests:
> 
> test/lfc/data/alldata.lzx?lzr=dhtml
> 
> lzpix/app.lzx?lzr=dhtml
> 
> 
> Files:
> M      WEB-INF/lps/lfc/kernel/dhtml/LzXMLParser.js
> 
> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090901-hqm-D.tar
> 
_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to