Now that const_get supports "Foo::Bar" syntax, so should const_defined?.
Object.const_get "Object::Foo" => Foo
Object.const_defined? "Object::Foo" => NameError (expected true).
object.c: nested path const_defined?
- object.c (rb_mod_const_defined): support nested class path as well as const_get. [Feature #7414]
#3 Updated by Erik Michaels-Ober almost 3 years ago
I also noticed this inconsistency. It seems very odd to have different behavior for Module#const_get and Module#const_defined?. I hope it's possible to get this change into 2.0.0. It's been a while since I've written any C but I'd be happy to contribute failing specs if that would be helpful.
#4 Updated by Ben Holley almost 3 years ago
I have been working on a patch, but I have some questions about expected behavior.
In the case when the parent is missing, for example:
# Foo is missing Object.const_defined?("Foo::Bar")
should it return false? Or raise NameError for "Foo"? Or call Object.const_missing(:Foo)?
Second, in the case when the parent is not a Module or Class, for example:
Foo = Object.new Object.const_defined?("Foo::Bar")
should it return false? Or raise error "Foo is not a class/module"?
#8 Updated by Nobuyoshi Nakada almost 2 years ago
- Status changed from Assigned to Closed
- % Done changed from 0 to 100