tUpdating path before save

Rodrigo Souto
1 parent f1bd1bbc
Showing 2 changed files with 24 additions and 0 deletions Show diff stats
app/models/article.rb
test/unit/article_test.rb
@@ -154,6 +154,12 @@ class Article &lt; ActiveRecord::Base
     article.advertise = true
   end
+  before_save do |article|
+    article.parent = article.parent_id ? Article.find(article.parent_id) : nil
+    parent_path = article.parent ? article.parent.path : nil
+    article.path = [parent_path, article.slug].compact.join('/')
+  end
+
   # retrieves all articles belonging to the given +profile+ that are not
   # sub-articles of any other article.
   named_scope :top_level_for, lambda { |profile|
@@ -1782,4 +1782,22 @@ class ArticleTest &lt; ActiveSupport::TestCase
     assert_equal license, article.license
   end
+  should 'update path if parent is changed' do
+    f1 = Folder.create!(:name => 'Folder 1', :profile => profile)
+    f2 = Folder.create!(:name => 'Folder 2', :profile => profile)
+    article = TinyMceArticle.create!(:name => 'Sample Article', :parent_id => f1.id, :profile => profile)
+    assert_equal [f1.path,article.slug].join('/'), article.path
+
+    article.parent = f2
+    article.save!
+    assert_equal [f2.path,article.slug].join('/'), article.path
+
+    article.parent = nil
+    article.save!
+    assert_equal article.slug, article.path
+
+    article.update_attributes({:parent_id => f2.id})
+    assert_equal [f2.path,article.slug].join('/'), article.path
+  end
+
 end
	@@ -154,6 +154,12 @@ class Article < ActiveRecord::Base		@@ -154,6 +154,12 @@ class Article < ActiveRecord::Base
154	article.advertise = true	154	article.advertise = true
155	end	155	end
156		156
		157	+ before_save do \|article\|
		158	+ article.parent = article.parent_id ? Article.find(article.parent_id) : nil
		159	+ parent_path = article.parent ? article.parent.path : nil
		160	+ article.path = [parent_path, article.slug].compact.join('/')
		161	+ end
		162	+
157	# retrieves all articles belonging to the given +profile+ that are not	163	# retrieves all articles belonging to the given +profile+ that are not
158	# sub-articles of any other article.	164	# sub-articles of any other article.
159	named_scope :top_level_for, lambda { \|profile\|	165	named_scope :top_level_for, lambda { \|profile\|
	@@ -1782,4 +1782,22 @@ class ArticleTest < ActiveSupport::TestCase		@@ -1782,4 +1782,22 @@ class ArticleTest < ActiveSupport::TestCase
1782	assert_equal license, article.license	1782	assert_equal license, article.license
1783	end	1783	end
1784		1784
		1785	+ should 'update path if parent is changed' do
		1786	+ f1 = Folder.create!(:name => 'Folder 1', :profile => profile)
		1787	+ f2 = Folder.create!(:name => 'Folder 2', :profile => profile)
		1788	+ article = TinyMceArticle.create!(:name => 'Sample Article', :parent_id => f1.id, :profile => profile)
		1789	+ assert_equal [f1.path,article.slug].join('/'), article.path
		1790	+
		1791	+ article.parent = f2
		1792	+ article.save!
		1793	+ assert_equal [f2.path,article.slug].join('/'), article.path
		1794	+
		1795	+ article.parent = nil
		1796	+ article.save!
		1797	+ assert_equal article.slug, article.path
		1798	+
		1799	+ article.update_attributes({:parent_id => f2.id})
		1800	+ assert_equal [f2.path,article.slug].join('/'), article.path
		1801	+ end
		1802	+
1785	end	1803	end