diff options
author | Ricardo Díaz <ricardotk002@gmail.com> | 2017-08-02 00:10:36 -0500 |
---|---|---|
committer | Ricardo Díaz <ricardotk002@gmail.com> | 2017-08-02 00:41:09 -0500 |
commit | 1d65185c90b0604d1b0a3ff3e9b53c3d6361dd3e (patch) | |
tree | 1062e04c189cf160eb0a45f21b7305135fcdd5b5 /activesupport/lib/active_support | |
parent | f9a43f28c087f8ffd35ff7c33a60c938b60f2be2 (diff) | |
download | rails-1d65185c90b0604d1b0a3ff3e9b53c3d6361dd3e.tar.gz rails-1d65185c90b0604d1b0a3ff3e9b53c3d6361dd3e.tar.bz2 rails-1d65185c90b0604d1b0a3ff3e9b53c3d6361dd3e.zip |
Update String#camelize to provide feedback when wrong option is passed
String#camelize was returning nil without any feedback when an
invalid option was passed as parameter. This update makes the method
to raises an ArgumentError when the option passed is invalid, similar
to what Ruby does for String#downcase (and others) in 2.4.1.
https://ruby-doc.org/core-2.4.1/String.html#method-i-downcase
Diffstat (limited to 'activesupport/lib/active_support')
-rw-r--r-- | activesupport/lib/active_support/core_ext/string/inflections.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/activesupport/lib/active_support/core_ext/string/inflections.rb b/activesupport/lib/active_support/core_ext/string/inflections.rb index 846600c623..b5bb385033 100644 --- a/activesupport/lib/active_support/core_ext/string/inflections.rb +++ b/activesupport/lib/active_support/core_ext/string/inflections.rb @@ -94,6 +94,8 @@ class String ActiveSupport::Inflector.camelize(self, true) when :lower ActiveSupport::Inflector.camelize(self, false) + else + raise ArgumentError, "Invalid option, use either :upper or :lower." end end alias_method :camelcase, :camelize |