Permalink
Browse files

Raise `ArgumentError` when `has_one` is used with `counter_cache`

Previously, the `has_one` macro incorrectly accepts the `counter_cache` option
due to a bug, although that options was never supported nor functional on
`has_one` and `has_one ... through` relationships. It now correctly raises an
`ArgumentError` when passed that option.

For reference, this bug was introduced in 52f8e4b.
  • Loading branch information...
1 parent 31c20a2 commit 35fd2d401938df1afc595de9b87dadd4421f44a5 @chancancode chancancode committed Nov 30, 2013
@@ -1,3 +1,10 @@
+* Previously, the `has_one` macro incorrectly accepts the `counter_cache`
+ option due to a bug, although that options was never supported nor
+ functional on `has_one` and `has_one ... through` relationships. It now
+ correctly raises an `ArgumentError` when passed that option.
+
+ *Godfrey Chan*
+
* Implement rename_index natively for MySQL >= 5.7.
*Cody Cutrer*
@@ -5,7 +5,7 @@ def self.macro
end
def self.valid_options(options)
- super + [:foreign_type, :polymorphic, :touch]
+ super + [:foreign_type, :polymorphic, :touch, :counter_cache]
end
def self.valid_dependent_options
@@ -3,7 +3,7 @@
module ActiveRecord::Associations::Builder
class SingularAssociation < Association #:nodoc:
def self.valid_options(options)
- super + [:remote, :dependent, :counter_cache, :primary_key, :inverse_of]
+ super + [:remote, :dependent, :primary_key, :inverse_of]
end
def self.define_accessors(model, reflection)
@@ -549,4 +549,12 @@ def test_has_one_autosave_with_primary_key_manually_set
assert_not_nil author.post
assert_equal author.post, post
end
+
+ def test_has_one_relationship_cannot_have_a_counter_cache
+ assert_raise(ArgumentError) do
+ Class.new(ActiveRecord::Base) do
+ has_one :thing, counter_cache: true
+ end
+ end
+ end
end
@@ -315,4 +315,12 @@ def test_assigning_has_one_through_belongs_to_with_new_record_owner
def test_has_one_through_with_custom_select_on_join_model_default_scope
assert_equal clubs(:boring_club), members(:groucho).selected_club
end
+
+ def test_has_one_through_relationship_cannot_have_a_counter_cache
+ assert_raise(ArgumentError) do
+ Class.new(ActiveRecord::Base) do
+ has_one :thing, through: :other_thing, counter_cache: true
+ end
+ end
+ end
end

0 comments on commit 35fd2d4

Please sign in to comment.