From c61b10b62be9f57dd847242471b4a30da81f16d4 Mon Sep 17 00:00:00 2001 From: Rick Olson Date: Tue, 25 Apr 2006 05:49:14 +0000 Subject: Raise error when trying to add to a has_many :through association. Use the Join Model instead. [Rick] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4265 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- activerecord/lib/active_record/associations.rb | 10 ++++++++++ .../active_record/associations/has_many_through_association.rb | 9 ++++++++- 2 files changed, 18 insertions(+), 1 deletion(-) (limited to 'activerecord/lib/active_record') diff --git a/activerecord/lib/active_record/associations.rb b/activerecord/lib/active_record/associations.rb index 4a637d5dab..2861a3940b 100755 --- a/activerecord/lib/active_record/associations.rb +++ b/activerecord/lib/active_record/associations.rb @@ -66,6 +66,16 @@ module ActiveRecord end end + class ReadOnlyAssociation < ActiveRecordError #:nodoc: + def initialize(reflection) + @reflection = reflection + end + + def message + "Can not add to a has_many :through association. Try adding to #{@reflection.through_reflection.name.inspect}." + end + end + module Associations # :nodoc: def self.append_features(base) super diff --git a/activerecord/lib/active_record/associations/has_many_through_association.rb b/activerecord/lib/active_record/associations/has_many_through_association.rb index 44054b42e1..8cafb26d44 100644 --- a/activerecord/lib/active_record/associations/has_many_through_association.rb +++ b/activerecord/lib/active_record/associations/has_many_through_association.rb @@ -8,7 +8,6 @@ module ActiveRecord construct_sql end - def find(*args) options = Base.send(:extract_options_from_args!, args) @@ -41,6 +40,14 @@ module ActiveRecord @loaded = false end + def <<(*args) + raise ActiveRecord::ReadOnlyAssociation, @reflection + end + + [:push, :concat, :create, :build].each do |method| + alias_method method, :<< + end + protected def method_missing(method, *args, &block) if @target.respond_to?(method) || (!@reflection.klass.respond_to?(method) && Class.respond_to?(method)) -- cgit v1.2.3