diff options
author | Xavier Noria <fxn@hashref.com> | 2016-07-25 00:31:35 +0200 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2016-07-25 00:31:35 +0200 |
commit | 7ebef567ce30d848dd8f9374fd70699fd91acb29 (patch) | |
tree | 9452737beec405626f309348b6d3c7fe4838ed91 /actionview/lib/action_view/template/handlers | |
parent | 11b463d8cbacfe2cc1662f9e314440de71806ca9 (diff) | |
download | rails-7ebef567ce30d848dd8f9374fd70699fd91acb29.tar.gz rails-7ebef567ce30d848dd8f9374fd70699fd91acb29.tar.bz2 rails-7ebef567ce30d848dd8f9374fd70699fd91acb29.zip |
systematic revision of =~ usage in AV
Where appropriate, prefer the more concise Regexp#match?,
String#include?, String#start_with?, or String#end_with?
Diffstat (limited to 'actionview/lib/action_view/template/handlers')
-rw-r--r-- | actionview/lib/action_view/template/handlers/erb.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/actionview/lib/action_view/template/handlers/erb.rb b/actionview/lib/action_view/template/handlers/erb.rb index 85a100ed4c..058b590c56 100644 --- a/actionview/lib/action_view/template/handlers/erb.rb +++ b/actionview/lib/action_view/template/handlers/erb.rb @@ -1,4 +1,5 @@ require 'erubis' +require 'active_support/core_ext/regexp' module ActionView class Template @@ -39,7 +40,7 @@ module ActionView def add_expr_literal(src, code) flush_newline_if_pending(src) - if code =~ BLOCK_EXPR + if BLOCK_EXPR.match?(code) src << '@output_buffer.append= ' << code else src << '@output_buffer.append=(' << code << ');' @@ -48,7 +49,7 @@ module ActionView def add_expr_escaped(src, code) flush_newline_if_pending(src) - if code =~ BLOCK_EXPR + if BLOCK_EXPR.match?(code) src << "@output_buffer.safe_expr_append= " << code else src << "@output_buffer.safe_expr_append=(" << code << ");" |