diff options
author | Godfrey Chan <godfreykfc@gmail.com> | 2014-02-13 11:49:19 -0800 |
---|---|---|
committer | Godfrey Chan <godfreykfc@gmail.com> | 2014-02-23 11:00:55 -0800 |
commit | 41554319f8454f667b5e315339d3e286dbf6e1a4 (patch) | |
tree | 6b429827488b2ca7f34095d5f74f208ea6a8301f /activerecord/test/cases/reload_models_test.rb | |
parent | 96759cf6c6a17053abb6a2b7cd87cdbd5a8420ba (diff) | |
download | rails-41554319f8454f667b5e315339d3e286dbf6e1a4.tar.gz rails-41554319f8454f667b5e315339d3e286dbf6e1a4.tar.bz2 rails-41554319f8454f667b5e315339d3e286dbf6e1a4.zip |
Fixed STI classes not defining an attribute method if there is a
conflicting private method defined on its ancestors.
The problem is that `method_defined_within?(name, klass, superklass)`
only works correclty when `klass` and `superklass` are both `Class`es.
If both `klass` and `superklass` are both `Class`es, they share the
same inheritance chain, so if a method is defined on `klass` but not
`superklass`, this method must be introduced at some point between
`klass` and `superklass`.
This does not work when `superklass` is a `Module`. A `Module`'s
inheritance chain contains just itself. So if a method is defined on
`klass` but not on `superklass`, the method could still be defined
somewhere upstream, e.g. in `Object`.
This fix works by avoiding calling `method_defined_within?` with a
module while still fufilling the requirement (checking that the
method is defined withing `superclass` but not is not a generated
attribute method).
4d8ee288 is likely an attempted partial fix for this problem. This
unrolls that fix and properly check the `superclass` as intended.
Fixes #11569.
Diffstat (limited to 'activerecord/test/cases/reload_models_test.rb')
0 files changed, 0 insertions, 0 deletions