hanami 2.1.0.beta2 → 2.1.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 06d7766d5bdb4f510945a2065cc15457acbf00f43970d691afda74219f0aeb02
-  data.tar.gz: 58935ca7f23c3d3612a91b277827ed941c30839d04c9bf577e35a095fa667f6f
+  metadata.gz: 3ba29263713547809e78df051e0138f202d947ce6e9cf2bc9d06c5bf79438cb7
+  data.tar.gz: 0677ae36e85bbb92ed1bdde4ae48e92ac062ccf89f6d77b86b85578fd8975a06
 SHA512:
-  metadata.gz: 50a785250f4cf249c5721dd37a5e5a7a240282e5d3820fe848a53c133b1727ea57e5beb351429e636bb4a7d89812bfc53794b5357865b134598a5d54e60666db
-  data.tar.gz: 677f40136e0477c57e493e41ed9a25666d9b53d6dd4ae4c23ee6e61d062f9537c39d86eb16d264f194eda4209e5eb0e38fa0554fd3efaf2d39ea23afa1ead27b
+  metadata.gz: f24d67421d88e019d3fdf1d282fd55cff3a303e59fa7195fc8b152a6b85d1990aafe1cf3c583eeb2b561cfcae4b76e4120ef9863f12cee424c8372526cfb178a
+  data.tar.gz: 8e9ab011e12426e7d2e03ed784858309f7ca33fa8aa84d23ae84cd7d9435563998f38cb9d53322e2631c017334465cec10e0b2f65a58d8dcb0290c360dff5b1c

data/CHANGELOG.md

@@ -2,26 +2,58 @@
 
 The web, with simplicity.
 
+## v2.1.0.rc1 - 2023-11-01
+
+### Added
+
+- [Aaron Moodie & Tim Riley] Render a welcome page when no routes are defined (#1353)
+- [Luca Guidi] Allow _Method Override_ by default, by mounting `Rack::MethodOverride` middleware. (#1344)
+
+### Fixed
+
+- [Luca Guidi] Ensure compatibility with Ruby Logger from stdlib (#1352)
+
+### Changed
+
+- [Philip Arndt] Add support for a slice's class definition file to exist inside either `config/slices/[slice_name].rb` or `slices/[slice_name]/config/slice.rb`, in that order of precedence, so that a slice can provide its own definition which the app, or the slice's parent slice if that exists, can override if needed.
+- [Tim Riley] Use `helpers` prefix to access helper methods from view parts (e.g. `helpers.format_number`)
+- [Tim Riley] Make it possible to directly initialize view parts to ease their unit tests (#1357)
+- [Tim Riley] Remove short names for assets helpers (#1356):
+  - Keep `javascript_tag` (remove `javascript` and `js` aliases)
+  - Keep `stylesheet_tag` (renamed from `stylesheet_link_tag`; remove `stylesheet` and `css` aliases)
+  - Keep `favicon_tag` (renamed from `favicon_link_tag`; remove `favicon` alias)
+  - Keep `image_tag` (remove `image` alias)
+  - Keep `video_tag` (remove `video` alias)
+  - Keep `audio_tag` (remove `audio` alias)
+
+
+## v2.1.0.beta2.1 - 2023-10-04
+
+### Added
+
+- [Tim Riley, Luca Guidi] Added assets helpers aliases (#1319, #1339):
+  - Added `#js` and `#javascript_tag` as alias for `#javascript`
+  - Added `#css` and `#stylesheet_link_tag` as alias for `#stylesheet`
+  - Added `#image_tag` as alias for `#image`
+  - Added `#favicon_link_tag` as alias for `#favicon`
+  - Added `#video_tag` as alias for `#video`
+  - Added `#audio_tag` as alias for `#audio`
+
 ## v2.1.0.beta2 - 2023-10-04
 
 ### Added
 
 - [Luca Guidi, Tim Riley] Reimplement assets integration (#1319, #1332, #1333, #1336)
 - [Tim Riley] Introduce `Hanami::Helpers::AssetsHelper`, automatically included in view templates, scopes and parts when hanami-assets is bundled (#1319)
-- [Tim Riley] Renamed assets helpers (#1319):
-  - Removed `#asset_path` in favour of `#asset_url` only
-  - Renamed `#javascript` to `#javascript_tag`, retaining `#js` alias
-  - Renamed `#stylesheet` to `#stylesheet_link_tag`, retaining `#css` alias
-  - Renamed `#image` to `#image_tag`
-  - Renamed `#favicon` to `#favicon_link_tag`, with `#favicon` retained as an alias
-  - Renamed `#video` to `#video_tag`
-  - Renamed `#audio` to `#audio_tag`
-
 
 ### Fixed
 
 - [Tim Riley] Return appropriate response statuses based on error type (#1330)
 
+### Changed
+
+- [Tim Riley] Removed `#asset_path` in favour of `#asset_url` only (#1319)
+
 ## v2.1.0.beta1 - 2023-06-29
 
 ### Added

data/hanami.gemspec

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 lib = File.expand_path("../lib", __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+$LOAD_PATH.prepend(lib) unless $LOAD_PATH.include?(lib)
 require "hanami/version"
 
 Gem::Specification.new do |spec|

data/lib/hanami/config/actions.rb

@@ -74,6 +74,20 @@ module Hanami
       # @since 2.0.0
       attr_accessor :content_security_policy
 
+      # @!attribute [rw] method_override
+      #   Sets or returns whether HTTP method override should be enabled for action classes.
+      #
+      #   Defaults to true. You can override this by explicitly setting a
+      #   true or false value.
+      #
+      #   When true, this will mount `Rack::MethodOverride` in the Rack middleware stack of the App.
+      #
+      #   @return [Boolean]
+      #
+      #   @api public
+      #   @since 2.1.0
+      setting :method_override, default: true
+
       # The following settings are for view and assets integration with actions, and are NOT
       # publicly released as of 2.0.0. We'll make full documentation available when these become
       # public in a subsequent release.

data/lib/hanami/extensions/action/slice_configured_action.rb

@@ -31,14 +31,14 @@ module Hanami
         # @see Hanami::Extensions::Action::InstanceMethods#initialize
         def define_new
           resolve_view = method(:resolve_paired_view)
-          resolve_view_context = method(:resolve_view_context)
+          view_context_class = method(:view_context_class)
           resolve_routes = method(:resolve_routes)
           resolve_rack_monitor = method(:resolve_rack_monitor)
 
           define_method(:new) do |**kwargs|
             super(
               view: kwargs.fetch(:view) { resolve_view.(self) },
-              view_context: kwargs.fetch(:view_context) { resolve_view_context.() },
+              view_context_class: kwargs.fetch(:view_context_class) { view_context_class.() },
               routes: kwargs.fetch(:routes) { resolve_routes.() },
               rack_monitor: kwargs.fetch(:rack_monitor) { resolve_rack_monitor.() },
               **kwargs,
@@ -128,9 +128,9 @@ module Hanami
           nil
         end
 
-        def resolve_view_context
+        def view_context_class
           if Hanami.bundled?("hanami-view")
-            return Extensions::View::Context.context_class(slice).new
+            return Extensions::View::Context.context_class(slice)
           end
 
           # If hanami-view isn't bundled, try and find a possible third party context class with the
@@ -139,7 +139,7 @@ module Hanami
             views_namespace = slice.namespace.const_get(:Views)
 
             if views_namespace.const_defined?(:Context)
-              views_namespace.const_get(:Context).new
+              views_namespace.const_get(:Context)
             end
           end
         end

data/lib/hanami/extensions/action.rb

@@ -40,7 +40,7 @@ module Hanami
         attr_reader :view
 
         # @api private
-        attr_reader :view_context
+        attr_reader :view_context_class
 
         # Returns the app or slice's {Hanami::Slice::RoutesHelper RoutesHelper} for use within
         # action instance methods.
@@ -70,9 +70,9 @@ module Hanami
         #
         #   @api public
         #   @since 2.0.0
-        def initialize(view: nil, view_context: nil, rack_monitor: nil, routes: nil, **kwargs)
+        def initialize(view: nil, view_context_class: nil, rack_monitor: nil, routes: nil, **kwargs)
           @view = view
-          @view_context = view_context
+          @view_context_class = view_context_class
           @routes = routes
           @rack_monitor = rack_monitor
 
@@ -104,7 +104,7 @@ module Hanami
 
         # @api private
         def view_options(request, response)
-          {context: view_context&.with(**view_context_options(request, response))}.compact
+          {context: view_context_class&.new(**view_context_options(request, response))}.compact
         end
 
         # @api private

data/lib/hanami/extensions/view/context.rb

@@ -104,17 +104,6 @@ module Hanami
               @content_for = source.instance_variable_get(:@content_for).dup
             end
 
-            def with(**args)
-              self.class.new(
-                inflector: @inflector,
-                settings: @settings,
-                assets: @assets,
-                routes: @routes,
-                request: @request,
-                **args
-              )
-            end
-
             def assets
               unless @assets
                 raise Hanami::ComponentLoadError, "the hanami-assets gem is required to access assets"

data/lib/hanami/extensions/view/part.rb

@@ -4,20 +4,69 @@ module Hanami
   module Extensions
     module View
       # @api private
+      # @since 2.1.0
       module Part
         def self.included(part_class)
           super
 
           part_class.extend(Hanami::SliceConfigurable)
-          part_class.include(StandardHelpers)
           part_class.extend(ClassMethods)
         end
 
         module ClassMethods
           def configure_for_slice(slice)
-            extend SliceConfiguredHelpers.new(slice)
+            extend SliceConfiguredPart.new(slice)
+
+            const_set :PartHelpers, Class.new(PartHelpers) { |klass|
+              klass.configure_for_slice(slice)
+            }
           end
         end
+
+        # Returns an object including the default Hanami helpers as well as the user-defined helpers
+        # for the part's slice.
+        #
+        # Use this when you need to access helpers inside your part classes.
+        #
+        # @return PartHelpers
+        #
+        # @api public
+        # @since 2.1.0
+        def helpers
+          @helpers ||= self.class.const_get(:PartHelpers).new(context: _context)
+        end
+      end
+
+      # Standalone helpers class including both {StandardHelpers} as well as the user-defined
+      # helpers for the slice.
+      #
+      # Used used where helpers should be addressed via an intermediary object (i.e. in parts),
+      # rather than mixed into a class directly.
+      #
+      # @api private
+      # @since 2.1.0
+      class PartHelpers
+        extend Hanami::SliceConfigurable
+
+        include StandardHelpers
+
+        def self.configure_for_slice(slice)
+          extend SliceConfiguredHelpers.new(slice)
+        end
+
+        # @api public
+        # @since 2.1.0
+        attr_reader :_context
+
+        # @api public
+        # @since 2.1.0
+        alias_method :context, :_context
+
+        # @api private
+        # @since 2.1.0
+        def initialize(context:)
+          @_context = context
+        end
       end
     end
   end

data/lib/hanami/extensions/view/slice_configured_part.rb

@@ -0,0 +1,72 @@
+# frozen_string_literal: true
+
+module Hanami
+  module Extensions
+    module View
+      # Provides slice-specific configuration and behavior for any view part class defined within a
+      # slice's module namespace.
+      #
+      # @api private
+      # @since 2.1.0
+      class SliceConfiguredPart < Module
+        attr_reader :slice
+
+        # @api private
+        # @since 2.1.0
+        def initialize(slice)
+          super()
+          @slice = slice
+        end
+
+        # @api private
+        # @since 2.1.0
+        def extended(klass)
+          define_new
+        end
+
+        # @api private
+        # @since 2.1.0
+        def inspect
+          "#<#{self.class.name}[#{slice.name}]>"
+        end
+
+        private
+
+        # Defines a `.new` method on the part class that provides a default `rendering:` argument of
+        # a rendering coming from a view configured for the slice. This means that any part can be
+        # initialized standalone (with a `value:` only) and still have access to all the integrated
+        # view facilities from the slice, such as helpers. This is helpful when unit testing parts.
+        #
+        # @example
+        #   module MyApp::Views::Parts
+        #     class Post < MyApp::View::Part
+        #       def title_tag
+        #         helpers.h1(value.title)
+        #       end
+        #     end
+        #   end
+        #
+        #   # Useful when unit testing parts
+        #   part = MyApp::Views::Parts::Post.new(value: hello_world_post)
+        #   part.title_tag # => "<h1>Hello world</h1>"
+        #
+        # @api private
+        # @since 2.1.0
+        def define_new
+          slice = self.slice
+
+          define_method(:new) do |**args|
+            return super(**args) if args.key?(:rendering)
+
+            slice_rendering = Class.new(Hanami::View)
+              .configure_for_slice(slice)
+              .new
+              .rendering
+
+            super(rendering: slice_rendering, **args)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/hanami/extensions/view/slice_configured_view.rb

@@ -108,7 +108,7 @@ module Hanami
           #   - We are a slice, and the view's inherited `paths` is identical to the parent's config
           #     (which would result in the view in a slice erroneously trying to find templates in
           #     the app)
-          if (!slice.parent && view_class.config.paths.empty?) ||
+          if view_class.config.paths.empty? ||
              (slice.parent && view_class.config.paths.map(&:dir) == [templates_path(slice.parent)])
             view_class.config.paths = templates_path(slice)
           end
@@ -178,7 +178,7 @@ module Hanami
             else
               views_namespace.const_set(:Part, Class.new(part_superclass).tap { |klass|
                 # Give the slice to `configure_for_slice`, since it cannot be inferred when it is
-                # called via `.inherited`, since the class is anonymous at this point
+                # called via `.inherited`, because the class is anonymous at this point
                 klass.configure_for_slice(slice)
               })
             end

data/lib/hanami/helpers/assets_helper.rb

@@ -173,10 +173,6 @@ module Hanami
         end
       end
 
-      # @api public
-      # @since 2.1.0
-      alias_method :js, :javascript_tag
-
       # Generate `link` tag for given source(s)
       #
       # It accepts one or more strings representing the name of the asset, if it
@@ -251,7 +247,7 @@ module Hanami
       #
       #   # <link href="https://assets.bookshelf.org/assets/application-28a6b886de2372ee3922fcaf3f78f2d8.css"
       #   #       type="text/css" rel="stylesheet">
-      def stylesheet_link_tag(*source_paths, **options)
+      def stylesheet_tag(*source_paths, **options)
         options = options.reject { |k, _| k.to_sym == :href }
 
         _safe_tags(*source_paths) do |source_path|
@@ -271,10 +267,6 @@ module Hanami
         end
       end
 
-      # @api public
-      # @since 2.1.0
-      alias_method :css, :stylesheet_link_tag
-
       # Generate `img` tag for given source
       #
       # It accepts one string representing the name of the asset, if it comes
@@ -305,37 +297,37 @@ module Hanami
       #
       # @example Basic Usage
       #
-      #   <%= image_tag "logo.png" %>
+      #   <%= image "logo.png" %>
       #
       #   # <img src="/assets/logo.png" alt="Logo">
       #
       # @example Custom alt Attribute
       #
-      #   <%= image_tag "logo.png", alt: "Application Logo" %>
+      #   <%= image "logo.png", alt: "Application Logo" %>
       #
       #   # <img src="/assets/logo.png" alt="Application Logo">
       #
       # @example Custom HTML Attributes
       #
-      #   <%= image_tag "logo.png", id: "logo", class: "image" %>
+      #   <%= image "logo.png", id: "logo", class: "image" %>
       #
       #   # <img src="/assets/logo.png" alt="Logo" id="logo" class="image">
       #
       # @example Absolute URL
       #
-      #   <%= image_tag "https://example-cdn.com/images/logo.png" %>
+      #   <%= image "https://example-cdn.com/images/logo.png" %>
       #
       #   # <img src="https://example-cdn.com/images/logo.png" alt="Logo">
       #
       # @example Fingerprint Mode
       #
-      #   <%= image_tag "logo.png" %>
+      #   <%= image "logo.png" %>
       #
       #   # <img src="/assets/logo-28a6b886de2372ee3922fcaf3f78f2d8.png" alt="Logo">
       #
       # @example CDN Mode
       #
-      #   <%= image_tag "logo.png" %>
+      #   <%= image "logo.png" %>
       #
       #   # <img src="https://assets.bookshelf.org/assets/logo-28a6b886de2372ee3922fcaf3f78f2d8.png" alt="Logo">
       def image_tag(source, options = {})
@@ -376,35 +368,35 @@ module Hanami
       #
       # @example Basic Usage
       #
-      #   <%= favicon_link_tag %>
+      #   <%= favicon %>
       #
       #   # <link href="/assets/favicon.ico" rel="shortcut icon" type="image/x-icon">
       #
       # @example Custom Path
       #
-      #   <%= favicon_link_tag "fav.ico" %>
+      #   <%= favicon "fav.ico" %>
       #
       #   # <link href="/assets/fav.ico" rel="shortcut icon" type="image/x-icon">
       #
       # @example Custom HTML Attributes
       #
-      #   <%= favicon_link_tag "favicon.ico", id: "fav" %>
+      #   <%= favicon "favicon.ico", id: "fav" %>
       #
       #   # <link id: "fav" href="/assets/favicon.ico" rel="shortcut icon" type="image/x-icon">
       #
       # @example Fingerprint Mode
       #
-      #   <%= favicon_link_tag %>
+      #   <%= favicon %>
       #
       #   # <link href="/assets/favicon-28a6b886de2372ee3922fcaf3f78f2d8.ico" rel="shortcut icon" type="image/x-icon">
       #
       # @example CDN Mode
       #
-      #   <%= favicon_link_tag %>
+      #   <%= favicon %>
       #
       #   # <link href="https://assets.bookshelf.org/assets/favicon-28a6b886de2372ee3922fcaf3f78f2d8.ico"
       #           rel="shortcut icon" type="image/x-icon">
-      def favicon_link_tag(source = DEFAULT_FAVICON, options = {})
+      def favicon_tag(source = DEFAULT_FAVICON, options = {})
         options = options.reject { |k, _| k.to_sym == :href }
 
         attributes = {
@@ -417,10 +409,6 @@ module Hanami
         tag.link(**attributes)
       end
 
-      # @api public
-      # @since 2.1.0
-      alias_method :favicon, :favicon_link_tag
-
       # Generate `video` tag for given source
       #
       # It accepts one string representing the name of the asset, if it comes
@@ -454,26 +442,26 @@ module Hanami
       #
       # @example Basic Usage
       #
-      #   <%= video_tag "movie.mp4" %>
+      #   <%= video "movie.mp4" %>
       #
       #   # <video src="/assets/movie.mp4"></video>
       #
       # @example Absolute URL
       #
-      #   <%= video_tag "https://example-cdn.com/assets/movie.mp4" %>
+      #   <%= video "https://example-cdn.com/assets/movie.mp4" %>
       #
       #   # <video src="https://example-cdn.com/assets/movie.mp4"></video>
       #
       # @example Custom HTML Attributes
       #
-      #   <%= video_tag("movie.mp4", autoplay: true, controls: true) %>
+      #   <%= video("movie.mp4", autoplay: true, controls: true) %>
       #
       #   # <video src="/assets/movie.mp4" autoplay="autoplay" controls="controls"></video>
       #
       # @example Fallback Content
       #
       #   <%=
-      #     video_tag("movie.mp4") do
+      #     video("movie.mp4") do
       #       "Your browser does not support the video tag"
       #     end
       #   %>
@@ -485,7 +473,7 @@ module Hanami
       # @example Tracks
       #
       #   <%=
-      #     video_tag("movie.mp4") do
+      #     video("movie.mp4") do
       #       tag.track(kind: "captions", src: asset_url("movie.en.vtt"),
       #             srclang: "en", label: "English")
       #     end
@@ -497,25 +485,25 @@ module Hanami
       #
       # @example Without Any Argument
       #
-      #   <%= video_tag %>
+      #   <%= video %>
       #
       #   # ArgumentError
       #
       # @example Without src And Without Block
       #
-      #   <%= video_tag(content: true) %>
+      #   <%= video(content: true) %>
       #
       #   # ArgumentError
       #
       # @example Fingerprint Mode
       #
-      #   <%= video_tag "movie.mp4" %>
+      #   <%= video "movie.mp4" %>
       #
       #   # <video src="/assets/movie-28a6b886de2372ee3922fcaf3f78f2d8.mp4"></video>
       #
       # @example CDN Mode
       #
-      #   <%= video_tag "movie.mp4" %>
+      #   <%= video "movie.mp4" %>
       #
       #   # <video src="https://assets.bookshelf.org/assets/movie-28a6b886de2372ee3922fcaf3f78f2d8.mp4"></video>
       def video_tag(source = nil, options = {}, &blk)
@@ -556,26 +544,26 @@ module Hanami
       #
       # @example Basic Usage
       #
-      #   <%= audio_tag "song.ogg" %>
+      #   <%= audio "song.ogg" %>
       #
       #   # <audio src="/assets/song.ogg"></audio>
       #
       # @example Absolute URL
       #
-      #   <%= audio_tag "https://example-cdn.com/assets/song.ogg" %>
+      #   <%= audio "https://example-cdn.com/assets/song.ogg" %>
       #
       #   # <audio src="https://example-cdn.com/assets/song.ogg"></audio>
       #
       # @example Custom HTML Attributes
       #
-      #   <%= audio_tag("song.ogg", autoplay: true, controls: true) %>
+      #   <%= audio("song.ogg", autoplay: true, controls: true) %>
       #
       #   # <audio src="/assets/song.ogg" autoplay="autoplay" controls="controls"></audio>
       #
       # @example Fallback Content
       #
       #   <%=
-      #     audio_tag("song.ogg") do
+      #     audio("song.ogg") do
       #       "Your browser does not support the audio tag"
       #     end
       #   %>
@@ -587,7 +575,7 @@ module Hanami
       # @example Tracks
       #
       #   <%=
-      #     audio_tag("song.ogg") do
+      #     audio("song.ogg") do
       #       tag.track(kind: "captions", src: asset_url("song.pt-BR.vtt"),
       #             srclang: "pt-BR", label: "Portuguese")
       #     end
@@ -599,25 +587,25 @@ module Hanami
       #
       # @example Without Any Argument
       #
-      #   <%= audio_tag %>
+      #   <%= audio %>
       #
       #   # ArgumentError
       #
       # @example Without src And Without Block
       #
-      #   <%= audio_tag(controls: true) %>
+      #   <%= audio(controls: true) %>
       #
       #   # ArgumentError
       #
       # @example Fingerprint Mode
       #
-      #   <%= audio_tag "song.ogg" %>
+      #   <%= audio "song.ogg" %>
       #
       #   # <audio src="/assets/song-28a6b886de2372ee3922fcaf3f78f2d8.ogg"></audio>
       #
       # @example CDN Mode
       #
-      #   <%= audio_tag "song.ogg" %>
+      #   <%= audio "song.ogg" %>
       #
       #   # <audio src="https://assets.bookshelf.org/assets/song-28a6b886de2372ee3922fcaf3f78f2d8.ogg"></audio>
       def audio_tag(source = nil, options = {}, &blk)

data/lib/hanami/routes.rb

@@ -60,6 +60,35 @@ module Hanami
       end
     end
 
+    # Wrapper class for the (otherwise opaque) proc returned from {.routes}, adding an `#empty?`
+    # method that returns true if no routes were defined.
+    #
+    # This is useful when needing to determine behaviour based on the presence of user-defined
+    # routes, such as determining whether to show the Hanami welcome page in {Slice#load_router}.
+    #
+    # @api private
+    # @since 2.1.0
+    class RoutesProc < DelegateClass(Proc)
+      # @api private
+      # @since 2.1.0
+      def self.empty
+        new(proc {}, empty: true)
+      end
+
+      # @api private
+      # @since 2.1.0
+      def initialize(proc, empty: false)
+        @empty = empty
+        super(proc)
+      end
+
+      # @api private
+      # @since 2.1.0
+      def empty?
+        !!@empty
+      end
+    end
+
     # @api private
     def self.routes
       @routes ||= build_routes
@@ -68,9 +97,9 @@ module Hanami
     class << self
       # @api private
       def build_routes(definitions = self.definitions)
-        return if definitions.empty?
+        return RoutesProc.empty if definitions.empty?
 
-        proc do
+        routes_proc = proc do
           definitions.each do |(name, args, kwargs, block)|
             if block
               public_send(name, *args, **kwargs, &block)
@@ -79,6 +108,8 @@ module Hanami
             end
           end
         end
+
+        RoutesProc.new(routes_proc)
       end
 
       # @api private