Download as
Browse Code »

Commit 16dd16e5a9c17ae31e726af9e4095ddab2fb143e

Authored by Rodrigo Souto
Committed by Daniela Feitosa
1 parent 2a7c4de5
Exists in staging and in 42 other branches all_pending_tasks_api, api-articles-period, api_roles, caching-rails4, captcha_serpro_plugin, comments_permissions, content-manager-hostspot, elasticsearch, elasticsearch_api, elasticsearch_categories, elasticsearch_filter, elasticsearch_sort, elasticsearch_to_merge, elasticsearch_view, environment-exposes-api, export-comment-api, export-comment-paragraph, export_data, external_followers, federation-webfinger, federation_followers, federation_followers_backend, federation_oauth_provider, federation_webfinger, fix_event_date_issue, fix_notification_email, fix_string_downcase_and_upcase, follower_permition, json_cookie_serializer, login-captcha, master, master_profile_followers, oauth_external_login, oauth_login, private-scraps, private-scraps-rebase, production, production-vendorized, profile_api_improvements, tasks_keep_filter_params, user_mention, webfinger_server

tUpdating path before save

Showing 2 changed files with 24 additions and 0 deletions   Show diff stats
app/models/article.rb
  Diff comments   View file @16dd16e
... ... @@ -154,6 +154,12 @@ class Article < ActiveRecord::Base
154 154 article.advertise = true
155 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 163 # retrieves all articles belonging to the given +profile+ that are not
158 164 # sub-articles of any other article.
159 165 named_scope :top_level_for, lambda { |profile|
... ...
test/unit/article_test.rb
  Diff comments   View file @16dd16e
... ... @@ -1782,4 +1782,22 @@ class ArticleTest < ActiveSupport::TestCase
1782 1782 assert_equal license, article.license
1783 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 1803 end
... ...