diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2005-11-19 06:47:26 +0000 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2005-11-19 06:47:26 +0000 |
commit | b51c3399eb0cbf07e64bd7a33d176eae3010d5c0 (patch) | |
tree | 85d4b0c59dbc72d36e3c2e59ed21bbc34acd0a17 /activerecord | |
parent | b3c65bc8490b072a0d314aa3858c680fc6d67e35 (diff) | |
download | rails-b51c3399eb0cbf07e64bd7a33d176eae3010d5c0.tar.gz rails-b51c3399eb0cbf07e64bd7a33d176eae3010d5c0.tar.bz2 rails-b51c3399eb0cbf07e64bd7a33d176eae3010d5c0.zip |
Don't generate read methods for columns whose names are not valid ruby method names. References #2946.
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@3089 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activerecord')
-rw-r--r-- | activerecord/CHANGELOG | 2 | ||||
-rwxr-xr-x | activerecord/lib/active_record/base.rb | 11 | ||||
-rwxr-xr-x | activerecord/test/base_test.rb | 7 |
3 files changed, 19 insertions, 1 deletions
diff --git a/activerecord/CHANGELOG b/activerecord/CHANGELOG index 1196065110..1ca945b8d3 100644 --- a/activerecord/CHANGELOG +++ b/activerecord/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Don't generate read methods for columns whose names are not valid ruby method names. #2946 [Stefan Kaes] + * Document :force option to create_table. #2921 [Blair Zajac <blair@orcaware.com>] * Don't add the same conditions twice in has_one finder sql. #2916 [Jeremy Evans] diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb index 550b657102..2f7812d17b 100755 --- a/activerecord/lib/active_record/base.rb +++ b/activerecord/lib/active_record/base.rb @@ -1563,7 +1563,16 @@ module ActiveRecord #:nodoc: self.class.read_methods << attr_name end - self.class.class_eval("def #{symbol}; #{access_code}; end") + begin + self.class.class_eval("def #{symbol}; #{access_code}; end") + rescue SyntaxError => err + self.class.read_methods.delete(attr_name) + if logger + logger.warn "Exception occured during reader method compilation." + logger.warn "Maybe #{attr_name} is not a valid Ruby identifier?" + logger.warn "#{err.message}" + end + end end # Returns true if the attribute is of a text column and marked for serialization. diff --git a/activerecord/test/base_test.rb b/activerecord/test/base_test.rb index e66c90c7fe..1ad6e6c3ee 100755 --- a/activerecord/test/base_test.rb +++ b/activerecord/test/base_test.rb @@ -221,6 +221,13 @@ class BasicsTest < Test::Unit::TestCase end end + def test_reader_for_invalid_column_names + # column names which aren't legal ruby ids + topic = Topic.find(:first) + topic.send(:define_read_method, "mumub-jumbo".to_sym, "mumub-jumbo", nil) + assert !Topic.read_methods.include?("mumub-jumbo") + end + def test_non_attribute_access_and_assignment topic = Topic.new assert !topic.respond_to?("mumbo") |