view_component_reflex 3.1.5 → 3.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 63a22f733ad6878a7014b1b22ea8de86df54d0f1a7be763f7436998970ea8823
-  data.tar.gz: e2c6b8ef91bf34ed84a974a8a4e2380c589c8d06dbf85475850390bf7966c599
+  metadata.gz: 16b2b912f26f5a3ac4cb26e659d443ce52bca8458154e4cc464ea29d484e9501
+  data.tar.gz: 5b320e62171c3ab7e436f39db45c98c52f7274b8f477a6f9ee11af13ac2b1899
 SHA512:
-  metadata.gz: 9c231c61349cf8e823f361bd9c2efa768028b2d2c2b794d6f540306068dee0d37fa9d4c72a4e8e8bd6733156f625e320c445dbcd47a6725184755776072612e9
-  data.tar.gz: 4c5c3727f07b6d9c997386468a13a1c9474e83feb66b0545f27389c9c22e36bac4b33b18b62147e3504fc8f862d60cd35eee36ba1f543d9ad2dcb779be39a502
+  metadata.gz: 22d9b03168e43de149a861420b8b819be452ffeb474b8b700cc74af37115cc98849284f2723c920408db60acc957121bf519a1430f65b4efeda503494ccd511f
+  data.tar.gz: b0dee0a65bdcf956cebaac0220c615f8fcdbd15d3ecbdeb147129a76612a00556813940416da1938d2f2f4c686898855e8bc5c861eda3441e5fddd4442fe6d17

data/app/components/view_component_reflex/component.rb

@@ -1,6 +1,7 @@
 module ViewComponentReflex
   class Component < ViewComponent::Base
     class_attribute :reflex_base_class, default: ViewComponentReflex::Reflex
+    attr_reader :key
 
     class << self
       def init_stimulus_reflex
@@ -55,7 +56,7 @@ module ViewComponentReflex
     end
 
     def component_controller(opts_or_tag = :div, opts = {}, &blk)
-      init_key
+      initialize_component
 
       tag = :div
       options = if opts_or_tag.is_a? Hash
@@ -77,11 +78,64 @@ module ViewComponentReflex
       omitted_from_state.empty?
     end
 
-    # key is required if you're using state
-    # We can't initialize the session state in the initial method
-    # because it doesn't have a view_context yet
-    # This is the next best place to do it
-    def init_key
+    # We can't truly initialize the component without the view_context,
+    # which isn't available in the `initialize` method. We require the 
+    # developer to wrap components in `component_controller`, so this is where
+    # we truly initialize the component.
+    # This method is overridden in reflex.rb when the component is re-rendered. The 
+    # override simply sets @key to element.dataset[:key]
+    # We don't want it to initialize the state again, and since we're rendering the component
+    # outside of the view, we need to skip the initialize_key method as well
+    def initialize_component
+      initialize_key
+      initialize_state
+    end
+
+    # Note to self:
+    # This has to be in the Component class because there are situations
+    # where the controller is the one rendering the component
+    # so we can't rely on the component created by the reflex
+    def initialize_state
+      return if state_initialized?
+      adapter = ViewComponentReflex::Engine.state_adapter
+      
+      # newly mounted
+      if !stimulus_reflex? || adapter.state(request, @key).empty?
+
+        new_state = create_safe_state
+
+        adapter.wrap_write_async do
+          adapter.store_state(request, @key, new_state)
+          adapter.store_state(request, "#{@key}_initial", new_state)
+        end
+
+      # updating a mounted component
+      else
+        initial_state = adapter.state(request, "#{@key}_initial")
+
+        parameters_changed = []
+        adapter.state(request, @key).each do |k, v|
+          instance_value = instance_variable_get(k)
+          if permit_parameter?(initial_state[k], instance_value)
+            parameters_changed << k
+            adapter.wrap_write_async do
+              adapter.set_state(request, controller, "#{@key}_initial", {k => instance_value})
+              adapter.set_state(request, controller, @key, {k => instance_value})
+            end
+          else
+            instance_variable_set(k, v)
+          end
+        end
+        after_state_initialized(parameters_changed)
+      end
+      @state_initialized = true
+    end
+
+    def state_initialized?
+      @state_initialized
+    end
+
+    def initialize_key
       # we want the erb file that renders the component. `caller` gives the file name,
       # and line number, which should be unique. We hash it to make it a nice number
       erb_file = caller.select { |p| p.match? /.\.html\.(haml|erb|slim)/ }[1]
@@ -137,44 +191,6 @@ module ViewComponentReflex
     #   # no op
     # end
 
-    def key
-      adapter = ViewComponentReflex::Engine.state_adapter
-
-      # initialize session state
-      if (!stimulus_reflex? || adapter.state(request, @key).empty?) && !@initialized_state
-
-        new_state = create_safe_state
-
-        adapter.wrap_write_async do
-          adapter.store_state(request, @key, new_state)
-          adapter.store_state(request, "#{@key}_initial", new_state)
-        end
-      elsif !@initialized_state
-        initial_state = adapter.state(request, "#{@key}_initial")
-
-        # incoming_params = safe_instance_variables.each_with_object({}) { |var, obj| obj[var] = instance_variable_get(var) }
-        # receive_params(ViewComponentReflex::Engine.state_adapter.state(request, @key), incoming_params)
-
-        parameters_changed = []
-        adapter.state(request, @key).each do |k, v|
-          instance_value = instance_variable_get(k)
-          if permit_parameter?(initial_state[k], instance_value)
-            parameters_changed << k
-            adapter.wrap_write_async do
-              adapter.set_state(request, controller, "#{@key}_initial", {k => instance_value})
-              adapter.set_state(request, controller, @key, {k => instance_value})
-            end
-          else
-            instance_variable_set(k, v)
-          end
-        end
-        after_state_initialized(parameters_changed)
-      end
-
-      @initialized_state = true
-      @key
-    end
-
     def safe_instance_variables
       instance_variables - unsafe_instance_variables - omitted_from_state
     end
@@ -185,10 +201,23 @@ module ViewComponentReflex
       [
         :@view_context, :@lookup_context, :@view_renderer, :@view_flow,
         :@virtual_path, :@variant, :@current_template, :@output_buffer, :@key,
-        :@helpers, :@controller, :@request, :@tag_builder, :@initialized_state
+        :@helpers, :@controller, :@request, :@tag_builder, :@state_initialized,
+        :@_content_evaluated, :@_render_in_block
       ]
     end
 
+    def content
+      if cached_content && !@_render_in_block
+        cached_content
+      else
+        super
+      end
+    end
+
+    def cached_content
+      @__cached_content__
+    end
+
     def create_safe_state
       new_state = {}
 
@@ -196,6 +225,9 @@ module ViewComponentReflex
       safe_instance_variables.each do |k|
         new_state[k] = instance_variable_get(k)
       end
+
+      new_state[:@__cached_content__] = content
+
       new_state
     end
 

data/lib/view_component_reflex/reflex.rb

@@ -5,6 +5,13 @@ module ViewComponentReflex
       attr_accessor :component_class
     end
 
+    # pretty sure I can't memoize this because we need
+    # to re-render every time
+    def controller_document
+      controller.process(params[:action])
+      Nokogiri::HTML(controller.response.body)
+    end
+
     def refresh!(primary_selector = nil, *rest)
       save_state
 
@@ -13,9 +20,8 @@ module ViewComponentReflex
       end
       if primary_selector
         prevent_refresh!
-
-        controller.process(params[:action])
-        document = Nokogiri::HTML(controller.response.body)
+        
+        document = controller_document
         [primary_selector, *rest].each do |s|
           html = document.css(s)
           if html.present?
@@ -24,14 +30,13 @@ module ViewComponentReflex
               html: html.inner_html,
               children_only: true,
               permanent_attribute_name: "data-reflex-permanent",
-              stimulus_reflex: stimulus_reflex_data
             )
           end
         end
       else
         refresh_component!
       end
-      CableReady::Channels.instance.broadcast
+      CableReady::Channels.instance[stream].broadcast
     end
 
     def stream
@@ -42,22 +47,35 @@ module ViewComponentReflex
       @stream = channel
     end
 
-    def refresh_component!
+    def component_document
       component.tap do |k|
-        k.define_singleton_method(:key) do
-          element.dataset[:key]
+        k.define_singleton_method(:initialize_component) do
+          @key = element.dataset[:key]
         end
       end
+
       document = Nokogiri::HTML(component.render_in(controller.view_context))
+    end
+
+    def refresh_component!
       CableReady::Channels.instance[stream].morph(
         selector: selector,
         children_only: true,
-        html: document.css(selector).inner_html,
+        html: component_document.css(selector).inner_html,
         permanent_attribute_name: "data-reflex-permanent",
-        stimulus_reflex: stimulus_reflex_data
       )
     end
 
+    def default_morph
+      save_state
+      html = if component.can_render_to_string?
+        component_document.css(selector).to_html
+      else
+        controller_document.css(selector).to_html
+      end
+      morph selector, html
+    end
+
     def stimulus_reflex_data
       {
         reflex_id: reflex_id,
@@ -96,27 +114,20 @@ module ViewComponentReflex
       !!name.to_proc
     end
 
-    # this is copied out of stimulus_reflex/reflex.rb and modified
-    def morph(selectors, html = "")
-      case selectors
-      when :nothing
-        @broadcaster = StimulusReflex::NothingBroadcaster.new(self)
-      else
-        @broadcaster = StimulusReflex::SelectorBroadcaster.new(self) unless broadcaster.selector?
-        broadcaster.morphs << [selectors, html]
-      end
-    end
-
     def method_missing(name, *args, &blk)
-      morph :nothing
       super unless respond_to_missing?(name)
+
       state.each do |k, v|
         component.instance_variable_set(k, v)
       end
 
       component.send(name, *args, &blk)
       
-      refresh! unless @prevent_refresh
+      if @prevent_refresh
+        morph :nothing
+      else
+        default_morph      
+      end
     end
 
     def prevent_refresh!
@@ -182,11 +193,23 @@ module ViewComponentReflex
     end
 
     def set_state(new_state = {})
-      ViewComponentReflex::Engine.state_adapter.set_state(request, controller, element.dataset[:key], new_state)
+      state_adapter.set_state(request, controller, key, new_state)
+    end
+
+    def key
+      element.dataset[:key]
+    end
+
+    def state_adapter
+      ViewComponentReflex::Engine.state_adapter
     end
 
     def state
-      ViewComponentReflex::Engine.state_adapter.state(request, element.dataset[:key])
+      state_adapter.state(request, key)
+    end
+
+    def initial_state
+      state_adapter.state(request, "#{key}_initial")
     end
 
     def save_state

data/lib/view_component_reflex/state_adapter/session.rb

@@ -9,6 +9,7 @@ module ViewComponentReflex
         new_state.each do |k, v|
           state(request, key)[k] = v
         end
+        p new_state
         store = request.session.instance_variable_get("@by")
         store.commit_session request, controller.response
       end
@@ -18,6 +19,7 @@ module ViewComponentReflex
         new_state.each do |k, v|
           request.session[key][k] = v
         end
+        p new_state
       end
 
       def self.wrap_write_async

data/lib/view_component_reflex/version.rb

@@ -1,3 +1,3 @@
 module ViewComponentReflex
-  VERSION = '3.1.5'
+  VERSION = '3.1.10'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: view_component_reflex
 version: !ruby/object:Gem::Version
-  version: 3.1.5
+  version: 3.1.10
 platform: ruby
 authors:
 - Joshua LeBlanc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-29 00:00:00.000000000 Z
+date: 2021-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -50,14 +50,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.28.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.28.0
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement