version 0.7.32 closes #88 closes #87 closes #86 closes #85 closes #83

Author: catmando

lib/react/component.rb

@@ -15,6 +15,7 @@ def self.included(base)
       base.include(API)
       base.include(React::Callbacks)
       base.instance_eval do
+
         class PropsWrapper
 
           def initialize(props_hash)
@@ -39,8 +40,7 @@ def [](prop)
         define_callback :before_unmount
 
         def deprecated_params_method(name, *args, &block)
-          puts "deprecated_params_method for #{self.class.name}.#{name}"
-          # TODO display deprecation message, but only once per access to a method
+          self.class.deprecation_warning "Direct access to param `#{name}`.  Use `params.#{name}` instead."
           params.send(name, *args, &block)
         end
 

lib/react/component/base.rb

@@ -1,3 +1,5 @@
+require 'react/component'
+
 module React
   module Component
     class Base

lib/react/component/class_methods.rb

@@ -18,6 +18,15 @@ def process_exception(e, component, reraise = nil)
         raise e if reraise
       end
 
+      def deprecation_warning(message)
+        @deprecation_messages ||= []
+        message = "Warning: Deprecated feature used in #{self.name}. #{message}"
+        unless @deprecation_messages.include? message
+          @deprecation_messages << message
+          IsomorphicHelpers.log message, :warning
+        end
+      end
+
       def validator
         @validator ||= React::Validator.new(self)
       end
@@ -105,12 +114,14 @@ def param(*args)
       end
 
       def required_param(name, options = {})
+        deprecation_warning "`required_param` is deprecated, use `param` instead."
         validator.requires(name, options)
       end
 
       alias_method :require_param, :required_param
 
       def optional_param(name, options = {})
+        deprecation_warning "`optional_param` is deprecated, use `param param_name: default_value` instead."
         validator.optional(name, options)
       end
 
@@ -144,13 +155,16 @@ def export_state(*states, &block)
 
       def define_state_methods(this, name, from = nil, &block)
         this.define_method("#{name}") do
+          self.class.deprecation_warning "Direct access to state `#{name}`.  Use `state.#{name}` instead." if from.nil? || from == this
           React::State.get_state(from || self, name)
         end
         this.define_method("#{name}=") do |new_state|
+          self.class.deprecation_warning "Direct assignment to state `#{name}`.  Use `#{(from && from != this) ? from : 'state'}.#{name}!` instead."
           yield name, React::State.get_state(from || self, name), new_state if block && block.arity > 0
           React::State.set_state(from || self, name, new_state)
         end
         this.define_method("#{name}!") do |*args|
+          self.class.deprecation_warning "Direct access to state `#{name}`.  Use `state.#{name}` instead."  if from.nil? or from == this
           if args.count > 0
             yield name, React::State.get_state(from || self, name), args[0] if block && block.arity > 0
             current_value = React::State.get_state(from || self, name)

lib/react/top_level.rb

@@ -1,6 +1,6 @@
 require "native"
 require 'active_support'
-require 'react/component'
+require 'react/component/base'
 
 module React
   HTML_TAGS = %w(a abbr address area article aside audio b base bdi bdo big blockquote body br
@@ -49,4 +49,13 @@ def self.render_to_static_markup(element)
   def self.unmount_component_at_node(node)
     `React.unmountComponentAtNode(node.$$class ? node[0] : node)`
   end
+
 end
+
+Element.instance_eval do
+  class ::Element::DummyContext < React::Component::Base
+  end
+  def render(&block)
+    React.render(React::RenderingContext.render(nil) {::Element::DummyContext.new.instance_eval &block}, self)
+  end
+end if Object.const_defined?("Element")

lib/reactive-ruby/version.rb

@@ -1,3 +1,3 @@
 module React
-  VERSION = "0.7.31"
+  VERSION = "0.7.32"
 end