diff options
author | Kasper Timm Hansen <kaspth@gmail.com> | 2019-07-26 22:00:56 +0200 |
---|---|---|
committer | Kasper Timm Hansen <kaspth@gmail.com> | 2019-07-26 22:08:34 +0200 |
commit | efa2299ac3bbf86feacfa337645654f87109e769 (patch) | |
tree | a16064fea665505ed59aeb5113a332f6f325c172 /activesupport/lib | |
parent | a0e58e687dec52d42ebadfbb72e1eae449b13ba5 (diff) | |
download | rails-efa2299ac3bbf86feacfa337645654f87109e769.tar.gz rails-efa2299ac3bbf86feacfa337645654f87109e769.tar.bz2 rails-efa2299ac3bbf86feacfa337645654f87109e769.zip |
Remove tough to grasp -1 + 1 = 0 from String#to
In case a negative position is provided that exceeds the size of the
string, we're relying on -1 returned from max to get 0 length by + 1
and let [] with a 0 length returning "" for us.
E.g. "hello".to(-7), where -7 + 5 size = -2. That's
lower than -1, so we use -1 instead and + 1 would turn it into 0.
Instead allow outer bounds access and always return "".
Diffstat (limited to 'activesupport/lib')
-rw-r--r-- | activesupport/lib/active_support/core_ext/string/access.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activesupport/lib/active_support/core_ext/string/access.rb b/activesupport/lib/active_support/core_ext/string/access.rb index 4894193665..4f70e7f5fc 100644 --- a/activesupport/lib/active_support/core_ext/string/access.rb +++ b/activesupport/lib/active_support/core_ext/string/access.rb @@ -61,8 +61,8 @@ class String # str.from(0).to(-1) # => "hello" # str.from(1).to(-2) # => "ell" def to(position) - position = [position + length, -1].max if position < 0 - self[0, position + 1] + position += size if position < 0 + self[0, position + 1].to_s end # Returns the first character. If a limit is supplied, returns a substring |