From: shugo@...
Date: 2015-12-15T04:28:50+00:00
Subject: [ruby-core:72130] [Ruby trunk - Feature #11547] remove top-level	constant lookup

Issue #11547 has been updated by Shugo Maeda.


Corin Langosch wrote:
> Hi Shugo. Just as it does now, it doesn't perform any magic (fallback to some other scope) and doesn't emit any warning. How is it related to my bug report/ feature request? Cheers, Corin.

In my example, `Baz::X` and `Baz::Y` refer to constants defined in ancestors of `Baz`.
In your example, `Twitter::Auth` also refers to a constant defined one of its ancestors, `Object`.

So, I'd like to clarify what behavior do you want in these cases.

Instead of changing the behavior of constant lookup, we may be able to introduce a variant of `const_missing` which is invoked when a constant is not directly defined in the target class.


----------------------------------------
Feature #11547: remove top-level constant lookup
https://bugs.ruby-lang.org/issues/11547#change-55541

* Author: Corin Langosch
* Status: Feedback
* Priority: Normal
* Assignee: 
----------------------------------------
If ruby cannot find a class in the specified scope it uses the top-level constant of the same name if it exists and emits a warning:

~~~
irb(main):006:0> class Auth; end
=> nil
irb(main):007:0> class Twitter; end
=> nil
irb(main):008:0> Twitter::Auth
(irb):8: warning: toplevel constant Auth referenced by Twitter::Auth
=> Auth
~~~

In some cases this is not playing nicely with rails autoloading as can be seen here: https://github.com/rails/rails/issues/6931. Many more issues like this exist.

Imo I don't see any reason why this fallback makes any sense. So I'd like to suggest to remove it completely or at least add an option to disable it.





-- 
https://bugs.ruby-lang.org/