diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2013-06-03 20:09:33 +0200 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2013-06-03 20:09:33 +0200 |
commit | 59caeb6c606cb78b92a24def2bea515c85b2421e (patch) | |
tree | 8399fd996a643f8606233534690fea0b1f8e55f2 | |
parent | f9c2f76bea41c4a7ad1419844c6102211787c458 (diff) | |
parent | 75135a97907b4bdeb51eefaaaf8bd35215206fa7 (diff) | |
download | rails-59caeb6c606cb78b92a24def2bea515c85b2421e.tar.gz rails-59caeb6c606cb78b92a24def2bea515c85b2421e.tar.bz2 rails-59caeb6c606cb78b92a24def2bea515c85b2421e.zip |
Merge branch 'master' of github.com:rails/rails
4 files changed, 4 insertions, 20 deletions
diff --git a/activerecord/lib/active_record/autosave_association.rb b/activerecord/lib/active_record/autosave_association.rb index 87d4daa6d9..a8a1847554 100644 --- a/activerecord/lib/active_record/autosave_association.rb +++ b/activerecord/lib/active_record/autosave_association.rb @@ -301,7 +301,7 @@ module ActiveRecord def association_valid?(reflection, record) return true if record.destroyed? || record.marked_for_destruction? - unless valid = record.valid?(validation_context) + unless valid = record.valid? if reflection.options[:autosave] record.errors.each do |attribute, message| attribute = "#{reflection.name}.#{attribute}" diff --git a/activerecord/lib/active_record/validations/associated.rb b/activerecord/lib/active_record/validations/associated.rb index 744780d069..b4785d3ba4 100644 --- a/activerecord/lib/active_record/validations/associated.rb +++ b/activerecord/lib/active_record/validations/associated.rb @@ -2,7 +2,7 @@ module ActiveRecord module Validations class AssociatedValidator < ActiveModel::EachValidator #:nodoc: def validate_each(record, attribute, value) - if Array.wrap(value).reject {|r| r.marked_for_destruction? || r.valid?(record.validation_context) }.any? + if Array.wrap(value).reject {|r| r.marked_for_destruction? || r.valid?}.any? record.errors.add(attribute, :invalid, options.merge(:value => value)) end end diff --git a/activerecord/test/cases/validations/association_validation_test.rb b/activerecord/test/cases/validations/association_validation_test.rb index 7ac34bc71e..7e92a2b127 100644 --- a/activerecord/test/cases/validations/association_validation_test.rb +++ b/activerecord/test/cases/validations/association_validation_test.rb @@ -118,21 +118,4 @@ class AssociationValidationTest < ActiveRecord::TestCase end end - def test_validates_associated_models_in_the_same_context - Topic.validates_presence_of :title, :on => :custom_context - Topic.validates_associated :replies - Reply.validates_presence_of :title, :on => :custom_context - - t = Topic.new('title' => '') - r = t.replies.new('title' => '') - - assert t.valid? - assert !t.valid?(:custom_context) - - t.title = "Longer" - assert !t.valid?(:custom_context), "Should NOT be valid if the associated object is not valid in the same context." - - r.title = "Longer" - assert t.valid?(:custom_context), "Should be valid if the associated object is not valid in the same context." - end end diff --git a/railties/test/application/rake_test.rb b/railties/test/application/rake_test.rb index 00a4ab06aa..cfa5cc1322 100644 --- a/railties/test/application/rake_test.rb +++ b/railties/test/application/rake_test.rb @@ -159,7 +159,8 @@ module ApplicationTests end RUBY - output = Dir.chdir(app_path){ `CONTROLLER=cart rake routes` } + ENV['CONTROLLER'] = 'cart' + output = Dir.chdir(app_path){ `rake routes` } assert_equal "Prefix Verb URI Pattern Controller#Action\ncart GET /cart(.:format) cart#show\n", output end |