Remove unnecessary code from define_read_method and add assertion to make sure the underscored version is actually generated

1 files changed, 4 insertions, 7 deletions

diff --git a/activerecord/lib/active_record/attribute_methods/read.rb b/activerecord/lib/active_record/attribute_methods/read.rb
index 43f736c89c..dd3f4699f7 100644
--- a/activerecord/lib/active_record/attribute_methods/read.rb
+++ b/activerecord/lib/active_record/attribute_methods/read.rb
@@ -70,13 +70,10 @@ module ActiveRecord
             if cache_attribute?(attr_name)
               access_code = "@attributes_cache['#{attr_name}'] ||= (#{access_code})"
             end
-            if symbol =~ /^[a-zA-Z_]\w*[!?=]?$/
-              generated_attribute_methods.module_eval("def _#{symbol}; #{access_code}; end; alias #{symbol} _#{symbol}", __FILE__, __LINE__)
-            else
-              generated_attribute_methods.module_eval do
-                define_method("_#{symbol}") { eval(access_code) }
-                alias_method(symbol, "_#{symbol}")
-              end
+
+            generated_attribute_methods.module_eval do
+              define_method("_#{attr_name}") { eval(access_code) }
+              alias_method(attr_name, "_#{attr_name}")
             end
           end
       end