effective_pages 3.10.0 → 3.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27068cf38aee344c7a13bfca4c21f5c5076c65df506085efc3c51c7ed3ebb88c
-  data.tar.gz: ccc55fbf35ace55fa7243f3236305e5a847287a8397874e737513f33fdde1f24
+  metadata.gz: e25596768c6b274d44a143a0c4b2733c4608ff8c058282c2d22e65e05486ec09
+  data.tar.gz: 03d2c179fe08a3ec54ef864d2e274350dc209df18925a1b3d2878491673d7ede
 SHA512:
-  metadata.gz: e5cfd09bc93c3caa960614850e26e877db3a754d4136ef427d645f376108cf5a5797deb4b089694865b618769344705481b9451aa22e5b848eeb5bd55239326b
-  data.tar.gz: 4a3b0af8da3d99f927a18322e7aefce5b080c4d22e2c8cd1633d1000093f3e0a4b6ecfc26c73e9aedfbb5789191902afa75ab4feb72aaefb8120abc0092512b3
+  metadata.gz: e3f38c4d20be9133ee318eecf8336753f8494d3fdb7ed82f6f5060e2efea267c49d6025eb3cbe385c3ec4f67b88374ff0d42fa7d721cf9dda8af1ad7701db43c
+  data.tar.gz: 310a83a4fd981312cfcdf48cdf7c2fcad3caeba517e82047b4d497ee610927bf7b6c61ce934fe12cdbb783271a40d856e598a673cae86fc3d3e2819431bf2965

data/app/helpers/effective_carousels_helper.rb

@@ -1,23 +1,14 @@
 # frozen_string_literal: true
 module EffectiveCarouselsHelper
 
-  def render_carousel(name, options = {}, &block)
-    name = name.to_s
-    carousel = Array(EffectivePages.carousels).find { |carousel| carousel.to_s == name }
+  def render_carousel(name, carousel_options = {})
+    carousel = Array(EffectivePages.carousels).find { |carousel| carousel.to_s == name.to_s }
 
     if carousel.blank?
       raise("unable to find carousel #{name}. Please add it to config/initializers/effective_pages.rb")
     end
 
-    carousel_items = Effective::CarouselItem.sorted.deep.where(carousel: carousel)
-    return if carousel_items.blank?
-
-    if block_given?
-      yield(carousel_items); nil
-    else
-      render('effective/carousels/carousel', carousel: carousel, carousel_items: carousel_items, carousel_options: options)
-    end
-
+    render('effective/carousels/carousel', carousel: carousel, carousel_options: carousel_options)
   end
 
 end

data/app/views/effective/carousels/_carousel.html.haml

@@ -1,27 +1,27 @@
-- uid = "effective-carousel-#{carousel}-#{Time.zone.now.to_i}"
-- options = carousel_options
+= cache([carousel, Effective::CarouselItem.maximum(:updated_at)]) do
+  - carousel_items = Effective::CarouselItem.sorted.deep.where(carousel: carousel).to_a
+  - uid = "effective-carousel-#{carousel}-#{Time.zone.now.to_i}"
 
-- raise('expected one or more carousel items') unless carousel_items.present?
+  - if carousel_items.present?
+    .carousel.slide{id: uid, 'data-ride': 'carousel', **carousel_options}
+      %ol.carousel-indicators
+        - carousel_items.each_with_index do |item, index|
+          %li{'data-target': '#' + uid, 'data-slide-to': index, class: ('active' if index == 0)}
 
-.carousel.slide{id: uid, 'data-ride': 'carousel', **options}
-  %ol.carousel-indicators
-    - carousel_items.each_with_index do |item, index|
-      %li{'data-target': '#' + uid, 'data-slide-to': index, class: ('active' if index == 0)}
+      .carousel-inner
+        - carousel_items.each_with_index do |item, index|
+          .carousel-item{class: ('active' if index == 0)}
+            - if item.caption.blank?
+            = image_tag(item.file, class: 'd-block w-100', alt: "Slide #{index+1}")
+            - else
+              = image_tag(item.file, alt: item.caption)
+              .carousel-caption.d-none.d-md-block
+                %p= item.caption
 
-  .carousel-inner
-    - carousel_items.each_with_index do |item, index|
-      .carousel-item{class: ('active' if index == 0)}
-        - if item.caption.blank?
-          = image_tag(item.file, class: 'd-block w-100', alt: "Slide #{index+1}")
-        - else
-          = image_tag(item.file, alt: item.caption)
-          .carousel-caption.d-none.d-md-block
-            %p= item.caption
+      %button.carousel-control-prev{'data-target': '#' + uid, type: 'button', 'data-slide': 'prev'}
+        %span.carousel-control-prev-icon{'aria-hidden': true}
+        %span.sr-only Previous
 
-  %button.carousel-control-prev{'data-target': '#' + uid, type: 'button', 'data-slide': 'prev'}
-    %span.carousel-control-prev-icon{'aria-hidden': true}
-    %span.sr-only Previous
-
-  %button.carousel-control-next{'data-target': '#' + uid, type: 'button', 'data-slide': 'next'}
-    %span.carousel-control-next-icon{'aria-hidden': true}
-    %span.sr-only Next
+      %button.carousel-control-next{'data-target': '#' + uid, type: 'button', 'data-slide': 'next'}
+        %span.carousel-control-next-icon{'aria-hidden': true}
+        %span.sr-only Next

data/app/views/effective/pages/_menu.html.haml

@@ -1,7 +1,7 @@
 - raise('expected a menu') unless menu.present?
 - menu = menu.to_s
 
-= cache([menu, current_user, Effective::Page.for_menu(menu).maximum(:updated_at)]) do
+= cache([menu, current_user, Effective::Page.maximum(:updated_at)]) do
   - # Renders menu_root? level pages and their immediate children
   - Effective::Page.for_menu_root(menu).each do |page|
     - next unless EffectiveResources.authorized?(self, :show, page)

data/lib/effective_pages/version.rb

@@ -1,3 +1,3 @@
 module EffectivePages
-  VERSION = '3.10.0'.freeze
+  VERSION = '3.10.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_pages
 version: !ruby/object:Gem::Version
-  version: 3.10.0
+  version: 3.10.1
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-05 00:00:00.000000000 Z
+date: 2024-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails