effective_pages 3.0.10 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ce5bc87bbf145bfcf88dfcf9b5efb9ab778876b4d103cc0a2b2cabfbae9f0f05
-  data.tar.gz: 613c1b974148fd8f0b31bcd204ce5cbd979545baf09d08468f9bb44f21ee4ab5
+  metadata.gz: 0f090f6237de5e5aabd329729b2c1d9b2098307bfd46510d4663b95783d5a233
+  data.tar.gz: c8516b8b3e013e8b41660e03cc14a519eee7f0897a80ab0e1cdc6356fd7ec977
 SHA512:
-  metadata.gz: 51e0ba41ee3f97be0fe574b4cd017ba47fe3421e0d7827c0d1fad8522f4d1194ff8dc4e7c721c06d4d0b84b8414ba31a201ccea0ced2ae5dbc43148f1512b216
-  data.tar.gz: 7ff18a08e8d591bc473609244604b73bd8f24a42cccf579492db53d24fb536b7e6c460f36718ea35fa59693ba9aa437fab6806c02e6729565a4330d1cbaf4e42
+  metadata.gz: fb40c0b8c3f7c86ae0374d183bd3217d737594637f469bffb8b93a91bd506a9b20101eb4439698939e835df00b33a38b6c16faf952aa728db9efbda35289c4aa
+  data.tar.gz: e9dd9e7c455f3f38be630aac0a2cea2d9fee080bb319dda6337bc8a5f4c90c62b8fd8021bb70795283c53a150c31ec22beae83a8946e6adaa7b3494150baa815

data/app/controllers/effective/pages_controller.rb

@@ -8,6 +8,8 @@ module Effective
 
       @page = @pages.find(params[:id])
 
+      raise ActionController::RoutingError.new('Not Found') if @page.menu_root_with_children?
+
       if @page.authenticate_user? || @page.roles.present?
         authenticate_user!
       end
@@ -22,9 +24,8 @@ module Effective
       if EffectiveResources.authorized?(self, :admin, :effective_pages)
         flash.now[:warning] = [
           'Hi Admin!',
-          ('You are viewing a hidden page.' unless @page.published?),
-          'Click here to',
-          ("<a href='#{effective_pages.edit_admin_page_path(@page)}' class='alert-link'>edit page settings</a>.")
+          ('You are viewing a draft page.' unless @page.published?),
+          ("<a href='#{effective_pages.edit_admin_page_path(@page)}' class='alert-link'>Click here to edit this page</a>.")
         ].compact.join(' ')
       end
 

data/app/datatables/effective_pages_datatable.rb

@@ -1,10 +1,10 @@
 class EffectivePagesDatatable < Effective::Datatable
 
   filters do
+    scope :all
     scope :published
     scope :draft
     scope :on_menu
-    scope :all
   end
 
   datatable do
@@ -15,6 +15,8 @@ class EffectivePagesDatatable < Effective::Datatable
     col :updated_at, visible: false
 
     col :title
+    col :menu_title
+    col :menu_name
 
     col :slug do |page|
       link_to(page.slug, effective_pages.page_path(page), target: '_blank')
@@ -25,10 +27,8 @@ class EffectivePagesDatatable < Effective::Datatable
     col :layout, visible: false
     col :tempate, visible: false
 
-    col :menu
-    col :menu_name, visible: false
     col :menu_url, visible: false
-    col :menu_parent, visible: false
+    col :menu_parent, search: { collection: admin_menu_parent_collection(), grouped: true }
     col :menu_position, visible: false
 
     col :authenticate_user, visible: false

data/app/datatables/effective_pages_menu_datatable.rb

@@ -13,18 +13,34 @@ class EffectivePagesMenuDatatable < Effective::Datatable
 
     col :menu_url, label: 'Redirect Url'
     col :menu_position, label: 'Position', visible: false
-    col :menu_children, label: 'Children'
+
+    # We only support depth 2 and 3.
+    col :menu_children, label: 'Children' do |page|
+      page.menu_children.map do |child|
+        content_tag(:div, class: 'col-resource_item') do
+          link = link_to(child, effective_pages.edit_admin_page_path(child))
+
+          list = child.menu_children.map do |child|
+            content_tag(:li, link_to(child, effective_pages.edit_admin_page_path(child)))
+          end
+
+          link + (content_tag(:ul, list.join.html_safe) if list.present?).to_s
+        end
+      end.join.html_safe
+    end
 
     actions_col(new: false, destroy: false)
   end
 
   collection(apply_belongs_to: false) do
-    scope = Effective::Page.deep.for_menu(menu)
+    scope = Effective::Page.deep.menuable
+
+    if attributes[:menu].present?
+      scope = scope.root_level.for_menu(menu)
+    end
 
-    scope = if attributes[:menu_parent_id].present?
-      scope.where(menu_parent_id: attributes[:menu_parent_id])
-    else
-      scope.where(menu_parent_id: nil)
+    if attributes[:page_id].present?
+      scope = scope.where(menu_parent_id: attributes[:page_id])
     end
 
     scope

data/app/helpers/effective_menus_helper.rb

@@ -12,18 +12,24 @@ module EffectiveMenusHelper
     content_tag(:ul, options) { render('effective/pages/menu', menu: menu) }
   end
 
+  def render_page_menu(page, options = {}, &block)
+    raise('expected a page with menu true') unless page.kind_of?(Effective::Page) && page.menu?
+
+    content_tag(:ul, options) { render('effective/pages/page_menu', page: page) }
+  end
+
   def render_breadcrumbs(menu, page = @page, root: 'Home')
     return breadcrumbs_root_url(page, root: root) if request.path == '/'
     return breadcrumbs_fallback(page, root: root) unless page.kind_of?(Effective::Page)
 
-    parents = [page.menu_parent].compact
+    parents = [page.menu_parent&.menu_parent, page.menu_parent].compact
 
     content_tag(:ol, class: 'breadcrumb') do
       (
         [content_tag(:li, link_to(root, root_path, title: root), class: 'breadcrumb-item')] +
         parents.map do |page|
           url = (page.menu_url.presence || effective_pages.page_path(page))
-          content_tag(:li, link_to(page, url, title: page.title), class: 'breadcrumb-item')
+          content_tag(:li, link_to(page, (page.menu_root_with_children? ? '#' : url), title: page.title), class: 'breadcrumb-item')
         end +
         [content_tag(:li, page, class: 'breadcrumb-item active', 'aria-current': 'page')]
       ).join.html_safe
@@ -49,4 +55,22 @@ module EffectiveMenusHelper
     end
   end
 
+  def admin_menu_parent_collection(page = nil)
+    raise('expected a page') if page.present? && !page.kind_of?(Effective::Page)
+
+    pages = Effective::Page.menuable.root_level.where.not(id: page)
+
+    if EffectivePages.max_menu_depth == 2
+      # You can only select root level pages
+      pages.group_by(&:menu_name)
+    elsif EffectivePages.max_menu_depth == 3
+      # You can only select root level pages and immediate children
+      pages.map do |page|
+        [[page.to_s, page.id, page.menu_name]] + page.menu_children.map do |child|
+          [' -- ' + child.to_s, child.id, child.menu_name]
+        end
+      end.flatten(1).group_by(&:last)
+    end
+  end
+
 end

data/app/models/effective/page.rb

@@ -59,7 +59,7 @@ module Effective
     validates :layout, presence: true
     validates :template, presence: true
 
-    validates :menu_name, if: -> { menu? && EffectivePages.menus.present? }, presence: true
+    validates :menu_name, if: -> { menu_root? && EffectivePages.menus.present? }, presence: true
 
     # validates :menu_position, if: -> { menu? },
     #   presence: true, uniqueness: { scope: [:menu_name, :menu_parent_id] }
@@ -72,7 +72,7 @@ module Effective
     scope :on_menu, -> { where(menu: true) }
     scope :except_home, -> { where.not(title: 'Home') }
 
-    scope :menuable, -> { where(menu: true).order(:menu_position) }
+    scope :menuable, -> { published.where(menu: true).order(:menu_position) }
     scope :menu_deep, -> { includes(:menu_parent, :menu_children) }
 
     scope :for_menu, -> (name) { menuable.where(menu_name: name) }
@@ -114,8 +114,26 @@ module Effective
       duplicate.tap { |page| page.save! }
     end
 
+    # When true, this should not appear in sitemap.xml and should return 404 if visited
+    def menu_root_with_children?
+      menu_root? && menu_parent?
+    end
+
+    # This is for the form
     def menu_root_level
-      menu? && menu_parent.blank?
+      menu_root?
+    end
+
+    def menu_root?
+      menu? && menu_parent_id.blank?
+    end
+
+    def menu_parent?
+      menu? && menu_children.to_a.present?
+    end
+
+    def menu_child?
+      menu? && menu_parent_id.present?
     end
 
   end

data/app/views/admin/menus/index.html.haml

@@ -1,11 +1,11 @@
 %h1= @page_title
 
 %p
-  Click Reorder to drag & drop reorder the menu.
+  Click Reorder to drag & drop reorder menu items.
   Edit an item to reorder its children items.
 
 - EffectivePages.menus.each do |menu|
-  %h2 #{menu.to_s.titleize} Menu
+  %h3 #{menu.to_s.titleize} Menu
   - datatable = EffectivePagesMenuDatatable.new(menu: menu)
   = render_datatable(datatable, simple: true, inline: true)
-  %hr
+  %hr.my-5

data/app/views/admin/pages/_form.html.haml

@@ -1,13 +1,6 @@
-- if inline_datatable? && inline_datatable.attributes[:menu]
-  - menu = inline_datatable.attributes[:menu]
-  - datatable = EffectivePagesMenuDatatable.new(menu: menu, menu_parent_id: page.id)
-
+- if inline_datatable?
   = render '/admin/pages/form_menu', page: page
 
-  - if datatable.present?(self)
-    %h3 Children
-    = render_datatable(datatable, simple: true, inline: true)
-
 - else
   = tabs do
     = tab 'Page' do
@@ -18,10 +11,6 @@
         = tab 'Menu' do
           = render '/admin/pages/form_menu', page: page
 
-      - unless page.menu_root_level
-        = tab 'Content' do
-          = render '/admin/pages/form_content', page: page
-
       = tab 'Access' do
         = render '/admin/pages/form_access', page: page
 

data/app/views/admin/pages/_form_access.html.haml

@@ -1,10 +1,7 @@
 = effective_form_with(model: page, url: page.persisted? ? effective_pages.admin_page_path(page.id) : effective_pages.admin_pages_path) do |f|
-  = f.check_box :authenticate_user, label: 'Yes, the user must be be signed in to view this page'
+  = f.check_box :authenticate_user, label: 'Restrict the page to any signed in user'
 
   - if EffectivePages.use_effective_roles
-    = f.checks :roles, EffectiveRoles.roles_collection(f.object)
-
-  %p.text-hint
-    * leave blank for a regular public page that anyone can view
+    = f.checks :roles, EffectiveRoles.roles_collection(f.object), label: 'Restrict the page to any signed in user that has the role:'
 
   = f.submit

data/app/views/admin/pages/_form_menu.html.haml

@@ -1,20 +1,20 @@
 = effective_form_with(model: page, url: page.persisted? ? effective_pages.admin_page_path(page.id) : effective_pages.admin_pages_path) do |f|
-  = f.check_box :menu, label: 'Yes, display this page on the menu'
+  = f.check_box :menu, label: 'Yes, display this page in a menu'
 
   = f.show_if :menu, true do
 
-    - if (menus = EffectivePages.menus).length > 1
-      = f.select :menu_name, menus
-    - else
-      = f.hidden_field :menu_name, value: menus.first
-
-    = f.check_box :menu_root_level, label: "This is a top level menu item. Do not show page content."
+    = f.check_box :menu_root_level, label: "This is a top level menu item. It has no content and cannot be viewed, but can have children pages"
 
     = f.show_if :menu_root_level, true do
       = f.hidden_field :menu_parent_id, value: nil
 
+      - if (menus = EffectivePages.menus).length > 1
+        = f.select :menu_name, menus, label: 'Top level menu item in this menu'
+      - else
+        = f.hidden_field :menu_name, value: menus.first
+
     = f.show_if :menu_root_level, false do
-      = f.select :menu_parent_id, Effective::Page.menuable.root_level.where.not(id: f.object), required: true
+      = f.select :menu_parent_id, admin_menu_parent_collection(f.object), required: true, grouped: true
 
     = f.text_field :menu_title, hint: "Display this title on menus instead of full page title"
 
@@ -22,3 +22,13 @@
       hint: "Must start with http(s):// or /"
 
   = f.submit
+
+- if page.menu_parent.present?
+  .mb-4
+    %h3 Parent
+    = link_to(page.menu_parent, effective_pages.edit_admin_page_path(page.menu_parent))
+
+- if page.menu_children.present?
+  %h3 Children
+  - datatable = EffectivePagesMenuDatatable.new(page: page)
+  = render_datatable(datatable, simple: true, inline: true)

data/app/views/admin/pages/_form_page.html.haml

@@ -1,11 +1,12 @@
 = effective_form_with(model: page, url: page.persisted? ? effective_pages.admin_page_path(page.id) : effective_pages.admin_pages_path) do |f|
-  = f.text_field :title, hint: 'The title of your page.'
+
+  = f.text_field :title
 
   = f.check_box :draft,
-    label: 'Save this page as a draft. It will not be accessible on the website.'
+    label: 'Save this page as a draft. It will not appear in a menu and can only be accessed by admin users.'
 
   = f.text_field :meta_description,
-    hint: "A one or two sentence summary of this page. Appears on Google search results underneath the page title.",
+    hint: "150 character summary. Appears on Google search results underneath the page title. ",
     input_html: { maxlength: 150 }
 
   - if (layouts = EffectivePages.layouts).length > 1
@@ -24,6 +25,19 @@
 
   = render partial: '/admin/pages/additional_fields', locals: { page: page, form: f, f: f }
 
+  - if f.object.persisted?
+    - if page.menu_root? && page.menu_parent?
+      .alert.alert-info.my-3.pl-0
+        %ul.mb-0
+          %li This page is a top level menu item with at least one child page
+          %li It will render on the menu as a dropdown link
+          %li Users will not be able to click access page via the menu
+          %li Visiting the page URL directly will return a 404 error
+          %li If you want this page to be accessible while also being a top level menu item, it needs to have no child pages
+
+    - else
+      = render '/admin/pages/rich_text_areas', page: page, f: f
+
   -# This is for duplicate
   - if f.object.new_record? && f.object.rich_texts.present?
     = render partial: '/admin/pages/rich_text_areas', locals: { page: page, form: f, f: f }

data/app/views/admin/pages/_rich_text_areas.html.haml

@@ -1,2 +1,6 @@
-= f.rich_text_area :rich_text_body, label: 'Body', hint: 'The main body of your page'
-= f.rich_text_area :rich_text_sidebar, label: 'Sidebar', hint: 'The sidebar content of your page'
+- if defined?(EffectiveArticleEditor)
+  = f.article_editor :rich_text_body, label: 'Body', hint: 'The main body of your page'
+  = f.article_editor :rich_text_sidebar, label: 'Sidebar', hint: 'The sidebar content of your page'
+- else
+  = f.rich_text_area :rich_text_body, label: 'Body', hint: 'The main body of your page'
+  = f.rich_text_area :rich_text_sidebar, label: 'Sidebar', hint: 'The sidebar content of your page'

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

@@ -1,15 +1,17 @@
 - raise('expected a menu') unless menu.present?
 - menu = menu.to_s
 
+- # 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)
   - next if (page.authenticate_user || page.roles.present?) && current_user.blank?
   - next if page.roles.present? && (current_user.roles & page.roles).blank?
 
-  - menu_children = page.menu_children.select { |page| page.menu_name == menu }
+  - menu_children = page.menu_children
 
   - if menu_children.blank?
     = nav_link_to((page.menu_title.presence || page.title), (page.menu_url.presence || effective_pages.page_path(page)))
+
   - else
     = nav_dropdown(page.to_s) do
       - menu_children.each do |page|

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

@@ -0,0 +1,13 @@
+- raise('expected a page') unless page.present?
+
+- menu_children = page.menu_children
+
+- if menu_children.blank? && page.menu_parent&.menu_children.present?
+  - menu_children = page.menu_parent&.menu_children
+
+- menu_children.each do |page|
+  - next unless EffectiveResources.authorized?(self, :show, page)
+  - next if (page.authenticate_user || page.roles.present?) && current_user.blank?
+  - next if page.roles.present? && (current_user.roles & page.roles).blank?
+
+  = nav_link_to((page.menu_title.presence || page.title), (page.menu_url.presence || effective_pages.page_path(page)))

data/config/effective_pages.rb

@@ -50,4 +50,10 @@ EffectivePages.setup do |config|
   # Layout Settings
   # config.layout = { admin: 'admin' }
 
+  # Menus
+  # Strict bootstrap only supports depth 2. A root level and one dropdown.
+  # Other sites can be configured such that the depth 3 menus are displayed on a sidebar.
+  # Only 2 or 3 are supported right now
+  config.max_menu_depth = 2
+
 end

data/lib/effective_pages/version.rb

@@ -1,3 +1,3 @@
 module EffectivePages
-  VERSION = '3.0.10'.freeze
+  VERSION = '3.1.1'.freeze
 end

data/lib/effective_pages.rb

@@ -12,7 +12,7 @@ module EffectivePages
       :site_og_image, :site_og_image_width, :site_og_image_height,
       :site_title, :site_title_suffix, :fallback_meta_description,
       :silence_missing_page_title_warnings, :silence_missing_meta_description_warnings, :silence_missing_canonical_url_warnings,
-      :use_effective_roles, :menus, :layout
+      :use_effective_roles, :menus, :layout, :max_menu_depth
     ]
   end
 
@@ -43,4 +43,10 @@ module EffectivePages
     end.compact.sort
   end
 
+  def self.max_menu_depth
+    depth = config[:max_menu_depth] || 2
+    raise('only depths 2 and 3 are supported') unless [2, 3].include?(depth)
+    depth
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_pages
 version: !ruby/object:Gem::Version
-  version: 3.0.10
+  version: 3.1.1
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-23 00:00:00.000000000 Z
+date: 2022-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -87,11 +87,11 @@ files:
 - app/views/admin/pages/_additional_fields.html.haml
 - app/views/admin/pages/_form.html.haml
 - app/views/admin/pages/_form_access.html.haml
-- app/views/admin/pages/_form_content.html.haml
 - app/views/admin/pages/_form_menu.html.haml
 - app/views/admin/pages/_form_page.html.haml
 - app/views/admin/pages/_rich_text_areas.html.haml
 - app/views/effective/pages/_menu.html.haml
+- app/views/effective/pages/_page_menu.html.haml
 - config/effective_pages.rb
 - config/routes.rb
 - db/migrate/01_create_effective_pages.rb.erb

data/app/views/admin/pages/_form_content.html.haml

@@ -1,3 +0,0 @@
-= effective_form_with(model: page, url: page.persisted? ? effective_pages.admin_page_path(page.id) : effective_pages.admin_pages_path) do |f|
-  = render '/admin/pages/rich_text_areas', page: page, f: f
-  = effective_submit(f)