From: matz@... Date: 2020-05-14T08:42:13+00:00 Subject: [ruby-core:98346] [Ruby master Feature#16832] Use #name rather than #inspect to build "uninitialized constant" error messages Issue #16832 has been updated by matz (Yukihiro Matsumoto). Sounds OK. Let's see how it works. Matz. ---------------------------------------- Feature #16832: Use #name rather than #inspect to build "uninitialized constant" error messages https://bugs.ruby-lang.org/issues/16832#change-85613 * Author: byroot (Jean Boussier) * Status: Open * Priority: Normal ---------------------------------------- While debugging a bug in Rails (https://github.com/rails/rails/pull/37632#issuecomment-623387954) I noticed `NameError` calls `inspect` on the `const_get` receiver to build its error message. The problem is that some libraries such as Active Record have been redefining `inspect` for years to provide human readable information, e.g.: ``` >> Shipit::Stack.inspect => "Shipit::Stack (call 'Shipit::Stack.connection' to establish a connection)" >> Shipit::Stack.connection; nil >> Shipit::Stack.inspect => "Shipit::Stack(id: integer, environment: string, ...)" ``` Which makes for fairly unhelpful error messages: ``` >> Shipit::Stack.const_get(:Foo) Traceback (most recent call last): 2: from (irb):4 1: from (irb):4:in `const_get' NameError (uninitialized constant #::Foo) ``` So perhaps it's Active Record that is at fault here, but from my understanding since the goal is to display the constant path that was requested, `name` is much more likely to return a relevant constant name. Proposed patch: https://github.com/ruby/ruby/pull/3080 -- https://bugs.ruby-lang.org/ Unsubscribe: