Bug #16242
closedRefinements method call to failed
Description
Refinements method call to failed with prepend
.
No problem¶
module M
# Methods that you want to call only within this context
refine M do
def refine_method
"refine_method"
end
end
using M
def hoge
# OK: `refine_method` calls the method
refine_method
end
end
class X
include M
end
pp X.new.hoge
# => "refine_method"
Problem¶
- If
prepend
another module, the search forrefine_method
fails
module OtherM
end
module M
# Added prepend
prepend OtherM
refine M do
def refine_method
"refine_method"
end
end
using M
def hoge
# Error: `hoge': undefined local variable or method `refine_method' for #<X:0x00007fa05a024390> (NameError)
refine_method
end
end
class X
include M
end
# Error
pp X.new.hoge
Environment¶
Reproduced in the following environment.
RUBY_VERSION # => "2.7.0"
RUBY_REVISION # => "02dfa0f16361c498e0f529054b00e3e09730892b"
RUBY_PLATFORM # => "x86_64-darwin17"
You can also confirm that it is reproduced here.
https://wandbox.org/permlink/I6tQesDRntT7JZqB
Updated by wanabe (_ wanabe) about 5 years ago
- Related to Bug #13446: refinements with prepend for module has strange behavior added
Updated by jeremyevans0 (Jeremy Evans) about 5 years ago
This issue is specific to modules that are refined and use prepend. The reason it does not work:
-
Refining a module or class that prepends other modules places the refinements in the class itself and not the origin iclass.
-
Inclusion of a module that prepends other modules skips the module itself, including only iclasses for the prepended modules and the origin iclass.
Those two behaviors combined meant that the method table for the refined methods for the included module never ends up in the method lookup chain for the class including the module.
Fix this by not skipping the module itself when the module is included (see https://github.com/ruby/ruby/pull/2550). This requires some code rearranging in rb_include_class_new to make sure the correct method tables and origin settings are used for the created iclass.
As origin iclasses shouldn't be exposed to Ruby, this also requires skipping modules that have origin iclasses in Module#ancestors (classes that have origin iclasses were already skipped).
Updated by osyo (manga osyo) about 5 years ago
Its greated!!
Thanks jeremy :)
I will read pull request.
Updated by jeremyevans (Jeremy Evans) almost 5 years ago
- Status changed from Open to Closed
Applied in changeset git|5069c5f5214ce68df8b3954321ad9114c5368dc3.
Honor refinements for modules that prepend other modules
This previously did not work, and the reason it did not work is
that:
-
Refining a module or class that prepends other modules places
the refinements in the class itself and not the origin iclass. -
Inclusion of a module that prepends other modules skips the
module itself, including only iclasses for the prepended modules
and the origin iclass.
Those two behaviors combined meant that the method table for the
refined methods for the included module never ends up in the
method lookup chain for the class including the module.
Fix this by not skipping the module itself when the module is
included. This requires some code rearranging in
rb_include_class_new to make sure the correct method tables and
origin settings are used for the created iclass.
As origin iclasses shouldn't be exposed to Ruby, this also
requires skipping modules that have origin iclasses in
Module#ancestors (classes that have origin iclasses were already
skipped).
Fixes [Bug #16242]