wrap lines at 80 char so I can see it with my eyes

Author: ajjahn

lib/react/component.rb

@@ -27,7 +27,9 @@ def self.included(base)
       base.extend(ClassMethods)
 
       if base.name
-        parent = base.name.split("::").inject([Module]) { |nesting, next_const| nesting + [nesting.last.const_get(next_const)] }[-2]
+        parent = base.name.split("::").inject([Module]) { |nesting, next_const|
+          nesting + [nesting.last.const_get(next_const)]
+        }[-2]
 
         class << parent
           def method_missing(n, *args, &block)
@@ -59,7 +61,8 @@ def render
     end unless method_defined?(:render)
 
     def deprecated_params_method(name, *args, &block)
-      self.class.deprecation_warning "Direct access to param `#{name}`.  Use `params.#{name}` instead."
+      notice = "Direct access to param `#{name}`. Use `params.#{name}` instead."
+      self.class.deprecation_warning(notice)
       params.send(name, *args, &block)
     end
 
@@ -108,13 +111,13 @@ def refs
     end
 
     def state
-      #raise "No native ReactComponent associated" unless @native
       @state_wrapper ||= StateWrapper.new(@native, self)
     end
 
     def update_react_js_state(object, name, value)
       if object
-        set_state({"***_state_updated_at-***" => Time.now.to_f, "#{object.class.to_s+'.' unless object == self}#{name}" => value})
+        set_state({"***_state_updated_at-***" => Time.now.to_f,
+                   "#{object.class.to_s+'.' unless object == self}#{name}" => value})
       else
         set_state({name => value})
       end rescue nil
@@ -128,7 +131,9 @@ def component_will_mount
       IsomorphicHelpers.load_context(true) if IsomorphicHelpers.on_opal_client?
       set_state! initial_state if initial_state
       State.initialize_states(self, initial_state)
-      State.set_state_context_to(self) { self.run_callback(:before_mount) }
+      State.set_state_context_to(self) do
+        self.run_callback(:before_mount)
+      end
     rescue Exception => e
       self.class.process_exception(e, self)
     end
@@ -143,9 +148,12 @@ def component_did_mount
     end
 
     def component_will_receive_props(next_props)
-      # need to rethink how this works in opal-react, or if its actually that useful within the react.rb environment
-      # for now we are just using it to clear processed_params
-      State.set_state_context_to(self) { self.run_callback(:before_receive_props, Hash.new(next_props)) }
+      # need to rethink how this works in opal-react, or if its actually that
+      # 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))
+      end
     rescue Exception => e
       self.class.process_exception(e, self)
     end
@@ -177,7 +185,9 @@ def should_component_update?(next_props, next_state)
     end
 
     def component_will_update(next_props, next_state)
-      State.set_state_context_to(self) { self.run_callback(:before_update, Hash.new(next_props), Hash.new(next_state)) }
+      State.set_state_context_to(self) do
+        self.run_callback(:before_update, next_props, Hash.new(next_state))
+      end
     rescue Exception => e
       self.class.process_exception(e, self)
     end
@@ -210,7 +220,9 @@ def p(*args, &block)
 
     def component?(name)
       name_list = name.split("::")
-      scope_list = self.class.name.split("::").inject([Module]) { |nesting, next_const| nesting + [nesting.last.const_get(next_const)] }.reverse
+      scope_list = self.class.name.split("::").inject([Module]) do |nesting, next_const|
+        nesting + [nesting.last.const_get(next_const)]
+      end.reverse
       scope_list.each do |scope|
         component = name_list.inject(scope) do |scope, class_name|
           scope.const_get(class_name)
@@ -221,7 +233,9 @@ def component?(name)
     end
 
     def method_missing(n, *args, &block)
-      return props[n] if props.key? n # TODO deprecate and remove - done so that params shadow tags, no longer needed
+      # TODO deprecate and remove - done so that params shadow tags, no longer
+      # needed
+      return props[n] if props.key?(n)
       name = n
       if name =~ /_as_node$/
         node_only = true
@@ -254,7 +268,11 @@ def define_state(*args, &block)
 
     def _render_wrapper
       State.set_state_context_to(self) do
-        RenderingContext.render(nil) {render || ""}.tap { |element| @waiting_on_resources = element.waiting_on_resources if element.respond_to? :waiting_on_resources }
+        RenderingContext.render(nil) {render || ""}.tap do |element|
+          if element.respond_to?(:waiting_on_resources)
+            @waiting_on_resources = element.waiting_on_resources
+          end
+        end
       end
     rescue Exception => e
       self.class.process_exception(e, self)

lib/react/state.rb

@@ -36,17 +36,23 @@ class State
     class << self
       attr_reader :current_observer
 
-      def initialize_states(object, initial_values) # initialize objects' name/value pairs
+      def initialize_states(object, initial_values)
+        # initialize objects' name/value pairs
         states[object].merge!(initial_values || {})
       end
 
       def get_state(object, name, current_observer = @current_observer)
-        # get current value of name for object, remember that the current object depends on this state, current observer can be overriden with last param
-        new_observers[current_observer][object] << name if current_observer && !new_observers[current_observer][object].include?(name)
+        # get current value of name for object, remember that the current object
+        # depends on this state, current observer can be overriden with last
+        # param
+        new_observers[current_observer][object] << name if current_observer &&
+          !new_observers[current_observer][object].include?(name)
         states[object][name]
       end
 
-      def set_state2(object, name, value)  # set object's name state to value, tell all observers it has changed.  Observers must implement update_react_js_state
+      def set_state2(object, name, value)
+        # set object's name state to value, tell all observers it has changed.
+        # Observers must implement update_react_js_state
         object_needs_notification = object.respond_to? :update_react_js_state
         observers_by_name[object][name].dup.each do |observer|
           observer.update_react_js_state(object, name, value)
@@ -82,7 +88,9 @@ def is_observing?(object, name, current_observer)
         current_observer && observers_by_name[object][name].include?(current_observer)
       end
 
-      def update_states_to_observe(current_observer = @current_observer)  # should be called after the last after_render callback, currently called after components render method
+      # should be called after the last after_render callback, currently called
+      # after components render method
+      def update_states_to_observe(current_observer = @current_observer)
         raise "update_states_to_observer called outside of watch block" unless current_observer
         current_observers[current_observer].each do |object, names|
           names.each do |name|
@@ -108,7 +116,9 @@ def remove # call after component is unmounted
         current_observers.delete(@current_observer)
       end
 
-      def set_state_context_to(observer) # wrap all execution that may set or get states in a block so we know which observer is executing
+      # wrap all execution that may set or get states in a block so we know
+      # which observer is executing
+      def set_state_context_to(observer)
         if `typeof window.reactive_ruby_timing !== 'undefined'`
           @nesting_level = (@nesting_level || 0) + 1
           start_time = Time.now.to_f
@@ -120,7 +130,9 @@ def set_state_context_to(observer) # wrap all execution that may set or get stat
         return_value
       ensure
         @current_observer = saved_current_observer
-        @nesting_level = [0, @nesting_level - 1].max if `typeof window.reactive_ruby_timing !== 'undefined'`
+        if `typeof window.reactive_ruby_timing !== 'undefined'`
+          @nesting_level = [0, @nesting_level - 1].max
+        end
         return_value
       end
 
@@ -134,7 +146,6 @@ def states
           instance_variable_set("@#{method_name}", Hash.new { |h, k| h[k] = Hash.new { |h, k| h[k] = [] } })
         end
       end
-
     end
   end
 end