alchemy_cms 7.0.0.pre.a → 7.0.0.pre.c

data/app/controllers/alchemy/admin/base_controller.rb

@@ -54,7 +54,7 @@ module Alchemy
         else
           respond_to do |format|
             format.html { render "500", status: 500 }
-            format.json { render json: { message: @notice }, status: 500 }
+            format.json { render json: {message: @notice}, status: 500 }
           end
         end
       end
@@ -101,7 +101,7 @@ module Alchemy
           flash[:notice] = Alchemy.t(flash_notice)
           do_redirect_to redirect_url
         else
-          render action: (params[:action] == "update" ? "edit" : "new")
+          render action: ((params[:action] == "update") ? "edit" : "new")
         end
       end
 
@@ -138,11 +138,11 @@ module Alchemy
       #
       def current_alchemy_site
         @current_alchemy_site ||= begin
-            site_id = params[:site_id] || session[:alchemy_site_id]
-            site = Site.find_by(id: site_id) || super
-            session[:alchemy_site_id] = site&.id
-            site
-          end
+          site_id = params[:site_id] || session[:alchemy_site_id]
+          site = Site.find_by(id: site_id) || super
+          session[:alchemy_site_id] = site&.id
+          site
+        end
       end
 
       def notify_error_tracker(exception)

data/app/controllers/alchemy/admin/clipboard_controller.rb

@@ -3,7 +3,7 @@
 module Alchemy
   module Admin
     class ClipboardController < Alchemy::Admin::BaseController
-      REMARKABLE_TYPES = %w(elements pages)
+      REMARKABLE_TYPES = %w[elements pages]
 
       authorize_resource class: :alchemy_admin_clipboard
       before_action :set_clipboard
@@ -20,7 +20,7 @@ module Alchemy
         unless @clipboard.detect { |item| item["id"] == remarkable_params[:remarkable_id] }
           @clipboard << {
             "id" => remarkable_params[:remarkable_id],
-            "action" => params[:remove] ? "cut" : "copy",
+            "action" => params[:remove] ? "cut" : "copy"
           }
         end
         respond_to do |format|

data/app/controllers/alchemy/admin/elements_controller.rb

@@ -29,10 +29,11 @@ module Alchemy
         @page_version = PageVersion.find(params[:element][:page_version_id])
         @page = @page_version.page
         Element.transaction do
-          if @paste_from_clipboard = params[:paste_from_clipboard].present?
-            @element = paste_element_from_clipboard
+          @paste_from_clipboard = params[:paste_from_clipboard].present?
+          @element = if @paste_from_clipboard
+            paste_element_from_clipboard
           else
-            @element = Element.new(create_element_params)
+            Element.new(create_element_params)
           end
           if @page.definition["insert_elements_at"] == "top"
             @insert_at_top = true
@@ -65,7 +66,7 @@ module Alchemy
       def destroy
         @richtext_ids = @element.richtext_ingredients_ids
         @element.destroy
-        @notice = Alchemy.t("Successfully deleted element") % { element: @element.display_name }
+        @notice = Alchemy.t("Successfully deleted element") % {element: @element.display_name}
       end
 
       def publish
@@ -80,7 +81,7 @@ module Alchemy
             # element over from another nestable element
             Element.find_by(id: element_id).update_columns(
               parent_element_id: params[:parent_element_id],
-              position: position,
+              position: position
             )
           end
           # Need to manually touch the parent because Rails does not do it
@@ -95,24 +96,38 @@ module Alchemy
         @page = @element.page
         # We do not want to trigger the touch callback or any validations
         @element.update_columns(folded: !@element.folded)
+        # Fold all nested elements if folded
+        if @element.folded?
+          ids = collapse_nested_elements_ids(@element)
+          Alchemy::Element.where(id: ids).update_all(folded: true)
+        end
       end
 
       private
 
+      def collapse_nested_elements_ids(element)
+        ids = []
+        element.all_nested_elements.includes(:all_nested_elements).reject(&:compact?).each do |nested_element|
+          ids.push nested_element.id if nested_element.expanded?
+          ids.concat collapse_nested_elements_ids(nested_element) if nested_element.all_nested_elements.reject(&:compact?).any?
+        end
+        ids
+      end
+
       def element_includes
         [
           {
-            ingredients: :related_object,
+            ingredients: :related_object
           },
           :tags,
           {
             all_nested_elements: [
               {
-                ingredients: :related_object,
+                ingredients: :related_object
               },
-              :tags,
-            ],
-          },
+              :tags
+            ]
+          }
         ]
       end
 
@@ -138,7 +153,7 @@ module Alchemy
           @source_element,
           {
             parent_element_id: create_element_params[:parent_element_id],
-            page_version_id: @page_version.id,
+            page_version_id: @page_version.id
           }
         )
         if element_from_clipboard["action"] == "cut"

data/app/controllers/alchemy/admin/languages_controller.rb

@@ -14,7 +14,7 @@ module Alchemy
       def new
         @language = Language.new(
           site: @current_site,
-          page_layout: Config.get(:default_language)["page_layout"],
+          page_layout: Config.get(:default_language)["page_layout"]
         )
       end
 

data/app/controllers/alchemy/admin/nodes_controller.rb

@@ -12,7 +12,7 @@ module Alchemy
       def new
         @node = Node.new(
           parent_id: params[:parent_id],
-          language: @current_language,
+          language: @current_language
         )
       end
 
@@ -28,7 +28,7 @@ module Alchemy
           :url,
           :title,
           :nofollow,
-          :external,
+          :external
         )
       end
     end

data/app/controllers/alchemy/admin/pages_controller.rb

@@ -109,7 +109,7 @@ module Alchemy
         @preview_urls = Alchemy.preview_sources.map do |klass|
           [
             klass.model_name.human,
-            klass.new(routes: Alchemy::Engine.routes).url_for(@page),
+            klass.new(routes: Alchemy::Engine.routes).url_for(@page)
           ]
         end
         @layoutpage = @page.layoutpage?
@@ -152,10 +152,10 @@ module Alchemy
         respond_to do |format|
           format.js do
             @redirect_url = if @page.layoutpage?
-                alchemy.admin_layoutpages_path
-              else
-                alchemy.admin_pages_path
-              end
+              alchemy.admin_layoutpages_path
+            else
+              alchemy.admin_pages_path
+            end
 
             render :redirect
           end
@@ -185,7 +185,7 @@ module Alchemy
           format.html do
             redirect_to(
               params[:redirect_to].presence || admin_pages_path,
-              allow_other_host: true,
+              allow_other_host: true
             )
           end
         end
@@ -248,7 +248,7 @@ module Alchemy
         Page.copy(
           language_root_to_copy_from,
           language_id: params[:languages][:new_lang_id],
-          language_code: @current_language.code,
+          language_code: @current_language.code
         )
       end
 
@@ -322,7 +322,7 @@ module Alchemy
       #
       def process_url(ancestors_path, item)
         default_urlname = (ancestors_path.blank? ? "" : "#{ancestors_path}/") + item["slug"].to_s
-        { my_urlname: default_urlname, children_path: default_urlname }
+        {my_urlname: default_urlname, children_path: default_urlname}
       end
 
       def load_resource
@@ -333,7 +333,7 @@ module Alchemy
         request.raw_post.split("&").map do |i|
           parts = i.split("=")
           {
-            parts[0].gsub(/[^0-9]/, "") => parts[1],
+            parts[0].gsub(/[^0-9]/, "") => parts[1]
           }
         end
       end
@@ -391,7 +391,7 @@ module Alchemy
         PageTreeSerializer.new(
           @page,
           ability: current_ability,
-          user: current_alchemy_user,
+          user: current_alchemy_user
         )
       end
 

data/app/controllers/alchemy/admin/pictures_controller.rb

@@ -44,7 +44,7 @@ module Alchemy
         render json: {
           url: @picture.url(options),
           alt: @picture.name,
-          title: Alchemy.t(:image_name, name: @picture.name),
+          title: Alchemy.t(:image_name, name: @picture.name)
         }
       end
 
@@ -64,15 +64,15 @@ module Alchemy
       end
 
       def update
-        if @picture.update(picture_params)
-          @message = {
+        @message = if @picture.update(picture_params)
+          {
             body: Alchemy.t(:picture_updated_successfully, name: @picture.name),
-            type: "notice",
+            type: "notice"
           }
         else
-          @message = {
+          {
             body: Alchemy.t(:picture_update_failed),
-            type: "error",
+            type: "error"
           }
         end
         render :update
@@ -103,7 +103,7 @@ module Alchemy
           if not_deletable.any?
             flash[:warn] = Alchemy.t(
               "These pictures could not be deleted, because they were in use",
-              names: not_deletable.to_sentence,
+              names: not_deletable.to_sentence
             )
           else
             flash[:notice] = Alchemy.t("Pictures deleted successfully", names: names.to_sentence)
@@ -111,7 +111,7 @@ module Alchemy
         else
           flash[:warn] = Alchemy.t("Could not delete Pictures")
         end
-      rescue StandardError => e
+      rescue => e
         flash[:error] = e.message
       ensure
         redirect_to_index
@@ -121,7 +121,7 @@ module Alchemy
         name = @picture.name
         @picture.destroy
         flash[:notice] = Alchemy.t("Picture deleted successfully", name: name)
-      rescue StandardError => e
+      rescue => e
         flash[:error] = e.message
       ensure
         redirect_to_index
@@ -153,8 +153,8 @@ module Alchemy
           end
         else
           cookies[:alchemy_pictures_per_page] = params[:per_page] ||
-                                                cookies[:alchemy_pictures_per_page] ||
-                                                pictures_per_page_for_size
+            cookies[:alchemy_pictures_per_page] ||
+            pictures_per_page_for_size
         end
       end
 
@@ -186,9 +186,9 @@ module Alchemy
       def search_filter_params
         @_search_filter_params ||= params.except(*COMMON_SEARCH_FILTER_EXCLUDES + [:picture_ids]).permit(
           *common_search_filter_includes + [
-          :size,
-          :form_field_id,
-        ],
+            :size,
+            :form_field_id
+          ]
         )
       end
 

data/app/controllers/alchemy/admin/resources_controller.rb

@@ -8,7 +8,7 @@ require "alchemy/resource_filter"
 module Alchemy
   module Admin
     class ResourcesController < Alchemy::Admin::BaseController
-      COMMON_SEARCH_FILTER_EXCLUDES = [:id, :utf8, :_method, :_, :format].freeze
+      COMMON_SEARCH_FILTER_EXCLUDES = %i[id utf8 _method _ format].freeze
 
       include Alchemy::ResourcesHelper
 
@@ -18,7 +18,7 @@ module Alchemy
         :resource_filters_for_select
 
       before_action :load_resource,
-        only: [:show, :edit, :update, :destroy]
+        only: %i[show edit update destroy]
 
       before_action :authorize_resource
 
@@ -27,26 +27,18 @@ module Alchemy
         @query.sorts = default_sort_order if @query.sorts.empty?
         items = @query.result
 
-        if contains_relations?
-          items = items.includes(*resource_relations_names)
-        end
-
-        if search_filter_params[:tagged_with].present?
-          items = items.tagged_with(search_filter_params[:tagged_with])
-        end
-
-        if search_filter_params[:filter].present?
-          items = apply_filters(items)
-        end
+        items = items.includes(*resource_relations_names) if contains_relations?
+        items = items.tagged_with(search_filter_params[:tagged_with]) if search_filter_params[:tagged_with].present?
+        items = apply_filters(items) if search_filter_params[:filter].present?
 
         respond_to do |format|
-          format.html {
+          format.html do
             items = items.page(params[:page] || 1).per(items_per_page)
             instance_variable_set("@#{resource_handler.resources_name}", items)
-          }
-          format.csv {
+          end
+          format.csv do
             instance_variable_set("@#{resource_handler.resources_name}", items)
-          }
+          end
         end
       end
 
@@ -58,7 +50,8 @@ module Alchemy
         render action: "edit"
       end
 
-      def edit; end
+      def edit
+      end
 
       def create
         instance_variable_set("@#{resource_handler.resource_name}", resource_handler.model.new(resource_params))
@@ -66,7 +59,7 @@ module Alchemy
         render_errors_or_redirect(
           resource_instance_variable,
           resources_path(resource_instance_variable.class, search_filter_params),
-          flash_notice_for_resource_action,
+          flash_notice_for_resource_action
         )
       end
 
@@ -75,7 +68,7 @@ module Alchemy
         render_errors_or_redirect(
           resource_instance_variable,
           resources_path(resource_instance_variable.class, search_filter_params),
-          flash_notice_for_resource_action,
+          flash_notice_for_resource_action
         )
       end
 
@@ -161,7 +154,10 @@ module Alchemy
         when :destroy
           verb = "removed"
         end
-        flash[:notice] = Alchemy.t("#{resource_handler.resource_name.classify} successfully #{verb}", default: Alchemy.t("Successfully #{verb}"))
+        flash[:notice] = Alchemy.t(
+          "#{resource_handler.resource_name.classify} successfully #{verb}",
+          default: Alchemy.t("Successfully #{verb}")
+        )
       end
 
       def is_alchemy_module?
@@ -198,18 +194,20 @@ module Alchemy
 
       def common_search_filter_includes
         search_filters = [
-          { q: [
-            resource_handler.search_field_name,
-            :s,
-          ] },
+          {
+            q: [
+              resource_handler.search_field_name,
+              :s
+            ]
+          },
           :tagged_with,
           :page,
-          :per_page,
+          :per_page
         ]
 
         if resource_has_filters
           search_filters << {
-            filter: resource_filters.map { |f| f[:name] },
+            filter: resource_filters.map { |f| f[:name] }
           }
         end
 
@@ -217,7 +215,8 @@ module Alchemy
       end
 
       def items_per_page
-        cookies[:alchemy_items_per_page] = params[:per_page] || cookies[:alchemy_items_per_page] || Alchemy::Config.get(:items_per_page)
+        cookies[:alchemy_items_per_page] =
+          params[:per_page] || cookies[:alchemy_items_per_page] || Alchemy::Config.get(:items_per_page)
       end
 
       def items_per_page_options

data/app/controllers/alchemy/admin/styleguide_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Alchemy
   module Admin
     class StyleguideController < BaseController

data/app/controllers/alchemy/admin/tags_controller.rb

@@ -6,31 +6,31 @@ module Alchemy
       before_action :load_tag, only: [:edit, :update, :destroy]
 
       def index
-        @query = Gutentag::Tag.ransack(search_filter_params[:q])
+        @query = Tag.ransack(search_filter_params[:q])
         @query.sorts = default_sort_order if @query.sorts.empty?
         @tags = @query
-                  .result
-                  .page(params[:page] || 1)
-                  .per(items_per_page)
-                  .order("name ASC")
+          .result
+          .page(params[:page] || 1)
+          .per(items_per_page)
+          .order("name ASC")
       end
 
       def new
-        @tag = Gutentag::Tag.new
+        @tag = Tag.new
       end
 
       def create
-        @tag = Gutentag::Tag.create(tag_params)
+        @tag = Tag.create(tag_params)
         render_errors_or_redirect @tag, admin_tags_path, Alchemy.t("New Tag Created")
       end
 
       def edit
-        @tags = Gutentag::Tag.order("name ASC").to_a - [@tag]
+        @tags = Tag.order("name ASC").to_a - [@tag]
       end
 
       def update
         if tag_params[:merge_to]
-          @new_tag = Gutentag::Tag.find(tag_params[:merge_to])
+          @new_tag = Tag.find(tag_params[:merge_to])
           Tag.replace(@tag, @new_tag)
           operation_text = Alchemy.t("Replaced Tag") % {old_tag: @tag.name, new_tag: @new_tag.name}
           @tag.destroy
@@ -57,7 +57,7 @@ module Alchemy
       private
 
       def load_tag
-        @tag = Gutentag::Tag.find(params[:id])
+        @tag = Tag.find(params[:id])
       end
 
       def tag_params
@@ -67,7 +67,7 @@ module Alchemy
       def tags_from_term(term)
         return [] if term.blank?
 
-        Gutentag::Tag.where(["LOWER(name) LIKE ?", "#{term.downcase}%"])
+        Tag.where(["LOWER(name) LIKE ?", "#{term.downcase}%"])
       end
 
       def json_for_autocomplete(items, attribute)

data/app/controllers/alchemy/api/base_controller.rb

@@ -12,11 +12,11 @@ module Alchemy
     private
 
     def render_not_authorized
-      render json: { error: "Not authorized" }, status: 403
+      render json: {error: "Not authorized"}, status: 403
     end
 
     def render_not_found
-      render json: { error: "Record not found" }, status: 404
+      render json: {error: "Record not found"}, status: 404
     end
   end
 end

data/app/controllers/alchemy/api/elements_controller.rb

@@ -10,18 +10,18 @@ module Alchemy
     #
     def index
       # Fix for cancancan not able to merge multiple AR scopes for logged in users
-      if cannot? :manage, Alchemy::Element
-        @elements = Alchemy::Element.accessible_by(current_ability, :index)
+      @elements = if cannot? :manage, Alchemy::Element
+        Alchemy::Element.accessible_by(current_ability, :index)
       else
-        @elements = Alchemy::Element.all
+        Alchemy::Element.all
       end
 
       @elements = @elements.not_nested.joins(:page_version).merge(PageVersion.published)
 
-      if params[:page_id].present?
-        @elements = @elements.includes(:page).where(alchemy_pages: { id: params[:page_id] })
+      @elements = if params[:page_id].present?
+        @elements.includes(:page).where(alchemy_pages: {id: params[:page_id]})
       else
-        @elements = @elements.includes(*element_includes)
+        @elements.includes(*element_includes)
       end
 
       if params[:named].present?
@@ -47,15 +47,15 @@ module Alchemy
         {
           nested_elements: [
             {
-              ingredients: :related_object,
+              ingredients: :related_object
             },
-            :tags,
-          ],
+            :tags
+          ]
         },
         {
-          ingredients: :related_object,
+          ingredients: :related_object
         },
-        :tags,
+        :tags
       ]
     end
   end

data/app/controllers/alchemy/api/ingredients_controller.rb

@@ -11,7 +11,7 @@ module Alchemy
 
       if params[:page_id].present?
         @ingredients = @ingredients
-          .where(alchemy_page_versions: { page_id: params[:page_id] })
+          .where(alchemy_page_versions: {page_id: params[:page_id]})
           .merge(Alchemy::PageVersion.drafts)
           .joins(element: :page_version)
       end

data/app/controllers/alchemy/api/nodes_controller.rb

@@ -43,7 +43,7 @@ module Alchemy
       {
         total_count: total_count_value,
         per_page: per_page_value,
-        page: page_value,
+        page: page_value
       }
     end
 

data/app/controllers/alchemy/api/pages_controller.rb

@@ -35,7 +35,7 @@ module Alchemy
         ability: current_ability,
         user: current_alchemy_user,
         elements: params[:elements],
-        full: true,
+        full: true
       )
     end
 
@@ -72,7 +72,7 @@ module Alchemy
 
       Language.current.pages.where(
         urlname: params[:urlname],
-        language_code: params[:locale] || Language.current.code,
+        language_code: params[:locale] || Language.current.code
       ).includes(page_includes).first
     end
 
@@ -80,7 +80,7 @@ module Alchemy
       {
         total_count: total_count_value,
         per_page: per_page_value,
-        page: page_value,
+        page: page_value
       }
     end
 
@@ -97,7 +97,7 @@ module Alchemy
     end
 
     def page_value
-      params[:page] ? params[:page].to_i : nil
+      params[:page]&.to_i
     end
 
     def page_includes
@@ -109,17 +109,17 @@ module Alchemy
             {
               nested_elements: [
                 {
-                  ingredients: :related_object,
+                  ingredients: :related_object
                 },
-                :tags,
-              ],
+                :tags
+              ]
             },
             {
-              ingredients: :related_object,
+              ingredients: :related_object
             },
-            :tags,
-          ],
-        },
+            :tags
+          ]
+        }
       ]
     end
   end

data/app/controllers/alchemy/attachments_controller.rb

@@ -13,8 +13,8 @@ module Alchemy
         {
           filename: @attachment.file_name,
           type: @attachment.file_mime_type,
-          disposition: "inline",
-        },
+          disposition: "inline"
+        }
       )
     end
 
@@ -24,7 +24,7 @@ module Alchemy
       send_file(
         @attachment.file.path, {
           filename: @attachment.file_name,
-          type: @attachment.file_mime_type,
+          type: @attachment.file_mime_type
         }
       )
     end