Changeset 2256

Timestamp:

02/21/07 17:32:31 (2 years ago)

Author:

crschmidt

Message:

Add support for Atom 1.0 to Layer.GeoRSS. Patch from Sean Gillies provided
in #506: tests are included to test that the links are properly created
for Atom 1.0 content (with a data sample provided in the patch).
Thanks, Sean!

Files:

• trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js (modified) (1 diff)
• trunk/openlayers/tests/atom-1.0.xml (added)
• trunk/openlayers/tests/test_Layer_GeoRSS.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/Layer/GeoRSS.js

@@ -131 +131 @@
-            catch (e) { description="No description."; }
-
-
+
+
+
+
+
+
+
+
+
+
+
+
-            data.icon = OpenLayers.Marker.defaultIcon();
-            data.popupSize = new OpenLayers.Size(250, 120);

trunk/openlayers/tests/test_Layer_GeoRSS.html

@@ -19 +19 @@
-        } );
-    }
+    
+    function test_01_Layer_GeoRSS_AtomParsing (t) {
+        t.plan( 6 );
+        layer = new OpenLayers.Layer.GeoRSS('Test Layer', "./atom-1.0.xml" );
+        t.ok( layer instanceof OpenLayers.Layer.GeoRSS, "new OpenLayers.Layer.GeoRSS returns object" );
+        t.eq( layer.location, "./atom-1.0.xml", "layer.location is correct" );
+        var markers;
+        t.delay_call( 1, function() {  
+            t.eq( layer.markers.length, 2, "marker length is correct" );
+            var ll = new OpenLayers.LonLat(29.9805, 36.7702);
+            t.ok( layer.markers[0].lonlat.equals(ll), "lonlat on first marker is correct" );
+            t.like( layer.features[0].data['popupContentHTML'], '<a class="link" href="http://pleiades.stoa.org/places/638896" target="_blank">Unnamed Tumulus</a>', "Link is correct.");
+            t.eq( layer.name, "tumulus", "Layer name is correct." );
+        } );
+    }
+
-    function test_02_Layer_GeoRSS_draw (t) { 
-//        t.plan(5);