diff options
author | Steve Klabnik <steve@steveklabnik.com> | 2012-11-28 13:57:07 -0800 |
---|---|---|
committer | Steve Klabnik <steve@steveklabnik.com> | 2012-11-28 13:58:51 -0800 |
commit | bd852603a18a3a9d20597382a97f9b0f5bb0fc94 (patch) | |
tree | 00c6d3415b4a90e3700a21bf73aafb88cc7e9936 | |
parent | 27b79f4c911fdfce186086cfaa6646f1c2083aa0 (diff) | |
download | rails-bd852603a18a3a9d20597382a97f9b0f5bb0fc94.tar.gz rails-bd852603a18a3a9d20597382a97f9b0f5bb0fc94.tar.bz2 rails-bd852603a18a3a9d20597382a97f9b0f5bb0fc94.zip |
Simplify String#mb_chars and stop proxying #class
This behavior mattered under Ruby 1.8, but that doesn't matter now
that we don't support it.
In addition, we don't want to proxy the #class method. A test was added
to prevent against regressions.
-rw-r--r-- | activesupport/CHANGELOG.md | 2 | ||||
-rw-r--r-- | activesupport/lib/active_support/core_ext/string/multibyte.rb | 6 | ||||
-rw-r--r-- | activesupport/test/multibyte_chars_test.rb | 5 |
3 files changed, 7 insertions, 6 deletions
diff --git a/activesupport/CHANGELOG.md b/activesupport/CHANGELOG.md index 504ebcb2fe..8ef9ffb8db 100644 --- a/activesupport/CHANGELOG.md +++ b/activesupport/CHANGELOG.md @@ -1,5 +1,7 @@ ## Rails 4.0.0 (unreleased) ## +* No longer proxy ActiveSupport::Multibyte#class. *Steve Klabnik* + * Deprecate `ActiveSupport::TestCase#pending` method, use `skip` from MiniTest instead. *Carlos Antonio da Silva* * `XmlMini.with_backend` now may be safely used with threads: diff --git a/activesupport/lib/active_support/core_ext/string/multibyte.rb b/activesupport/lib/active_support/core_ext/string/multibyte.rb index 0ea5fe3571..a124202936 100644 --- a/activesupport/lib/active_support/core_ext/string/multibyte.rb +++ b/activesupport/lib/active_support/core_ext/string/multibyte.rb @@ -33,11 +33,7 @@ class String # For more information about the methods defined on the Chars proxy see ActiveSupport::Multibyte::Chars. For # information about how to change the default Multibyte behavior see ActiveSupport::Multibyte. def mb_chars - if ActiveSupport::Multibyte.proxy_class.consumes?(self) - ActiveSupport::Multibyte.proxy_class.new(self) - else - self - end + ActiveSupport::Multibyte.proxy_class.new(self) end def is_utf8? diff --git a/activesupport/test/multibyte_chars_test.rb b/activesupport/test/multibyte_chars_test.rb index ef289692bc..0088a06c34 100644 --- a/activesupport/test/multibyte_chars_test.rb +++ b/activesupport/test/multibyte_chars_test.rb @@ -47,7 +47,7 @@ class MultibyteCharsTest < ActiveSupport::TestCase end def test_methods_are_forwarded_to_wrapped_string_for_byte_strings - assert_equal BYTE_STRING.class, BYTE_STRING.mb_chars.class + assert_equal BYTE_STRING.length, BYTE_STRING.mb_chars.length end def test_forwarded_method_with_non_string_result_should_be_returned_vertabim @@ -673,6 +673,9 @@ class MultibyteCharsExtrasTest < ActiveSupport::TestCase assert_equal "𥤤", chars(byte_string).tidy_bytes(true) end + def test_class_is_not_forwarded + assert_equal BYTE_STRING.dup.mb_chars.class, ActiveSupport::Multibyte::Chars + end private |