patch for AtomFeedBurnerEntry to fall back on rel=alternate

eladmeidar · Feb 18, 2009 · 5f12e3d · 5f12e3d
1 parent 4074877
commit 5f12e3d
Show file tree

Hide file tree

Showing 3 changed files with 180 additions and 0 deletions.
diff --git a/lib/feedzirra/atom_feed_burner_entry.rb b/lib/feedzirra/atom_feed_burner_entry.rb
@@ -4,6 +4,7 @@ class AtomFeedBurnerEntry
     include FeedEntryUtilities
     element :title
     element :name, :as => :author
+    element :link, :as => :url, :value => :href, :with => {:type => "text/html", :rel => "alternate"}
     element :"feedburner:origLink", :as => :url
     element :summary
     element :content

diff --git a/spec/feedzirra/atom_feed_burner_entry_spec.rb b/spec/feedzirra/atom_feed_burner_entry_spec.rb
@@ -11,6 +11,11 @@
     @entry.title.should == "Making a Ruby C library even faster"
   end
 
+  it "should be able to fetch a url via the 'alternate' rel if no origLink exists" do
+    entry = Feedzirra::AtomFeedBurner.parse(File.read("#{File.dirname(__FILE__)}/../sample_feeds/PaulDixExplainsNothingAlternate.xml")).entries.first
+    entry.url.should == 'http://feeds.feedburner.com/~r/PaulDixExplainsNothing/~3/519925023/making-a-ruby-c-library-even-faster.html'
+  end
+
   it "should parse the url" do
     @entry.url.should == "http://www.pauldix.net/2009/01/making-a-ruby-c-library-even-faster.html"
   end