#to_n already done the magic

zetachang · zetachang · commit 0907f2a900dc · 2015-02-22T20:44:53.000+08:00
diff --git a/lib/react/api.rb b/lib/react/api.rb
@@ -59,24 +59,24 @@ def self.create_element(type, properties = {}, &block)
       end
 
       # Passed in properties
-      props = `{}`
+      props = {}
       properties.map do |key, value|
          if key == "class_name" && value.is_a?(Hash)
-           `props[#{lower_camelize(key)}] = React.addons.classSet(#{value.to_n})`
+           props[lower_camelize(key)] = `React.addons.classSet(#{value.to_n})`
          else
-           `props[#{lower_camelize(key)}] = #{value}`
+           props[lower_camelize(key)] = value
          end
       end
       params << props
 
       # Children Nodes
       if block_given?
         children = [yield].flatten.each do |ele|
-          params << ele.to_n
+          params << ele
         end
       end
 
-      return React::Element.new(`React.createElement.apply(null, #{params})`)
+      return React::Element.new(`React.createElement.apply(null, #{params.to_n})`)
     end
 
     def self.clear_component_class_cache
diff --git a/spec/component_spec.rb b/spec/component_spec.rb
@@ -254,6 +254,17 @@ def render
         element = renderToDocument(Foo, prop: "foobar")
         expect(element.getDOMNode.textContent).to eq("foobar")
       end
+
+      it "should access nested params as orignal Ruby object" do
+        Foo.class_eval do
+          def render
+            React.create_element("div") { params[:prop][:foo] }
+          end
+        end
+
+        element = renderToDocument(Foo, prop: {foo: 10})
+        expect(element.getDOMNode.textContent).to eq("10")
+      end
     end
 
     describe "Prop validation" do
