mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
db5d26c9d7
It was causing error when using `with_options` passing a lambda as its
last argument.
class User < ActiveRecord::Base
with_options dependent: :destroy do |assoc|
assoc.has_many :profiles, -> { where(active: true) }
end
end
It was happening because the `option_merger` was taking the last
argument and checking if it was a Hash. This breaks the HasMany usage,
because its last argument can be a Hash or a Proc.
As the behavior described in this test:
https://github.com/rails/rails/blob/master/activesupport/test/option_merger_test.rb#L69
the method will only accept the lambda, this way it will keep the expected behavior. See 9eaa0a34
10 lines
260 B
Ruby
10 lines
260 B
Ruby
require_dependency 'models/arunit2_model'
|
|
require 'active_support/core_ext/object/with_options'
|
|
|
|
class College < ARUnit2Model
|
|
has_many :courses
|
|
|
|
with_options dependent: :destroy do |assoc|
|
|
assoc.has_many :students, -> { where(active: true) }
|
|
end
|
|
end
|