diff --git a/src/ol/format/wmsgetfeatureinfoformat.js b/src/ol/format/wmsgetfeatureinfoformat.js
index 916ae7790c..6f3178339f 100644
--- a/src/ol/format/wmsgetfeatureinfoformat.js
+++ b/src/ol/format/wmsgetfeatureinfoformat.js
@@ -8,7 +8,6 @@ goog.require('goog.object');
goog.require('goog.string');
goog.require('ol.format.GML');
goog.require('ol.format.GML2');
-goog.require('ol.format.GMLBase');
goog.require('ol.format.XMLFeature');
goog.require('ol.xml');
@@ -16,16 +15,14 @@ goog.require('ol.xml');
/**
* @classdesc
- * Format for reading GetFeatureInfo format.It uses
- * ol.format.GML2 to read features.
+ * Format for reading WMSGetFeatureInfo format. It uses
+ * {@link ol.format.GML2} to read features.
*
* @constructor
- * @param {olx.format.GMLOptions=} opt_options
- * Optional configuration object.
* @extends {ol.format.XMLFeature}
* @api
*/
-ol.format.GetFeatureInfo = function(opt_options) {
+ol.format.WMSGetFeatureInfo = function() {
/**
* @private
@@ -36,27 +33,13 @@ ol.format.GetFeatureInfo = function(opt_options) {
/**
* @private
- * @type {string}
- */
- this.featureIdentifier_ = '_feature';
-
-
- /**
- * @private
- * @type {string}
- */
- this.layerIdentifier_ = '_layer';
-
-
- /**
- * @private
- * @type {ol.format.GMLBase}
+ * @type {ol.format.GML2}
*/
this.gmlFormat_ = new ol.format.GML2();
goog.base(this);
};
-goog.inherits(ol.format.GetFeatureInfo, ol.format.XMLFeature);
+goog.inherits(ol.format.WMSGetFeatureInfo, ol.format.XMLFeature);
/**
@@ -64,7 +47,15 @@ goog.inherits(ol.format.GetFeatureInfo, ol.format.XMLFeature);
* @type {string}
* @private
*/
-ol.format.GetFeatureInfo.schemaLocation_ = '';
+ol.format.WMSGetFeatureInfo.featureIdentifier_ = '_feature';
+
+
+/**
+ * @const
+ * @type {string}
+ * @private
+ */
+ol.format.WMSGetFeatureInfo.layerIdentifier_ = '_layer';
/**
@@ -73,11 +64,13 @@ ol.format.GetFeatureInfo.schemaLocation_ = '';
* @return {Array.
} Features.
* @private
*/
-ol.format.GetFeatureInfo.prototype.readFeatures_ = function(node, objectStack) {
+ol.format.WMSGetFeatureInfo.prototype.readFeatures_ =
+ function(node, objectStack) {
node.namespaceURI = this.featureNS_;
goog.asserts.assert(node.nodeType == goog.dom.NodeType.ELEMENT);
var localName = ol.xml.getLocalName(node);
+ /** @type {Array.} */
var features = [];
if (node.childNodes.length === 0) {
return features;
@@ -90,10 +83,12 @@ ol.format.GetFeatureInfo.prototype.readFeatures_ = function(node, objectStack) {
var context = objectStack[0];
goog.asserts.assert(goog.isObject(context));
- goog.asserts.assert(layer.localName.indexOf(this.layerIdentifier_) >= 0);
+ goog.asserts.assert(layer.localName.indexOf(
+ ol.format.WMSGetFeatureInfo.layerIdentifier_) >= 0);
var featureType = goog.string.remove(layer.localName,
- this.layerIdentifier_) + this.featureIdentifier_;
+ ol.format.WMSGetFeatureInfo.layerIdentifier_) +
+ ol.format.WMSGetFeatureInfo.featureIdentifier_;
goog.object.set(context, 'featureType', featureType);
goog.object.set(context, 'featureNS', this.featureNS_);
@@ -107,17 +102,17 @@ ol.format.GetFeatureInfo.prototype.readFeatures_ = function(node, objectStack) {
var layerFeatures = ol.xml.pushParseAndPop(
[], parsersNS, layer, objectStack, this.gmlFormat_);
if (goog.isDef(layerFeatures)) {
- goog.array.extend(/** @type {Array} */ (features), layerFeatures);
+ goog.array.extend(features, layerFeatures);
}
}, this);
}
if (localName == 'FeatureCollection') {
- features = ol.xml.pushParseAndPop([],
+ var gmlFeatures = ol.xml.pushParseAndPop([],
this.gmlFormat_.FEATURE_COLLECTION_PARSERS, node,
[{}], this.gmlFormat_);
- }
- if (!goog.isDef(features)) {
- features = [];
+ if (goog.isDef(gmlFeatures)) {
+ features = gmlFeatures;
+ }
}
return features;
};
@@ -126,7 +121,7 @@ ol.format.GetFeatureInfo.prototype.readFeatures_ = function(node, objectStack) {
/**
* @inheritDoc
*/
-ol.format.GetFeatureInfo.prototype.readFeaturesFromNode =
+ol.format.WMSGetFeatureInfo.prototype.readFeaturesFromNode =
function(node, opt_options) {
var options = {
'featureType': this.featureType,
diff --git a/test/spec/ol/format/wmsgetfeatureinfoformat.test.js b/test/spec/ol/format/wmsgetfeatureinfoformat.test.js
index d402d37161..f4ec041a59 100644
--- a/test/spec/ol/format/wmsgetfeatureinfoformat.test.js
+++ b/test/spec/ol/format/wmsgetfeatureinfoformat.test.js
@@ -12,7 +12,7 @@ describe('ol.format.WMSGetFeatureInfo', function() {
proj4.defs('urn:x-ogc:def:crs:EPSG:4326', proj4.defs('EPSG:4326'));
afterLoadText('spec/ol/format/wms/getfeatureinfo.xml', function(data) {
try {
- features = new ol.format.GetFeatureInfo().readFeatures(data);
+ features = new ol.format.WMSGetFeatureInfo().readFeatures(data);
} catch (e) {
done(e);
}
@@ -47,7 +47,7 @@ describe('ol.format.WMSGetFeatureInfo', function() {
' ' +
' ' +
'';
- var features = new ol.format.GetFeatureInfo().readFeatures(text);
+ var features = new ol.format.WMSGetFeatureInfo().readFeatures(text);
expect(features.length).to.be(0);
});
@@ -72,7 +72,7 @@ describe('ol.format.WMSGetFeatureInfo', function() {
' ' +
' ' +
'';
- var features = new ol.format.GetFeatureInfo().readFeatures(text);
+ var features = new ol.format.WMSGetFeatureInfo().readFeatures(text);
expect(features.length).to.be(1);
expect(features[0].get('FOO')).to.be('bar');
// FIXME is that really wanted ?
@@ -132,7 +132,7 @@ describe('ol.format.WMSGetFeatureInfo', function() {
' ' +
' ' +
'';
- var features = new ol.format.GetFeatureInfo().readFeatures(text);
+ var features = new ol.format.WMSGetFeatureInfo().readFeatures(text);
expect(features.length).to.be(2);
expect(features[0].get('OBJECTID')).to.be('287');
expect(features[1].get('OBJECTID')).to.be('1251');
@@ -188,7 +188,7 @@ describe('ol.format.WMSGetFeatureInfo', function() {
' ' +
' ' +
'';
- var features = new ol.format.GetFeatureInfo().readFeatures(text);
+ var features = new ol.format.WMSGetFeatureInfo().readFeatures(text);
expect(features.length).to.be(1);
expect(features[0].get('cat')).to.be('3');
expect(features[0].getGeometry().getType()).to.be('MultiLineString');