Merge pull request #66 from eins78/master

remove internally used properties before server render

Author: zpao

lib/react/rails/view_helper.rb

@@ -15,8 +15,11 @@ def react_component(name, args = {}, options = {}, &block)
           data[:react_class] = name
           data[:react_props] = args.to_json unless args.empty?
         end
-        html_tag = html_options.delete(:tag) || :div
-
+        html_tag = html_options[:tag] || :div
+        
+        # remove internally used properties so they aren't rendered to DOM
+        [:tag, :prerender].each{|prop| html_options.delete(prop)}
+        
         content_tag(html_tag, '', html_options, &block)
       end
 

test/view_helper_test.rb

@@ -87,4 +87,10 @@ class ViewHelperTest < ActionDispatch::IntegrationTest
     assert_match /data-react-checksum/, page.html
     assert_match /yep/, page.find("#status").text
   end
+  
+  test 'react server rendering does not include internal properties' do
+    visit '/server/1'
+    assert_no_match /tag=/, page.html
+    assert_no_match /prerender=/, page.html
+  end
 end