Skip to content

Commit dbecefa

Browse files
committed
- Use hash.except! instead of manually deleting hash options.
- No need to explicitly check if options[:prerender]'s value is true
1 parent 9f0acb0 commit dbecefa

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/react/rails/view_helper.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ module ViewHelper
88
#
99
def react_component(name, args = {}, options = {}, &block)
1010
options = {:tag => options} if options.is_a?(Symbol)
11-
block = Proc.new{concat React::Renderer.render(name, args)} if options[:prerender] == true
11+
block = Proc.new{concat React::Renderer.render(name, args)} if options[:prerender]
1212

1313
html_options = options.reverse_merge(:data => {})
1414
html_options[:data].tap do |data|
@@ -18,7 +18,7 @@ def react_component(name, args = {}, options = {}, &block)
1818
html_tag = html_options[:tag] || :div
1919

2020
# remove internally used properties so they aren't rendered to DOM
21-
[:tag, :prerender].each{|prop| html_options.delete(prop)}
21+
html_options.except!(:tag, :prerender)
2222

2323
content_tag(html_tag, '', html_options, &block)
2424
end

0 commit comments

Comments
 (0)