fix should_component_update?

Author: janbiedermann

lib/react/api.rb

@@ -73,7 +73,7 @@ class extends React.Component {
           }
           shouldComponentUpdate(next_props, next_state) {
             var instance = this._getOpalInstance.apply(this);
-            return #{`instance`.should_component_update?(Hash.new(`next_props`), Hash.new(`next_state`)) if type.method_defined? :should_component_update?};
+            return #{`instance`.should_component_update?(`Opal.hash(next_props !== null ? next_props : {})`, `Opal.hash(next_state !== null ? next_state : {})`) if type.method_defined? :should_component_update?};
           }
           componentWillUpdate(next_props, next_state) {
             var instance = this._getOpalInstance.apply(this);

lib/react/component/should_component_update.rb

@@ -25,7 +25,7 @@ module Component
     module ShouldComponentUpdate
       def should_component_update?(native_next_props, native_next_state)
         State.set_state_context_to(self, false) do
-          next_params = Hash.new(native_next_props)
+          next_params = native_next_props.clone
           # rubocop:disable Style/DoubleNegation # we must return true/false to js land
           if respond_to?(:needs_update?)
             !!call_needs_update(next_params, native_next_state)
@@ -65,6 +65,7 @@ def call_needs_update(next_params, native_next_state)
 
       # rubocop:disable Metrics/MethodLength # for effeciency we want this to be one method
       def native_state_changed?(next_state)
+        next_state = next_state_hash.to_n
         %x{
           var current_state = #{@native}.state
           var normalized_next_state =