Merge pull request #2888 from fgravin/boundedBy
Element boundedBy must not be set as geometry field on GML reading
This commit is contained in:
@@ -159,6 +159,7 @@ ol.format.GMLBase.prototype.readFeature_ = function(node, objectStack) {
|
|||||||
var values = {}, geometryName;
|
var values = {}, geometryName;
|
||||||
for (n = node.firstElementChild; !goog.isNull(n);
|
for (n = node.firstElementChild; !goog.isNull(n);
|
||||||
n = n.nextElementSibling) {
|
n = n.nextElementSibling) {
|
||||||
|
var localName = ol.xml.getLocalName(n);
|
||||||
// Assume attribute elements have one child node and that the child
|
// Assume attribute elements have one child node and that the child
|
||||||
// is a text node. Otherwise assume it is a geometry node.
|
// is a text node. Otherwise assume it is a geometry node.
|
||||||
if (n.childNodes.length === 0 ||
|
if (n.childNodes.length === 0 ||
|
||||||
@@ -168,10 +169,13 @@ ol.format.GMLBase.prototype.readFeature_ = function(node, objectStack) {
|
|||||||
if (goog.string.isEmpty(value)) {
|
if (goog.string.isEmpty(value)) {
|
||||||
value = undefined;
|
value = undefined;
|
||||||
}
|
}
|
||||||
values[ol.xml.getLocalName(n)] = value;
|
values[localName] = value;
|
||||||
} else {
|
} else {
|
||||||
geometryName = ol.xml.getLocalName(n);
|
// boundedBy is an extent and must not be considered as a geometry
|
||||||
values[geometryName] = this.readGeometryElement(n, objectStack);
|
if (localName !== 'boundedBy') {
|
||||||
|
geometryName = localName;
|
||||||
|
}
|
||||||
|
values[localName] = this.readGeometryElement(n, objectStack);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
var feature = new ol.Feature(values);
|
var feature = new ol.Feature(values);
|
||||||
|
|||||||
Reference in New Issue
Block a user