Download as
Browse Code »

Commit 798c80a29afc96c109a523b84e7a517bd384236a

Authored by AntonioTerceiro
1 parent d262a9ed
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

ActionItem53: when removing profile, remove also its pages 



git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@401 3f533792-8f58-4932-b0fe-aaf55b0a4547
Showing 2 changed files with 13 additions and 0 deletions   Show diff stats
app/models/profile.rb
  Diff comments   View file @798c80a
... ... @@ -11,6 +11,11 @@ class Profile < ActiveRecord::Base
11 11 homepage.save!
12 12 end
13 13  
  14 + after_destroy do |profile|
  15 + Comatose::Page.find_by_path(profile.identifier).destroy
  16 + end
  17 +
  18 +
14 19 # Valid identifiers must match this format.
15 20 IDENTIFIER_FORMAT = /^[a-z][a-z0-9_]*[a-z0-9]$/
16 21  
... ...
test/unit/profile_test.rb
  Diff comments   View file @798c80a
... ... @@ -85,4 +85,12 @@ class ProfileTest < Test::Unit::TestCase
85 85 assert Profile.search('bla').include?(p)
86 86 assert ! Profile.search('not_wanted').include?(p)
87 87 end
  88 +
  89 + def test_should_remove_pages_when_removing_profile
  90 + profile = Profile.create(:name => 'To bee removed', :identifier => 'to_be_removed')
  91 + assert Comatose::Page.find_by_path('to_be_removed')
  92 + profile.destroy
  93 + assert !Comatose::Page.find_by_path('to_be_removed')
  94 + end
  95 +
88 96 end
... ...