closes #124

catmando · catmando · commit c5cb6d3a6cf0 · 2016-01-30T14:01:59.000-05:00
diff --git a/lib/react/component/props_wrapper.rb b/lib/react/component/props_wrapper.rb
@@ -33,7 +33,7 @@ def self.define_param(name, param_type, owner)
           end
         else
           define_method("#{name}") do
-            if param_type.respond_to? :_react_param_conversion
+            @processed_params[name] ||= if param_type.respond_to? :_react_param_conversion
               param_type._react_param_conversion props[name]
             elsif param_type.is_a?(Array) &&
               param_type[0].respond_to?(:_react_param_conversion)
@@ -49,6 +49,7 @@ def self.define_param(name, param_type, owner)
 
       def initialize(props)
         @props = props || {}
+        @processed_params = {}
       end
 
       def [](prop)
diff --git a/spec/react/param_declaration_spec.rb b/spec/react/param_declaration_spec.rb
@@ -195,6 +195,27 @@ def render
       expect(`window.dummy_log`).to eq(["Warning: Failed propType: In component `Foo`\nProvided prop `foo` could not be converted to BazWoggle"])
     end
 
+    it "can will only convert once" do
+      stub_const "BazWoggle", Class.new
+      BazWoggle.class_eval do
+        def initialize(kind)
+          @kind = kind
+        end
+        attr_accessor :kind
+        def self._react_param_conversion(json, validate_only)
+          new(json[:bazwoggle]) if json[:bazwoggle]
+        end
+      end
+      Foo.class_eval do
+        param :foo, type: BazWoggle
+        def render
+          params.foo.kind = params.foo.kind+1
+          "#{params.foo.kind}"
+        end
+      end
+      expect(React.render_to_static_markup(React.create_element(Foo, foo: {bazwoggle: 1}))).to eq('<span>2</span>')
+    end
+
     it "will alias a Proc type param" do
       Foo.class_eval do
         param :foo, type: Proc
