view_component_reflex 3.1.4 → 3.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e1c5a6bb6d424caf8b1840123432a3c50262739780a9c653c73d3bb4d7154906
-  data.tar.gz: 9314ec969dd8c0d31ac5751a40a8fa58cdc1f88fac3139775803b15a6284ae30
+  metadata.gz: 8df0782e33267ea91ff1e99610107ee004cdec87947087750227907af88a3674
+  data.tar.gz: 5f488b266c6ade0cc81317da9ed0cb3d08cb8680654aef045348b3809997c9f4
 SHA512:
-  metadata.gz: f2b2dbeca7e309d7611271a81d39bbda1d62c8d4680c81376ad149818bb6b866e5cf8c2d97c378ed5ca2b31d6e1dc05bd2accacc4e56f6d46463ef24c53f95d1
-  data.tar.gz: a24726fe210f24efe596f82931b74d99fb60dcd89e7c3ce4d4b71dbc8ca6974018bd92c954b3e6942ca6e5b282e469458be8eeaf2de2f978d856fe654e10c5d4
+  metadata.gz: 281166ac7c1e2589d9b97733ad3edca6986720c87b239dd779c55529644bff5f8fad352a46abd43dc53c2e46947525d3517335a0621c8fa6478ad23767ef2b31
+  data.tar.gz: dece5fcacf66ee99a5fb539b42bb37cd71904435851ac71e8804b5ed75f0d67b8473b4a7e25e288dfbf81299a9c9f1834c418241b3a3055042136d2308f3f4ff

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
 
@@ -14,24 +21,21 @@ module ViewComponentReflex
       if primary_selector
         prevent_refresh!
 
-        controller.process(params[:action])
-        document = Nokogiri::HTML(controller.response.body)
         [primary_selector, *rest].each do |s|
-          html = document.css(s)
+          html = controller_document.css(s)
           if html.present?
             CableReady::Channels.instance[stream].morph(
               selector: s,
               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 +46,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 +113,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 +192,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.4'
+  VERSION = '3.1.9'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: view_component_reflex
 version: !ruby/object:Gem::Version
-  version: 3.1.4
+  version: 3.1.9
 platform: ruby
 authors:
 - Joshua LeBlanc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-28 00:00:00.000000000 Z
+date: 2021-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -36,28 +36,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: 3.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: 3.4.1
 - !ruby/object:Gem::Dependency
   name: view_component
   requirement: !ruby/object:Gem::Requirement
     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