Changeset 3096

Timestamp:

04/26/07 09:31:46 (2 years ago)

Author:

crschmidt

Message:

Add GML format test, which for the time being only has a constructor and
getFID tests, to test for broken fid parser, patch provided by Andreas
Hocevar in #679 .

Files:

• trunk/openlayers/lib/OpenLayers/Format/GML.js (modified) (6 diffs)
• trunk/openlayers/tests/Format/test_GML.html (added)
• trunk/openlayers/tests/list-tests.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Format/GML.js

@@ -85 +85 @@
-        var feature = new OpenLayers.Feature.Vector();
-
-        if (xmlNode.firstChild.attributes && xmlNode.firstChild.attributes['fid']) {
-            feature.fid = xmlNode.firstChild.attributes['fid'].nodeValue;
-        }
-        
-        // match MultiPolygon
-        if (OpenLayers.Ajax.getElementsByTagNameNS(xmlNode, this.gmlns, "gml", "MultiPolygon").length != 0) {
-            var multipolygon = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode, this.gmlns, "gml", "MultiPolygon")[0];
+            feature.fid = multipolygon.parentNode.parentNode.getAttribute('fid');
+
-            geom = new OpenLayers.Geometry.MultiPolygon();
-            var polygons = OpenLayers.Ajax.getElementsByTagNameNS(multipolygon,
@@ -105 +103 @@
-            var multilinestring = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode,
-            this.gmlns, "gml", "MultiLineString")[0];
+            feature.fid = multilinestring.parentNode.parentNode.getAttribute('fid');
-            
-            geom = new OpenLayers.Geometry.MultiLineString();
@@ -123 +122 @@
-            var multiPoint = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode,
-                this.gmlns, "gml", "MultiPoint")[0];
+            feature.fid = multiPoint.parentNode.parentNode.getAttribute('fid');
-                
-            geom = new OpenLayers.Geometry.MultiPoint();
@@ -139 +139 @@
-            var polygon = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode,
-                this.gmlns, "gml", "Polygon")[0];
+            feature.fid = polygon.parentNode.parentNode.getAttribute('fid');
-            
-            geom = this.parsePolygonNode(polygon);
@@ -147 +148 @@
-            var lineString = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode,
-                this.gmlns, "gml", "LineString")[0];
+            feature.fid = lineString.parentNode.parentNode.getAttribute('fid');
+
-            p = this.parseCoords(lineString);
-            if (p.points) {
@@ -158 +161 @@
-            var point = OpenLayers.Ajax.getElementsByTagNameNS(xmlNode,
-                this.gmlns, "gml", "Point")[0];
+            feature.fid = point.parentNode.parentNode.getAttribute('fid');
-            
-            p = this.parseCoords(point);

trunk/openlayers/tests/list-tests.html

@@ -18 +18 @@
-    <li>Geometry/test_Surface.html</li>
-    <li>test_Format.html</li>
+    <li>Format/test_GML.html</li>
-    <li>Format/test_WKT.html</li>
-    <li>test_Icon.html</li>