From 89ff80556bfc12532221dad7889f1a672ee83fdd Mon Sep 17 00:00:00 2001
From: Antonio Terceiro <terceiro@colivre.coop.br>
Date: Thu, 28 May 2009 16:05:30 -0300
Subject: [PATCH] ActionItem1063: removing external feed together with blog

---
 app/models/blog.rb     |  2 +-
 test/unit/blog_test.rb |  9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/app/models/blog.rb b/app/models/blog.rb
index b4c72d4..130103b 100644
--- a/app/models/blog.rb
+++ b/app/models/blog.rb
@@ -65,7 +65,7 @@ class Blog < Folder
     end
   end
 
-  has_one :external_feed, :foreign_key => 'blog_id'
+  has_one :external_feed, :foreign_key => 'blog_id', :dependent => :destroy
 
   attr_accessor :external_feed_data
   def external_feed_builder=(efeed)
diff --git a/test/unit/blog_test.rb b/test/unit/blog_test.rb
index 1fd1bcf..64cda89 100644
--- a/test/unit/blog_test.rb
+++ b/test/unit/blog_test.rb
@@ -136,4 +136,13 @@ class BlogTest < ActiveSupport::TestCase
     assert post.save!
   end
 
+  should 'remove external feed when removing blog' do
+    p = create_user('testuser').person
+    blog = Blog.create!(:name => 'Blog test', :profile => p, :external_feed_builder => {:enabled => true, :address => "http://bli.org/feed"})
+    assert blog.external_feed
+    assert_difference ExternalFeed, :count, -1 do
+      blog.destroy
+    end
+  end
+
 end
--
libgit2 0.21.2