TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

refactor String#truncate not to use mb_chars

Browse Source
This commit is contained in:
Sergey Nartimov 2012-01-04 22:30:07 +03:00
parent 33386b065c
commit 5a7513593f
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
activesupport/lib/active_support/core_ext/string/filters.rb
View File

@ -36,14 +36,13 @@ def squish!
# "And they found that many people were sleeping better.".truncate(25, :omission => "... (continued)")
# # => "And they f... (continued)"
def truncate(length, options = {})
text = self.dup
return self.dup unless self.length > length
options[:omission] ||= "..."
length_with_room_for_omission = length - options[:omission].mb_chars.length
chars = text.mb_chars
length_with_room_for_omission = length - options[:omission].length
stop = options[:separator] ?
(chars.rindex(options[:separator].mb_chars, length_with_room_for_omission) || length_with_room_for_omission) : length_with_room_for_omission
(rindex(options[:separator], length_with_room_for_omission) || length_with_room_for_omission) : length_with_room_for_omission
(chars.length > length ? chars[0...stop] + options[:omission] : text).to_s
self[0...stop] + options[:omission]
end
end