diff options
| author | schneems <richard.schneeman@gmail.com> | 2014-10-07 16:53:50 -0500 |
|---|---|---|
| committer | schneems <richard.schneeman@gmail.com> | 2014-10-07 16:54:38 -0500 |
| commit | 03d30ce71ca17cec82774a86d2acc87e2f153e53 (patch) | |
| tree | 798c64e6ca5c523ee34c49eee5128a849f58e34b /actionview/lib | |
| parent | 01aefa5b328ad0107160112fa64dec35dcb70e32 (diff) | |
| download | rails-03d30ce71ca17cec82774a86d2acc87e2f153e53.tar.gz rails-03d30ce71ca17cec82774a86d2acc87e2f153e53.tar.bz2 rails-03d30ce71ca17cec82774a86d2acc87e2f153e53.zip | |
remove un-needed sort
HTML doesn't care what order the elements are rendered in, so why should we?
Updates tests to use proper `assert_dom_equal` instead of `assert_equal` /cc @jeremy
Diffstat (limited to 'actionview/lib')
| -rw-r--r-- | actionview/lib/action_view/helpers/tag_helper.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/actionview/lib/action_view/helpers/tag_helper.rb b/actionview/lib/action_view/helpers/tag_helper.rb index c20800598f..385d57a7a5 100644 --- a/actionview/lib/action_view/helpers/tag_helper.rb +++ b/actionview/lib/action_view/helpers/tag_helper.rb @@ -160,7 +160,7 @@ module ActionView attrs << tag_option(key, value, escape) end end - " #{attrs.sort! * ' '}" unless attrs.empty? + " #{attrs * ' '}" unless attrs.empty? end def prefix_tag_option(prefix, key, value, escape) |