diff options
author | Hincu Petru <hincupetru@gmail.com> | 2013-12-19 17:30:01 +0000 |
---|---|---|
committer | Hincu Petru <hincupetru@gmail.com> | 2013-12-19 17:30:01 +0000 |
commit | 4cb30d9f8b091c9eb1d7bf90c2974b96ea3a59f8 (patch) | |
tree | 1238c4ec89bd4e09d0208dbd1ddd4c4675248007 | |
parent | cc1f0b4c2df0fa12ad6f3fd3b496e72e6f428e9c (diff) | |
download | rails-4cb30d9f8b091c9eb1d7bf90c2974b96ea3a59f8.tar.gz rails-4cb30d9f8b091c9eb1d7bf90c2974b96ea3a59f8.tar.bz2 rails-4cb30d9f8b091c9eb1d7bf90c2974b96ea3a59f8.zip |
Fixed configurable.rb regular expression name check
-rw-r--r-- | activesupport/lib/active_support/configurable.rb | 2 | ||||
-rw-r--r-- | activesupport/test/configurable_test.rb | 6 |
2 files changed, 7 insertions, 1 deletions
diff --git a/activesupport/lib/active_support/configurable.rb b/activesupport/lib/active_support/configurable.rb index e0d39d509f..bafcadff9a 100644 --- a/activesupport/lib/active_support/configurable.rb +++ b/activesupport/lib/active_support/configurable.rb @@ -107,7 +107,7 @@ module ActiveSupport options = names.extract_options! names.each do |name| - raise NameError.new('invalid config attribute name') unless name =~ /^[_A-Za-z]\w*$/ + raise NameError.new('invalid config attribute name') unless name =~ /\A[_A-Za-z]\w*\Z/ reader, reader_line = "def #{name}; config.#{name}; end", __LINE__ writer, writer_line = "def #{name}=(value); config.#{name} = value; end", __LINE__ diff --git a/activesupport/test/configurable_test.rb b/activesupport/test/configurable_test.rb index d00273a028..cb11a4a867 100644 --- a/activesupport/test/configurable_test.rb +++ b/activesupport/test/configurable_test.rb @@ -95,6 +95,12 @@ class ConfigurableActiveSupport < ActiveSupport::TestCase config_accessor "invalid attribute name" end end + assert_raises NameError do + Class.new do + include ActiveSupport::Configurable + config_accessor "invalid attribute\nname" + end + end end def assert_method_defined(object, method) |