Hash native state/props before passing to callback

Author: ajjahn

lib/react/api.rb

@@ -31,19 +31,19 @@ def self.create_native_react_class(type)
           },
           componentWillReceiveProps: function(next_props) {
             var instance = this._getOpalInstance.apply(this);
-            return #{`instance`.component_will_receive_props(`next_props`) if type.method_defined? :component_will_receive_props};
+            return #{`instance`.component_will_receive_props(Hash.new(`next_props`)) if type.method_defined? :component_will_receive_props};
           },
           shouldComponentUpdate: function(next_props, next_state) {
             var instance = this._getOpalInstance.apply(this);
-            return #{`instance`.should_component_update?(`next_props`, `next_state`) if type.method_defined? :should_component_update?};
+            return #{`instance`.should_component_update?(Hash.new(`next_props`), Hash.new(`next_state`)) if type.method_defined? :should_component_update?};
           },
           componentWillUpdate: function(next_props, next_state) {
             var instance = this._getOpalInstance.apply(this);
-            return #{`instance`.component_will_update(`next_props`, `next_state`) if type.method_defined? :component_will_update};
+            return #{`instance`.component_will_update(Hash.new(`next_props`), Hash.new(`next_state`)) if type.method_defined? :component_will_update};
           },
           componentDidUpdate: function(prev_props, prev_state) {
             var instance = this._getOpalInstance.apply(this);
-            return #{`instance`.component_did_update(`prev_props`, `prev_state`) if type.method_defined? :component_did_update};
+            return #{`instance`.component_did_update(Hash.new(`prev_props`), Hash.new(`prev_state`)) if type.method_defined? :component_did_update};
           },
           componentWillUnmount: function() {
             var instance = this._getOpalInstance.apply(this);

lib/react/component.rb

@@ -116,7 +116,7 @@ def component_will_receive_props(next_props)
       # useful within the react.rb environment for now we are just using it to
       # clear processed_params
       State.set_state_context_to(self) do
-        self.run_callback(:before_receive_props, Hash.new(next_props))
+        self.run_callback(:before_receive_props, next_props)
       end
     rescue Exception => e
       self.class.process_exception(e, self)
@@ -129,9 +129,8 @@ def props_changed?(next_props)
 
     def should_component_update?(next_props, next_state)
       State.set_state_context_to(self) do
-        next_props = Hash.new(next_props)
         if self.respond_to?(:needs_update?)
-          !!self.needs_update?(next_props, Hash.new(next_state))
+          !!self.needs_update?(next_props, next_state)
         elsif false # switch to true to force updates per standard react
           true
         elsif props_changed? next_props
@@ -150,15 +149,15 @@ def should_component_update?(next_props, next_state)
 
     def component_will_update(next_props, next_state)
       State.set_state_context_to(self) do
-        self.run_callback(:before_update, Hash.new(next_props), Hash.new(next_state))
+        self.run_callback(:before_update, next_props, next_state)
       end
     rescue Exception => e
       self.class.process_exception(e, self)
     end
 
     def component_did_update(prev_props, prev_state)
       State.set_state_context_to(self) do
-        self.run_callback(:after_update, Hash.new(prev_props), Hash.new(prev_state))
+        self.run_callback(:after_update, prev_props, prev_state)
         State.update_states_to_observe
       end
     rescue Exception => e