Not sure -- I don't have enough knowledge to evaluate that without doing 
some additional poking around.  Since this part of the code was a 
copy-paste job, it is likely that this same problem (if it is one) 
occurs elsewhere.


On 2013-09-12 20:01, Bart van den Eijnden wrote:
>
> In lib/OpenLayers/Format/WCSDescribeCoverage/v1.js:
>
> > +     *
> > +     * Parameters:
> > +     * data - {DOMElement|String} A WCS DescribeCoverage document.
> > +     *
> > +     * Returns:
> > +     * {Object} An object representing the WCS DescribeCoverage response.
> > +     */
> > +    read: function(data) {
> > +        if(typeof data == "string") {
> > +            data = OpenLayers.Format.XML.prototype.read.apply(this, [data]);
> > +        }
> > +        if(data && data.nodeType == 9) {
> > +            data = data.documentElement;
> > +        }
> > +        var schema = {};
> > +        if (data.nodeName.split(":").pop() === 'ExceptionReport') {
>
> do we really need this? Should be handled by VersionedOGC base class 
> if errorProperty is set or not?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/openlayers/openlayers/pull/846/files#r6329330>.
>