ilex 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4f98672db74bb9e94ee80eb636e5878900d5a348fd4879633d5e44dcbc289053
-  data.tar.gz: 6cccb6d5f4a506995fe4549f249ba0cd5150d8e3b26955985f1a5b25d9d4b359
+  metadata.gz: e1cef48991c440c63571d00490a85cfdd407e6b7c2efbdd560660948ed47bd57
+  data.tar.gz: 925406a78b7b417f82b979294b66254847a7d352cecf4e0ab9b739fdb82eefd9
 SHA512:
-  metadata.gz: 97243cd6826be3e81e93dd847adcbc50a36045865bc67c325524a4945dc7690479a9a15037b061edfd7b2c18c377ebda444d80a44b009a8afca2c2b78a022d81
-  data.tar.gz: db12e3bb02e3e7277bbf2a15e40839616b64b283733fe9ba1cf17bf28029383ca9b2c246aaa7eef2ddaa56a1cd5f0c1009605d040853133416e3d64732f92a87
+  metadata.gz: 7e56764de81c18320e29c3753c959ad146aca5b5ccdfc7aedb1c5ddbddaece23be2010b1c58b905332ca72597f9cec2b974910e009b93ed423bdd960b1ed787c
+  data.tar.gz: 6992772c2f5ac33665a01c182e11f4f3e7655f6d5cf9625c8c25fc7e7c11784922364e2849ab275ad7ed7c6031b867a461ff6f2eb61904c65129a1147c3170b9

data/.versionrc.json

@@ -1,7 +1,7 @@
-{
-  "skip": {
-    "tag": true
-  },
-  "NAME": "Ilex",
-  "VERSION": "0.1.6"
-}
+{
+  "skip": {
+    "tag": true
+  },
+  "NAME": "Ilex",
+  "VERSION": "0.1.6"
+}

data/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [0.1.7](https://github.com/joshleblanc/ilex/compare/v0.1.6...v0.1.7) (2021-06-16)
+
+
+### Bug Fixes
+
+* try and convert component arrays into strings ([13e0438](https://github.com/joshleblanc/ilex/commit/13e0438d1da6ebd0bc0b46c9ca7cc6828bd99a7f))
+
 ### [0.1.6](https://github.com/joshleblanc/cedar/compare/v0.1.5...v0.1.6) (2021-04-18)
 
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ilex (0.1.6)
+    ilex (0.1.7)
       arbre
       rails (>= 5.2, < 7.0)
 
@@ -91,6 +91,8 @@ GEM
     nio4r (2.5.7)
     nokogiri (1.11.2-x64-mingw32)
       racc (~> 1.4)
+    nokogiri (1.11.2-x86_64-linux)
+      racc (~> 1.4)
     parallel (1.20.1)
     parser (3.0.0.0)
       ast (~> 2.4.1)
@@ -164,6 +166,7 @@ GEM
 
 PLATFORMS
   x64-mingw32
+  x86_64-linux
 
 DEPENDENCIES
   ilex!

data/lib/ilex.rb

@@ -7,6 +7,7 @@ require_relative "ilex/component_wardens"
 require_relative "ilex/context"
 require_relative "ilex/component"
 require_relative "ilex/arbre_ext/element"
+require_relative "ilex/arbre_ext/element/builder_methods"
 require_relative "ilex/rails_ext/action_view/base"
 require_relative "ilex/engine"
 

data/lib/ilex/arbre_ext/element.rb

@@ -1,27 +1,27 @@
-module Ilex
-  module ArbreExt
-    module Element
-      ruby2_keywords def method_missing(name, *args, &block)
-        if current_arbre_element.respond_to?(name)
-          current_arbre_element.send name, *args, &block
-        elsif assigns && assigns.has_key?(name)
-          assigns[name]
-        elsif helpers.respond_to?(name)
-          helper_capture(name, *args, &block)
-        else
-          super
-        end
-      end
-
-      # The helper might have a block that builds Arbre elements
-      # which will be rendered (#to_s) inside ActionView::Base#capture.
-      # We do not want such elements added to self, so we push a dummy
-      # current_arbre_element.
-      def helper_capture(name, *args, &block)
-        s = ""
-        within(Element.new) { s = helpers.send(name, *args, &block) }
-        s.is_a?(Element) ? s.to_s : s
-      end
-    end
-  end
-end
+module Ilex
+  module ArbreExt
+    module Element
+      ruby2_keywords def method_missing(name, *args, &block)
+        if current_arbre_element.respond_to?(name)
+          current_arbre_element.send name, *args, &block
+        elsif assigns && assigns.has_key?(name)
+          assigns[name]
+        elsif helpers.respond_to?(name)
+          helper_capture(name, *args, &block)
+        else
+          super
+        end
+      end
+
+      # The helper might have a block that builds Arbre elements
+      # which will be rendered (#to_s) inside ActionView::Base#capture.
+      # We do not want such elements added to self, so we push a dummy
+      # current_arbre_element.
+      def helper_capture(name, *args, &block)
+        s = ""
+        within(Element.new) { s = helpers.send(name, *args, &block) }
+        s.is_a?(Element) ? s.to_s : s
+      end
+    end
+  end
+end

data/lib/ilex/arbre_ext/element/builder_methods.rb

@@ -0,0 +1,20 @@
+module Ilex
+  module ArbreExt
+    module Element
+      module BuilderMethods
+        private
+
+        def append_return_block(tag)
+          return nil if current_arbre_element.children?
+          return unless appendable_tag?(tag)
+
+          current_arbre_element << if tag.is_a? Array
+                                     Arbre::HTML::TextNode.from_string(tag.join.html_safe)
+                                   else
+                                     Arbre::HTML::TextNode.from_string(tag.to_s)
+                                   end
+        end
+      end
+    end
+  end
+end

data/lib/ilex/component.rb

@@ -1,49 +1,49 @@
-# frozen_string_literal: true
-
-module Ilex
-
-  ##
-  # InlineRender provides a `render` dsl to your components
-  # which will render any arbre tree within.
-  #
-  # It will also convert snake case component names to component instances
-  # for example, this would be equivalent to `ButtonComponent.new(label: "Test")`
-  #
-  # ```ruby
-  # render do
-  #   div do
-  #     button label: "Test"
-  #   end
-  # end
-  # ```
-  module Component
-    include Arbre::HTML
-
-    def render(*args, &blk)
-      define_method :call do
-        ctx = Context.new(self)
-        ctx.instance_eval(&blk).to_s
-      end
-    end
-
-    def find_component(name, base_module = nil)
-      target = if base_module
-        base_module
-      else
-        self
-      end
-
-      if target.const_defined?(name)
-        target.const_get(name)
-      else
-        if target.superclass.respond_to?(:find_component)
-          target.superclass.find_component(name)
-        else
-          adjusted_name = name.chomp("Component").underscore
-          raise(NameError, "undefined local variable or method `#{adjusted_name}` for #{self}")
-        end
-
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Ilex
+
+  ##
+  # InlineRender provides a `render` dsl to your components
+  # which will render any arbre tree within.
+  #
+  # It will also convert snake case component names to component instances
+  # for example, this would be equivalent to `ButtonComponent.new(label: "Test")`
+  #
+  # ```ruby
+  # render do
+  #   div do
+  #     button label: "Test"
+  #   end
+  # end
+  # ```
+  module Component
+    include Arbre::HTML
+
+    def render(*args, &blk)
+      define_method :call do
+        ctx = Context.new(self)
+        ctx.instance_eval(&blk).to_s
+      end
+    end
+
+    def find_component(name, base_module = nil)
+      target = if base_module
+        base_module
+      else
+        self
+      end
+
+      if target.const_defined?(name)
+        target.const_get(name)
+      else
+        if target.superclass.respond_to?(:find_component)
+          target.superclass.find_component(name)
+        else
+          adjusted_name = name.chomp("Component").underscore
+          raise(NameError, "undefined local variable or method `#{adjusted_name}` for #{self}")
+        end
+
+      end
+    end
+  end
+end

data/lib/ilex/component_warden.rb

@@ -1,65 +1,65 @@
-# frozen_string_literal: true
-
-module Ilex
-  ##
-  # A ComponentWarden is responsible for locating potential components
-  # given a snake_case representation of that component.
-  #
-  # If a component exists, the warden is also responsible for calling the correct
-  # instantiation method, depending if the component is a collection or not
-  class ComponentWarden
-    def initialize(component, string)
-      @component = component
-      @input = string.to_s
-
-      @collection = @input.end_with? "_collection"
-      @nested = @input.include? "__"
-
-      process_input!(@input)
-    end
-
-    def collection?
-      @collection
-    end
-
-    def nested?
-      @nested
-    end
-
-    def component_name
-      "#{@input}_component".camelize
-    end
-
-    def component_class
-      @component_class ||= @component.class.find_component(component_name, @base_module)
-    end
-
-    def exists?
-      component_class.present?
-    end
-
-    def new(*args, &block)
-      return nil unless exists?
-
-      if collection?
-        component_class.with_collection(*args)
-      else
-        component_class.new(*args)
-      end
-    end
-
-    private
-
-    def process_input!(input)
-      @input.chomp! "_collection"
-      @input.chomp! "_component"
-
-      if nested?
-        parts = @input.split("__")
-        @input = parts.pop
-
-        @base_module = parts.map(&:camelize).join("::").constantize
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Ilex
+  ##
+  # A ComponentWarden is responsible for locating potential components
+  # given a snake_case representation of that component.
+  #
+  # If a component exists, the warden is also responsible for calling the correct
+  # instantiation method, depending if the component is a collection or not
+  class ComponentWarden
+    def initialize(component, string)
+      @component = component
+      @input = string.to_s
+
+      @collection = @input.end_with? "_collection"
+      @nested = @input.include? "__"
+
+      process_input!(@input)
+    end
+
+    def collection?
+      @collection
+    end
+
+    def nested?
+      @nested
+    end
+
+    def component_name
+      "#{@input}_component".camelize
+    end
+
+    def component_class
+      @component_class ||= @component.class.find_component(component_name, @base_module)
+    end
+
+    def exists?
+      component_class.present?
+    end
+
+    def new(*args, &block)
+      return nil unless exists?
+
+      if collection?
+        component_class.with_collection(*args)
+      else
+        component_class.new(*args)
+      end
+    end
+
+    private
+
+    def process_input!(input)
+      @input.chomp! "_collection"
+      @input.chomp! "_component"
+
+      if nested?
+        parts = @input.split("__")
+        @input = parts.pop
+
+        @base_module = parts.map(&:camelize).join("::").constantize
+      end
+    end
+  end
+end

data/lib/ilex/component_wardens.rb

@@ -1,18 +1,18 @@
-# frozen_string_literal: true
-
-module Ilex
-  ##
-  # Thin wrapper around a hash of component wardens
-  # such that the default is always an instance of ComponentWarden
-  class ComponentWardens
-    def initialize(component)
-      @wardens = Hash.new do |hash, key|
-        hash[key] = ComponentWarden.new(component, key)
-      end
-    end
-
-    def [](key)
-      @wardens[key]
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Ilex
+  ##
+  # Thin wrapper around a hash of component wardens
+  # such that the default is always an instance of ComponentWarden
+  class ComponentWardens
+    def initialize(component)
+      @wardens = Hash.new do |hash, key|
+        hash[key] = ComponentWarden.new(component, key)
+      end
+    end
+
+    def [](key)
+      @wardens[key]
+    end
+  end
+end

data/lib/ilex/context.rb

@@ -1,62 +1,62 @@
-# frozen_string_literal: true
-
-module Ilex
-  ##
-  # Inject a component's instance variables into arbre's default context
-  # as well as provide implicit handling of snake-cased component names as
-  # tags
-  class Context < Arbre::Context
-
-    def initialize(component, &blk)
-      @component = component
-
-
-      # Copy all of the instance variables from the component to the context,
-      # so we have access to them when rendering
-      @component.instance_variables.each do |iv|
-        instance_variable_set(iv, @component.instance_variable_get(iv))
-      end
-
-      super({}, component, &blk)
-    end
-
-    def content
-      @component.send :content
-    end
-
-    def component_wardens
-      @component_wardens ||= ComponentWardens.new(@component)
-    end
-
-    # This is overriding arbre::rails::rendering
-    # It performs the same actions, but returns html_safe on a passed block
-    #
-    # Not 100% sure this is needed, but view_components won't render their
-    # contents without it, when rendered in an arbre tree
-    def render(*args)
-      rendered = helpers.render(*args) do
-        yield.html_safe if block_given?
-      end
-      case rendered
-      when Arbre::Context
-        current_arbre_element.add_child rendered
-      else
-        text_node rendered
-      end
-    end
-
-    def respond_to_missing?(method, include_all)
-      @component.respond_to?(method) || component_wardens[method].exists? || super
-    end
-
-    def method_missing(method, *args, &content_block)
-      if @component.respond_to?(method)
-        @component.send(method, *args, &content_block)
-      elsif component_wardens[method].exists?
-        render component_wardens[method].new(*args), &content_block
-      else
-        super
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Ilex
+  ##
+  # Inject a component's instance variables into arbre's default context
+  # as well as provide implicit handling of snake-cased component names as
+  # tags
+  class Context < Arbre::Context
+
+    def initialize(component, &blk)
+      @component = component
+
+
+      # Copy all of the instance variables from the component to the context,
+      # so we have access to them when rendering
+      @component.instance_variables.each do |iv|
+        instance_variable_set(iv, @component.instance_variable_get(iv))
+      end
+
+      super({}, component, &blk)
+    end
+
+    def content
+      @component.send :content
+    end
+
+    def component_wardens
+      @component_wardens ||= ComponentWardens.new(@component)
+    end
+
+    # This is overriding arbre::rails::rendering
+    # It performs the same actions, but returns html_safe on a passed block
+    #
+    # Not 100% sure this is needed, but view_components won't render their
+    # contents without it, when rendered in an arbre tree
+    def render(*args)
+      rendered = helpers.render(*args) do
+        yield.html_safe if block_given?
+      end
+      case rendered
+      when Arbre::Context
+        current_arbre_element.add_child rendered
+      else
+        text_node rendered
+      end
+    end
+
+    def respond_to_missing?(method, include_all)
+      @component.respond_to?(method) || component_wardens[method].exists? || super
+    end
+
+    def method_missing(method, *args, &content_block)
+      if @component.respond_to?(method)
+        @component.send(method, *args, &content_block)
+      elsif component_wardens[method].exists?
+        render component_wardens[method].new(*args), &content_block
+      else
+        super
+      end
+    end
+  end
+end

data/lib/ilex/engine.rb

@@ -1,9 +1,10 @@
-module Ilex
-  class Engine < ::Rails::Engine
-
-    config.to_prepare do
-      ActionView::Base.prepend RailsExt::ActionView::Base
-      Arbre::Element.prepend ArbreExt::Element
-    end
-  end
-end
+module Ilex
+  class Engine < ::Rails::Engine
+
+    config.to_prepare do
+      ActionView::Base.prepend RailsExt::ActionView::Base
+      Arbre::Element.prepend ArbreExt::Element
+      Arbre::Element::BuilderMethods.prepend ArbreExt::Element::BuilderMethods
+    end
+  end
+end

data/lib/ilex/rails_ext/action_view/base.rb

@@ -1,21 +1,21 @@
-module Ilex
-  module RailsExt
-    module ActionView
-      module Base
-        def capture(*args)
-          value = nil
-          buffer = with_output_buffer { value = yield(*args) }
-
-          # Override to handle Arbre elements inside helper blocks.
-          # See https://github.com/rails/rails/issues/17661
-          # and https://github.com/rails/rails/pull/18024#commitcomment-8975180
-          value = value.to_s if value.is_a?(Arbre::Element)
-
-          if (string = buffer.presence || value) && string.is_a?(String)
-            ERB::Util.html_escape string
-          end
-        end
-      end
-    end
-  end
-end
+module Ilex
+  module RailsExt
+    module ActionView
+      module Base
+        def capture(*args)
+          value = nil
+          buffer = with_output_buffer { value = yield(*args) }
+
+          # Override to handle Arbre elements inside helper blocks.
+          # See https://github.com/rails/rails/issues/17661
+          # and https://github.com/rails/rails/pull/18024#commitcomment-8975180
+          value = value.to_s if value.is_a?(Arbre::Element)
+
+          if (string = buffer.presence || value) && string.is_a?(String)
+            ERB::Util.html_escape string
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ilex/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Ilex
-  VERSION = "0.1.6"
+  VERSION = "0.1.7"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ilex
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Joshua LeBlanc
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-04-18 00:00:00.000000000 Z
+date: 2021-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: arbre
@@ -67,6 +67,7 @@ files:
 - cedar.gemspec
 - lib/ilex.rb
 - lib/ilex/arbre_ext/element.rb
+- lib/ilex/arbre_ext/element/builder_methods.rb
 - lib/ilex/component.rb
 - lib/ilex/component_warden.rb
 - lib/ilex/component_wardens.rb
@@ -82,7 +83,7 @@ metadata:
   homepage_uri: https://github.com/joshleblanc/ilex
   source_code_uri: https://github.com/joshleblanc/ilex
   changelog_uri: https://github.com/joshleblanc/ilex
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -97,8 +98,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key:
+rubygems_version: 3.2.15
+signing_key: 
 specification_version: 4
 summary: Provides inline rendering to view components
 test_files: []