low_node 0.3.1 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 75eb52cd5e606c086a9672c8675710979878a5b240428a619b3ab454ec8534f0
-  data.tar.gz: 476ce5d261f9034066b21223a31be47d137cf64d6ec3921605bcf5060b594ea4
+  metadata.gz: 84888260b641cba4399689afd91aaddc305d704ab4711c15f20520d32915a9bc
+  data.tar.gz: be064e6d1be460ab4d4761443112a4e32d5f21364be923d3e4a16b961ffe649d
 SHA512:
-  metadata.gz: 9712d807495ee5299d83fc6fcd17355d009978ba9c6bbc8709da9098decb858ff4d52e035505213bc28c745561537a161b24412d6175552618cd0c8ef1f96d4e
-  data.tar.gz: 889af9b4f7caca1f61a559743f3a2ba2792d177320a37d9d9a2a8258af5713d4ca088cdf07991e8eb11415bb85defa2a839fa438450f331be4b33bd85fbb0c75
+  metadata.gz: 051e636700adbb503c89d6e6e0e2187db8ce716a27a7c37765339d77470ef9cad63402b2a7da9404e6aa52377f2e3d015cf5e1964a46d1dcca0d68b314116eef
+  data.tar.gz: a2915fa0fbd329b523cf407407c45f715da8ea71de50d4a6c1af9facaddc4560357ea908555532c8aa486f983ac9f0ad5f4f89489fd7cfba1273cdf860281d83

data/lib/low_node.rb

@@ -17,17 +17,18 @@ class LowNode
 
   attr_reader :event
 
-  def initialize(event:)
+  def initialize(event: nil)
     @event = event
   end
 
+  # TODO: Rename "handle" method/action to more specific "route" and maybe split out into module too.
   def handle(event:)
     nil
   end
 
   class << self
     def handle(event:)
-      self.new(event:).handle(event:)
+      new(event:).handle(event:)
     end
 
     def inherited(child)

data/lib/templates/renderer.rb

@@ -4,26 +4,22 @@ require_relative 'template'
 
 module Low
   module Templates
+    # Use the Method Factory pattern to .render -> .new -> #initialize -> #render a low node.
     module Renderer
-      def render(event: nil)
-        nil
-      end
-
       # When render() contains RBX/Antlers then LowLoad builds a template to render with instead.
-      def render_template(event:, parent_binding: nil, slot_node: nil, props: {})
+      def render_template(parent_binding: nil, slot_node: nil, props: {}, **kwargs)
         template = self.class.template
 
         # NOTE: We currently set local variables on this local binding, not the receiver's binding.
         # TODO: Should we create an "instance_binding" method on LowNode and use that binding instead?
         current_binding = binding
-        current_binding.local_variable_set(:event, event)
 
-        # Pass in props from parent component as keyword arguments.
         template.params.each do |param|
-          current_binding.local_variable_set(param, props[param]) if props[param]
+          variable = props[param] || kwargs[param] || raise(ArgumentError, "Missing argument: '#{param}'")
+          current_binding.local_variable_set(param, variable)
         end
 
-        template.engine.render(ast: template.ast, current_binding:, parent_binding:, slot_node:, namespace: template.namespace)
+        template.engine.render(ast: template.ast, current_binding:, parent_binding:, slot_node:)
       end
 
       def self.included(base)
@@ -31,28 +27,34 @@ module Low
       end
 
       module ClassMethods
+        # Accepts all arguments but only sends arguments required by the sub class.
+        def render(**)
+          # GOAL: Make return value configurable; ResponseEvent, Response, or body.
+          body = response_body(**)
+
+          return if body.nil? || body == ''
+
+          response = Low::Factories::ResponseFactory.html(body:)
+          Low::Events::ResponseEvent.new(response:).tap(&:branch)
+        end
+
+        # LowLoad hook.
         def template
           @template
         end
 
+        # LowLoad hook.
         def build_template(template:, params:, engine:, namespace:)
           @template = Template.new(template:, params:, engine:, namespace:)
         end
 
-        # TODO: Handle situation where node is tested in a unit test and args come in here; expose them to the template.
-        def render(event: nil)
-          node = self.new(event:)
-
-          # GOAL: Make return value configurable; ResponseEvent, Response, or body.
-          response = Low::Factories::ResponseFactory.html(body: response_body(node:, event:))
-          Low::Events::ResponseEvent.new(response:).tap { it.branch }
-        end
-
         private
 
-        def response_body(node:, event:)
-          return node.render_template(event:) if @template
-          return node.render(event:) if node.method(:render).arity > 0
+        def response_body(**kwargs)
+          node = new(**kwargs)
+
+          return node.render_template(**kwargs) if @template
+          return node.render(**kwargs) if node.method(:render).arity > 0
 
           node.render
         end

data/lib/templates/template.rb

@@ -11,7 +11,7 @@ module Low
         @namespace = namespace
 
         @engine = engine
-        @ast = engine.ast(template)
+        @ast = engine.ast(template:, namespace:)
       end
     end
   end

data/lib/version.rb

@@ -2,6 +2,6 @@
 
 module Low
   module Node
-    VERSION = '0.3.1'
+    VERSION = '0.5.0'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: low_node
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.5.0
 platform: ruby
 authors:
 - maedi
@@ -108,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.6
 specification_version: 4
 summary: Flexible building blocks
 test_files: []