Download as
Browse Code »

Commit f185c568fad2fe11e2d47e86d593a4f94c7ebe09

Authored by AntonioTerceiro
1 parent 7787940b
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

ActionItem70: don't allow duplicated slugs 



git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@522 3f533792-8f58-4932-b0fe-aaf55b0a4547
Showing 2 changed files with 11 additions and 1 deletions   Show diff stats
app/models/category.rb
  Diff comments   View file @f185c56
1 1 class Category < ActiveRecord::Base
2 2  
3 3 validates_presence_of :name, :environment_id
  4 + validates_uniqueness_of :slug,:scope => [ :environment_id, :parent_id ], :message => N_('%{fn} is already being used by another category.')
4 5 belongs_to :environment
5 6  
6 7 acts_as_tree :order => 'name'
... ...
test/unit/category_test.rb
  Diff comments   View file @f185c56
... ... @@ -118,7 +118,7 @@ class CategoryTest &lt; Test::Unit::TestCase
118 118 assert_equal 'parent/child', c2.path
119 119 end
120 120  
121   - def test_save_
  121 + def test_should_set_path_correctly_before_saving
122 122 c1 = Category.create!(:name => 'parent', :environment_id => @env.id)
123 123  
124 124 c2 = Category.new(:name => 'child', :environment_id => @env.id)
... ... @@ -128,4 +128,13 @@ class CategoryTest &lt; Test::Unit::TestCase
128 128 assert_equal 'parent/child', c2.path
129 129 end
130 130  
  131 + def test_should_refuse_to_duplicate_slug_under_the_same_parent
  132 + c1 = Category.create!(:name => 'test category', :environment_id => @env.id)
  133 + c2 = Category.new(:name => 'Test: Category', :environment_id => @env.id)
  134 +
  135 + assert !c2.valid?
  136 + assert c2.errors.invalid?(:slug)
  137 +
  138 + end
  139 +
131 140 end
... ...