Fix parsing tags

Due to changes in 5669cb7, this is now correctly being parsed as XML, not HTML. As such, attribute names are case-sensitive, so it is htmlUrl and xmlUrl, not htmlurl and xmlurl.
horde · Apr 17, 2014 · 9044a6e · 9044a6e
1 parent 201114d
commit 9044a6e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/framework/Feed/lib/Horde/Feed/Blogroll.php b/framework/Feed/lib/Horde/Feed/Blogroll.php
@@ -48,7 +48,7 @@ protected function _buildListItemCache()
     {
         $entries = array();
         foreach ($this->_element->getElementsByTagName('outline') as $child) {
-            if ($child->attributes->getNamedItem('xmlurl')) {
+            if ($child->attributes->getNamedItem('xmlUrl')) {
                 $entries[] = $child;
             }
         }

diff --git a/framework/Feed/test/Horde/Feed/BlogrollTest.php b/framework/Feed/test/Horde/Feed/BlogrollTest.php
@@ -27,13 +27,13 @@ public function testValidBlogrolls($file)
 
         $this->assertInstanceOf('Horde_Feed_Entry_Blogroll', $entry);
         $this->assertGreaterThan(0, strlen($entry->text));
-        $this->assertGreaterThan(0, strlen($entry->xmlurl));
+        $this->assertGreaterThan(0, strlen($entry->xmlUrl));
 
         $this->assertEquals($entry->text, $entry['text']);
         $this->assertEquals($entry->description, $entry['description']);
         $this->assertEquals($entry->title, $entry['title']);
-        $this->assertEquals($entry->htmlurl, $entry['htmlurl']);
-        $this->assertEquals($entry->xmlurl, $entry['xmlurl']);
+        $this->assertEquals($entry->htmlUrl, $entry['htmlUrl']);
+        $this->assertEquals($entry->xmlUrl, $entry['xmlUrl']);
     }
 
     public function testGroupedBlogrolls()