Ticket #731: georss.patch

tests/Layer/test_GeoRSS.html

@@ -28 +28 @@
-            t.eq( layer.name, "Crschmidt's Places At Platial", "Layer name is correct." );
-        } );
-    }
+    
+    function test_Layer_GeoRSS_dontUseFeedTitle (t) {
+        t.plan( 1 );
+        layer = new OpenLayers.Layer.GeoRSS('Test Layer', georss_txt, {'useFeedTitle': false} );
+        t.delay_call( 1, function() {  
+            t.eq( layer.name, "Test Layer", "Layer name is correct when not used from feed." );
+        } );
+    }
+    
-    function test_01_Layer_GeoRSS_AtomParsing (t) {
-        t.plan( 6 );
-        layer = new OpenLayers.Layer.GeoRSS('Test Layer', atom_xml );

lib/OpenLayers/Layer/GeoRSS.js

@@ -42 +42 @@
-     */
-    icon: null,
-    
+    /** 
+     * APIProperty: useFeedTitle 
+     * {Boolean} Set layer.name to the first <title> element in the feed. Default is true. 
+     */
+    useFeedTitle: true,
+    
-    /**
-    * Constructor: OpenLayers.Layer.GeoRSS
-    * Create a GeoRSS Layer.
@@ -80 +86 @@
-            doc = OpenLayers.parseXMLString(ajaxRequest.responseText);
-        }
-        
-
-
-
+
+
+
+
+
+
+
+
+
+
+
-        }
-        catch (e) {
-            this.name = doc.getElementsByTagName('title')[0].firstChild.nodeValue;
-        }
-       
-        /* Try RSS items first, then Atom entries */
-        var itemlist = null;