view_component 2.47.0 → 2.48.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c94e6fdeccccb7180494d100eef8c78a8dc8e3e9bc30e5647017b440788c6eb
-  data.tar.gz: c3836df9c15e8039ba673895ad5a9544aebe55325f9fbd418b96f4bb4238b8d6
+  metadata.gz: 3abce9e08dcb71614178ed68b9c8443211ff78790911057752d8fda6fbb194b7
+  data.tar.gz: 3ac0a8ff1d7cafd2eb12e04a24f1d8f3362675833e4fda780bc238b8fc5ed41f
 SHA512:
-  metadata.gz: 644f964985c8418371fdcac45e1e7b1771d89ee65ed8a692ec51c50cbed8a578937ba1bb7b9bf1d3b3e3729bd418df5fb2e9d1aee3bbb8215bfbed4504457074
-  data.tar.gz: c4df9e14be4dc20e529beb794c96baf424087f7e36423b301402325e1b1ce0ae8ab575fe1b69334b3e27b5fa9e2fee227e9bd20e0e99fe5f384f2f535b43f80c
+  metadata.gz: dba31d0d37e73b7a1e26863d167f5d1eaefb8c5f53d9f542e47667090802b5f53b5c6b9c4b0a9188bf76f42d375c5d7a2cd3024065620aa9ff40b15911706cb8
+  data.tar.gz: f329872b25f36fe8fa0d018c2a5847ed0b6e47b095d16a27e89d2b0c7180f938d09730d0868405483a8e7c4fe4075afed56927a018b7a273ac4c94950d174fac

data/docs/CHANGELOG.md

@@ -7,6 +7,60 @@ title: Changelog
 
 ## main
 
+## 2.48.0
+
+* Correct path in example test command in Contributing docs.
+
+    *Mark Wilkinson*
+
+* Update link to GOV.UK Components library in the resources list.
+
+    *Peter Yates*
+
+* Add Lookbook to Resources docs page.
+
+    *Mark Perkins*
+
+* Add blocking compiler mode for use in Rails development and testing modes, improving thread safety.
+
+    *Horia Radu*
+
+* Add generators to support `tailwindcss-rails`.
+
+    *Dino Maric*, *Hans Lemuet*
+
+* Add a namespaced component example to docs.
+
+    *Hans Lemuet*
+
+* Setup `Appraisal` to add flexibility when testing ViewComponent against multiple Rails versions.
+
+    *Hans Lemuet*
+
+* Return correct values for `request.path` and `request.query_string` methods when using the `with_request_url` test helper.
+
+    *Vasiliy Matyushin*
+
+* Improve style in generators docs.
+
+    *Hans Lemuet*
+
+* Correctly type Ruby version strings and update Rails versions used in CI configuration.
+
+    *Hans Lemuet*
+
+* Make `ViewComponent::Collection` act like a collection of view components.
+
+    *Sammy Henningsson*
+
+* Update `@param` of `#render_inline` to include `ViewComponent::Collection`.
+
+    *Yutaka Kamei*
+
+* Add Wecasa to users list.
+
+    *Mohamed Ziata*
+
 ## 2.47.0
 
 * Display preview source on previews that exclusively use templates.
@@ -17,7 +71,7 @@ title: Changelog
 
     *Simon Fish*
 
-* Add WEBrick as a depenency to the docs application.
+* Add WEBrick as a depenency to the application.
 
     *Connor McQuillan*
 
@@ -79,6 +133,10 @@ title: Changelog
 
     *Bob Maerten*
 
+* Add config option `config.view_component.generate_sidecar` to always generate in the sidecar directory.
+
+    *Gleydson Tavares*
+
 ## 2.46.0
 
 * Add thread safety to the compiler.

data/lib/rails/generators/abstract_generator.rb

@@ -15,7 +15,7 @@ module ViewComponent
     end
 
     def destination_directory
-      if options["sidecar"]
+      if sidecar?
         File.join(component_path, class_path, destination_file_name)
       else
         File.join(component_path, class_path)
@@ -42,5 +42,9 @@ module ViewComponent
           gsub("/", "--")
       end
     end
+
+    def sidecar?
+      options["sidecar"] || ViewComponent::Base.generate_sidecar
+    end
   end
 end

data/lib/rails/generators/locale/component_generator.rb

@@ -35,7 +35,7 @@ module Locale
 
       def destination(locale = nil)
         extention = ".#{locale}" if locale
-        if options["sidecar"]
+        if sidecar?
           File.join(component_path, class_path, "#{file_name}_component", "#{file_name}_component#{extention}.yml")
         else
           File.join(component_path, class_path, "#{file_name}_component#{extention}.yml")

data/lib/rails/generators/stimulus/component_generator.rb

@@ -23,7 +23,7 @@ module Stimulus
       private
 
       def destination
-        if options["sidecar"]
+        if sidecar?
           File.join(component_path, class_path, "#{file_name}_component", "#{file_name}_component_controller.js")
         else
           File.join(component_path, class_path, "#{file_name}_component_controller.js")

data/lib/rails/generators/tailwindcss/component_generator.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+require "rails/generators/erb/component_generator"
+
+module Tailwindcss
+  module Generators
+    class ComponentGenerator < Erb::Generators::ComponentGenerator
+      source_root File.expand_path("templates", __dir__)
+    end
+  end
+end

data/lib/rails/generators/tailwindcss/templates/component.html.erb.tt

@@ -0,0 +1 @@
+<div<%= data_attributes %>>Add <%= class_name %> template here</div>

data/lib/view_component/base.rb

@@ -303,6 +303,7 @@ module ViewComponent
     #     config.view_component.view_component_path = "app/my_components"
     #
     # Defaults to `app/components`.
+    #
     mattr_accessor :view_component_path, instance_writer: false, default: "app/components"
 
     # Parent class for generated components
@@ -310,8 +311,16 @@ module ViewComponent
     #     config.view_component.component_parent_class = "MyBaseComponent"
     #
     # Defaults to "ApplicationComponent" if defined, "ViewComponent::Base" otherwise.
-    mattr_accessor :component_parent_class,
-                   instance_writer: false
+    #
+    mattr_accessor :component_parent_class, instance_writer: false
+
+    # Always generate a component with a sidecar directory:
+    #
+    #     config.view_component.generate_sidecar = true
+    #
+    # Defaults to `false`.
+    #
+    mattr_accessor :generate_sidecar, instance_writer: false, default: false
 
     class << self
       # @private

data/lib/view_component/collection.rb

@@ -4,20 +4,34 @@ require "action_view/renderer/collection_renderer" if Rails.version.to_f >= 6.1
 
 module ViewComponent
   class Collection
+    include Enumerable
     attr_reader :component
 
     delegate :format, to: :component
+    delegate :size, to: :@collection
 
     def render_in(view_context, &block)
+      components.map do |component|
+        component.render_in(view_context, &block)
+      end.join.html_safe # rubocop:disable Rails/OutputSafety
+    end
+
+    def components
+      return @components if defined? @components
+
       iterator = ActionView::PartialIteration.new(@collection.size)
 
       component.validate_collection_parameter!(validate_default: true)
 
-      @collection.map do |item|
-        content = component.new(**component_options(item, iterator)).render_in(view_context, &block)
-        iterator.iterate!
-        content
-      end.join.html_safe # rubocop:disable Rails/OutputSafety
+      @components = @collection.map do |item|
+        component.new(**component_options(item, iterator)).tap do |component|
+          iterator.iterate!
+        end
+      end
+    end
+
+    def each(&block)
+      components.each(&block)
     end
 
     private
@@ -42,7 +56,7 @@ module ViewComponent
     def component_options(item, iterator)
       item_options = { component.collection_parameter => item }
       item_options[component.collection_counter_parameter] = iterator.index + 1 if component.counter_argument_present?
-      item_options[component.collection_iteration_parameter] = iterator if component.iteration_argument_present?
+      item_options[component.collection_iteration_parameter] = iterator.dup if component.iteration_argument_present?
 
       @options.merge(item_options)
     end

data/lib/view_component/compiler.rb

@@ -5,6 +5,15 @@ module ViewComponent
     # Lock required to be obtained before compiling the component
     attr_reader :__vc_compiler_lock
 
+    # Compiler mode. Can be either:
+    # * development (a blocking mode which ensures thread safety when redefining the `call` method for components,
+    #                default in Rails development and test mode)
+    # * production (a non-blocking mode, default in Rails production mode)
+    DEVELOPMENT_MODE = :development
+    PRODUCTION_MODE = :production
+
+    class_attribute :mode, default: PRODUCTION_MODE
+
     def initialize(component_class)
       @component_class = component_class
       @__vc_compiler_lock = Monitor.new
@@ -14,10 +23,14 @@ module ViewComponent
       CompileCache.compiled?(component_class)
     end
 
+    def development?
+      self.class.mode == DEVELOPMENT_MODE
+    end
+
     def compile(raise_errors: false)
       return if compiled?
 
-      __vc_compiler_lock.synchronize do
+      with_lock do
         CompileCache.invalidate_class!(component_class)
 
         subclass_instance_methods = component_class.instance_methods(false)
@@ -57,10 +70,10 @@ module ViewComponent
           end
 
           component_class.class_eval <<-RUBY, template[:path], -1
-            def #{method_name}
-              @output_buffer = ActionView::OutputBuffer.new
-              #{compiled_template(template[:path])}
-            end
+          def #{method_name}
+            @output_buffer = ActionView::OutputBuffer.new
+            #{compiled_template(template[:path])}
+          end
           RUBY
         end
 
@@ -72,6 +85,14 @@ module ViewComponent
       end
     end
 
+    def with_lock(&block)
+      if development?
+        __vc_compiler_lock.synchronize(&block)
+      else
+        block.call
+      end
+    end
+
     private
 
     attr_reader :component_class
@@ -85,16 +106,30 @@ module ViewComponent
         "elsif variant.to_sym == :#{variant}\n    #{call_method_name(variant)}"
       end.join("\n")
 
-      component_class.class_eval <<-RUBY, __FILE__, __LINE__ + 1
+      body = <<-RUBY
+        if variant.nil?
+          call
+        #{variant_elsifs}
+        else
+          call
+        end
+      RUBY
+
+      if development?
+        component_class.class_eval <<-RUBY, __FILE__, __LINE__ + 1
         def render_template_for(variant = nil)
-          if variant.nil?
-            call
-          #{variant_elsifs}
-          else
-            call
+          self.class.compiler.with_lock do
+            #{body}
           end
         end
-      RUBY
+        RUBY
+      else
+        component_class.class_eval <<-RUBY, __FILE__, __LINE__ + 1
+        def render_template_for(variant = nil)
+          #{body}
+        end
+        RUBY
+      end
     end
 
     def template_errors

data/lib/view_component/engine.rb

@@ -115,6 +115,14 @@ module ViewComponent
       end
     end
 
+    initializer "compiler mode" do |app|
+      ViewComponent::Compiler.mode = if Rails.env.development? || Rails.env.test?
+                                       ViewComponent::Compiler::DEVELOPMENT_MODE
+                                     else
+                                       ViewComponent::Compiler::PRODUCTION_MODE
+                                     end
+    end
+
     config.after_initialize do |app|
       options = app.config.view_component
 

data/lib/view_component/test_helpers.rb

@@ -38,7 +38,7 @@ module ViewComponent
     # assert_text("Hello, World!")
     # ```
     #
-    # @param component [ViewComponent::Base] The instance of the component to be rendered.
+    # @param component [ViewComponent::Base, ViewComponent::Collection] The instance of the component to be rendered.
     # @return [Nokogiri::HTML]
     def render_inline(component, **args, &block)
       @rendered_component =
@@ -113,16 +113,22 @@ module ViewComponent
     #
     # @param path [String] The path to set for the current request.
     def with_request_url(path)
+      old_request_path_info = request.path_info
       old_request_path_parameters = request.path_parameters
       old_request_query_parameters = request.query_parameters
+      old_request_query_string = request.query_string
       old_controller = defined?(@controller) && @controller
 
+      request.path_info = path
       request.path_parameters = Rails.application.routes.recognize_path(path)
       request.set_header("action_dispatch.request.query_parameters", Rack::Utils.parse_query(path.split("?")[1]))
+      request.set_header(Rack::QUERY_STRING, path.split("?")[1])
       yield
     ensure
+      request.path_info = old_request_path_info
       request.path_parameters = old_request_path_parameters
       request.set_header("action_dispatch.request.query_parameters", old_request_query_parameters)
+      request.set_header(Rack::QUERY_STRING, old_request_query_string)
       @controller = old_controller
     end
 

data/lib/view_component/version.rb

@@ -3,7 +3,7 @@
 module ViewComponent
   module VERSION
     MAJOR = 2
-    MINOR = 47
+    MINOR = 48
     PATCH = 0
 
     STRING = [MAJOR, MINOR, PATCH].join(".")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: view_component
 version: !ruby/object:Gem::Version
-  version: 2.47.0
+  version: 2.48.0
 platform: ruby
 authors:
 - GitHub Open Source
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-20 00:00:00.000000000 Z
+date: 2022-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -44,6 +44,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
   requirement: !ruby/object:Gem::Requirement
@@ -268,7 +282,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - opensource+view_component@github.com
 executables: []
@@ -304,6 +318,8 @@ files:
 - lib/rails/generators/slim/templates/component.html.slim.tt
 - lib/rails/generators/stimulus/component_generator.rb
 - lib/rails/generators/stimulus/templates/component_controller.js.tt
+- lib/rails/generators/tailwindcss/component_generator.rb
+- lib/rails/generators/tailwindcss/templates/component.html.erb.tt
 - lib/rails/generators/test_unit/component_generator.rb
 - lib/rails/generators/test_unit/templates/component_test.rb.tt
 - lib/view_component.rb
@@ -342,7 +358,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -357,8 +373,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: View components for Rails
 test_files: []