activeadmin 3.4.0 → 4.0.0.beta18

data/lib/active_admin/dynamic_settings_node.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
-require "active_admin/dynamic_setting"
-require "active_admin/settings_node"
+require_relative "dynamic_setting"
+require_relative "settings_node"
 
 module ActiveAdmin
 
@@ -10,18 +10,18 @@ module ActiveAdmin
         class_attribute "#{name}_setting"
         add_reader(name)
         add_writer(name, type)
-        send "#{name}=", value
+        send :"#{name}=", value
       end
 
       def add_reader(name)
         define_singleton_method(name) do |*args|
-          send("#{name}_setting").value(*args)
+          send(:"#{name}_setting").value(*args)
         end
       end
 
       def add_writer(name, type)
-        define_singleton_method("#{name}=") do |value|
-          send("#{name}_setting=", DynamicSetting.build(value, type))
+        define_singleton_method(:"#{name}=") do |value|
+          send(:"#{name}_setting=", DynamicSetting.build(value, type))
         end
       end
     end

data/lib/active_admin/engine.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 module ActiveAdmin
   class Engine < ::Rails::Engine
+    isolate_namespace ActiveAdmin
+
     # Set default values for app_path and load_paths before running initializers
     initializer "active_admin.load_app_path", before: :load_config_initializers do |app|
       ActiveAdmin::Application.setting :app_path, app.root
@@ -8,18 +10,32 @@ module ActiveAdmin
     end
 
     initializer "active_admin.precompile", group: :all do |app|
-      unless ActiveAdmin.application.use_webpacker
-        ActiveAdmin.application.stylesheets.each do |path, _|
-          app.config.assets.precompile << path
-        end
-        ActiveAdmin.application.javascripts.each do |path, _|
-          app.config.assets.precompile << path
+      if app.config.respond_to?(:assets)
+        app.config.assets.precompile += %w(active_admin.js active_admin.css active_admin_manifest.js)
+      end
+    end
+
+    initializer "active_admin.importmap", after: "importmap" do |app|
+      # Skip if importmap-rails is not installed
+      next unless app.config.respond_to?(:importmap)
+
+      ActiveAdmin.importmap.draw(Engine.root.join("config", "importmap.rb"))
+      package_path = Engine.root.join("app/javascript")
+      if app.config.respond_to?(:assets)
+        app.config.assets.paths << package_path
+        app.config.assets.paths << Engine.root.join("vendor/javascript")
+      end
+
+      if app.config.importmap.sweep_cache
+        ActiveAdmin.importmap.cache_sweeper(watches: package_path)
+        ActiveSupport.on_load(:action_controller_base) do
+          before_action { ActiveAdmin.importmap.cache_sweeper.execute_if_updated }
         end
       end
     end
 
     initializer "active_admin.routes" do
-      require "active_admin/helpers/routes/url_helpers"
+      require_relative "helpers/routes/url_helpers"
     end
 
     initializer "active_admin.deprecator" do |app|

data/lib/active_admin/filters/active.rb

@@ -1,21 +1,25 @@
 # frozen_string_literal: true
-require "active_admin/filters/active_filter"
+require_relative "active_filter"
 
 module ActiveAdmin
   module Filters
-
     class Active
-      attr_accessor :filters, :resource
+      attr_reader :filters, :resource, :scopes
 
       # Instantiate a `Active` object containing collection of current active filters
 
       # @param resource [ActiveAdmin::Resource] current resource
       # @param search [Ransack::Search] search object
       #
-      # @see ActiveAdmin::ResourceController::DataAcces#apply_filtering
+      # @see ActiveAdmin::ResourceController::DataAccess#apply_filtering
       def initialize(resource, search)
         @resource = resource
         @filters = build_filters(search.conditions)
+        @scopes = search.instance_variable_get(:@scope_args)
+      end
+
+      def all_blank?
+        filters.blank? && scopes.blank?
       end
 
       private
@@ -23,8 +27,6 @@ module ActiveAdmin
       def build_filters(conditions)
         conditions.map { |condition| ActiveFilter.new(resource, condition.dup) }
       end
-
     end
-
   end
 end

data/lib/active_admin/filters/active_filter.rb

@@ -3,8 +3,6 @@ module ActiveAdmin
   module Filters
 
     class ActiveFilter
-      include ActiveAdmin::ViewHelpers
-
       attr_reader :resource, :condition, :related_class
 
       # Instantiate a `ActiveFilter`
@@ -40,13 +38,11 @@ module ActiveAdmin
       end
 
       def predicate_name
-        I18n.t(
-          "ransack.predicates.#{condition.predicate.name}",
-          default: ransack_predicate_name)
+        Ransack::Translate.predicate(condition.predicate.name)
       end
 
       def html_options
-        { class: "current_filter current_filter_#{condition.key}" }
+        { "data-filter": condition.key }
       end
 
       private
@@ -80,10 +76,6 @@ module ActiveAdmin
         condition_attribute.attr_name
       end
 
-      def ransack_predicate_name
-        Ransack::Translate.predicate(condition.predicate.name)
-      end
-
       def find_class?
         ["eq", "in"].include? condition.predicate.arel_predicate
       end

data/lib/active_admin/filters/forms.rb

@@ -7,9 +7,6 @@ module ActiveAdmin
       include ::ActiveAdmin::Filters::FormtasticAddons
       self.input_namespaces = [::Object, ::ActiveAdmin::Inputs::Filters, ::ActiveAdmin::Inputs, ::Formtastic::Inputs]
 
-      # TODO: remove input class finders after formtastic 4 (where it will be default)
-      self.input_class_finder = ::Formtastic::InputClassFinder
-
       def filter(method, options = {})
         if method.present? && options[:as] ||= default_input_type(method)
           template.concat input(method, options)
@@ -21,7 +18,7 @@ module ActiveAdmin
       # Returns the default filter type for a given attribute. If you want
       # to use a custom search method, you have to specify the type yourself.
       def default_input_type(method, options = {})
-        if method =~ /_(eq|cont|start|end)\z/
+        if /_(eq|cont|start|end)\z/.match?(method)
           :string
         elsif klass._ransackers.key?(method.to_s)
           klass._ransackers[method.to_s].type
@@ -41,46 +38,5 @@ module ActiveAdmin
         end
       end
     end
-
-    # This module is included into the view
-    module ViewHelper
-
-      # Helper method to render a filter form
-      def active_admin_filters_form_for(search, filters, options = {})
-        defaults = { builder: ActiveAdmin::Filters::FormBuilder,
-                     url: collection_path,
-                     html: { class: "filter_form" } }
-        required = { html: { method: :get },
-                     as: :q }
-        options = defaults.deep_merge(options).deep_merge(required)
-
-        form_for search, options do |f|
-          filters.each do |attribute, opts|
-            next if opts.key?(:if) && !call_method_or_proc_on(self, opts[:if])
-            next if opts.key?(:unless) && call_method_or_proc_on(self, opts[:unless])
-
-            filter_opts = opts.except(:if, :unless)
-            filter_opts[:input_html] = instance_exec(&filter_opts[:input_html]) if filter_opts[:input_html].is_a?(Proc)
-
-            f.filter attribute, filter_opts
-          end
-
-          buttons = content_tag :div, class: "buttons" do
-            f.submit(I18n.t("active_admin.filters.buttons.filter")) +
-              link_to(I18n.t("active_admin.filters.buttons.clear"), "#", class: "clear_filters_btn") +
-              hidden_field_tags_for(params, except: except_hidden_fields)
-          end
-
-          f.template.concat buttons
-        end
-      end
-
-      private
-
-      def except_hidden_fields
-        [:q, :page]
-      end
-    end
-
   end
 end

data/lib/active_admin/filters/resource_extension.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+require_relative "active"
+
 module ActiveAdmin
   module Filters
-
     class Disabled < RuntimeError
       def initialize(action)
         super "Cannot #{action} a filter when filters are disabled. Enable filters with 'config.filters = true'"
@@ -9,11 +10,10 @@ module ActiveAdmin
     end
 
     module ResourceExtension
-
       def initialize(*)
         super
         add_filters_sidebar_section
-        add_search_status_sidebar_section
+        add_active_search_sidebar_section
       end
 
       # Returns the filters for this resource. If filters are not enabled,
@@ -98,7 +98,7 @@ module ActiveAdmin
         end
 
         if @filters_to_remove
-          @filters_to_remove.each &filters.method(:delete)
+          @filters_to_remove.each { |filter| filters.delete(filter) }
         end
 
         filters
@@ -139,14 +139,14 @@ module ActiveAdmin
             end
 
             # Remove high-arity associations with no searchable column
-            high_arity = high_arity.select(&method(:searchable_column_for))
+            high_arity = high_arity.select { |r| searchable_column_for(r) }
 
             high_arity = high_arity.map { |r| r.name.to_s + "_" + searchable_column_for(r) + namespace.filter_method_for_large_association }
 
             filters = poly.map(&:foreign_type) + low_arity.map(&:name) + high_arity
           end
 
-          filters.map &:to_sym
+          filters.map(&:to_sym)
         else
           []
         end
@@ -165,16 +165,24 @@ module ActiveAdmin
       end
 
       def filters_sidebar_section
-        ActiveAdmin::SidebarSection.new :filters, only: :index, if: -> { active_admin_config.filters.any? } do
+        name = :filters
+        ActiveAdmin::SidebarSection.new name, only: :index, if: -> { active_admin_config.filters.any? } do
+          h3 I18n.t("active_admin.sidebars.#{name}", default: name.to_s.titlecase), class: "filters-form-title"
           active_admin_filters_form_for assigns[:search], **active_admin_config.filters
         end
       end
 
-      def add_search_status_sidebar_section
-        self.sidebar_sections << ActiveAdmin::Filters::ActiveSidebar.new
+      def add_active_search_sidebar_section
+        self.sidebar_sections << active_search_sidebar_section
       end
 
+      def active_search_sidebar_section
+        name = :active_search
+        ActiveAdmin::SidebarSection.new name, only: :index, if: -> { active_admin_config.current_filters_enabled? && (params[:q] || params[:scope]) } do
+          filters = ActiveAdmin::Filters::Active.new(active_admin_config, assigns[:search])
+          render "active_filters", active_filters: filters
+        end
+      end
     end
-
   end
 end

data/lib/active_admin/filters.rb

@@ -1,12 +1,10 @@
 # frozen_string_literal: true
-require "active_admin/filters/dsl"
-require "active_admin/filters/resource_extension"
-require "active_admin/filters/formtastic_addons"
-require "active_admin/filters/forms"
-require "active_admin/helpers/optional_display"
-require "active_admin/filters/active_sidebar"
+require_relative "filters/dsl"
+require_relative "filters/resource_extension"
+require_relative "filters/formtastic_addons"
+require_relative "filters/forms"
+require_relative "helpers/optional_display"
 
 # Add our Extensions
 ActiveAdmin::ResourceDSL.send :include, ActiveAdmin::Filters::DSL
 ActiveAdmin::Resource.send :include, ActiveAdmin::Filters::ResourceExtension
-ActiveAdmin::ViewHelpers.send :include, ActiveAdmin::Filters::ViewHelper

data/lib/active_admin/form_builder.rb

@@ -17,12 +17,9 @@ module ActiveAdmin
 
     self.input_namespaces = [::Object, ::ActiveAdmin::Inputs, ::Formtastic::Inputs]
 
-    # TODO: remove both class finders after formtastic 4 (where it will be default)
-    self.input_class_finder = ::Formtastic::InputClassFinder
-    self.action_class_finder = ::Formtastic::ActionClassFinder
-
     def cancel_link(url = { action: "index" }, html_options = {}, li_attrs = {})
-      li_attrs[:class] ||= "cancel"
+      li_attrs[:class] ||= "action cancel"
+      html_options[:class] ||= "cancel-link"
       li_content = template.link_to I18n.t("active_admin.cancel"), url, html_options
       template.content_tag(:li, li_content, li_attrs)
     end
@@ -48,7 +45,7 @@ module ActiveAdmin
       @assoc = assoc
       @options = extract_custom_settings!(options.dup)
       @options.reverse_merge!(for: assoc)
-      @options[:class] = [options[:class], "inputs has_many_fields"].compact.join(" ")
+      @options[:class] = [options[:class], "inputs has-many-fields"].compact.join(" ")
 
       if sortable_column
         @options[:for] = [assoc, sorted_children(sortable_column)]
@@ -57,7 +54,7 @@ module ActiveAdmin
 
     def render(&block)
       html = "".html_safe
-      html << template.content_tag(:h3) { heading } if heading.present?
+      html << template.content_tag(:h3, class: "has-many-fields-title") { heading } if heading.present?
       html << template.capture { content_has_many(&block) }
       html = wrap_div_or_li(html)
       template.concat(html) if template.output_buffer
@@ -78,8 +75,7 @@ module ActiveAdmin
     end
 
     def default_heading
-      assoc_klass.model_name.
-        human(count: ::ActiveAdmin::Helpers::I18n::PLURAL_MANY_COUNT)
+      assoc_klass.model_name.human(count: 2.1)
     end
 
     def assoc_klass
@@ -108,23 +104,23 @@ module ActiveAdmin
 
     def has_many_actions(form_builder, contents)
       if form_builder.object.new_record?
-        contents << template.content_tag(:li) do
+        contents << template.content_tag(:li, class: "input") do
           remove_text = remove_record.is_a?(String) ? remove_record : I18n.t("active_admin.has_many_remove")
-          template.link_to remove_text, "#", class: "button has_many_remove"
+          template.link_to remove_text, "#", class: "has-many-remove"
         end
       elsif allow_destroy?(form_builder.object)
         form_builder.input(
           :_destroy, as: :boolean,
-                     wrapper_html: { class: "has_many_delete" },
+                     wrapper_html: { class: "has-many-delete" },
                      label: I18n.t("active_admin.has_many_delete"))
       end
 
       if sortable_column
         form_builder.input sortable_column, as: :hidden
 
-        contents << template.content_tag(:li, class: "handle") do
-          I18n.t("active_admin.move")
-        end
+        # contents << template.content_tag(:li, class: "handle") do
+        #   I18n.t("active_admin.move")
+        # end
       end
 
       contents
@@ -169,7 +165,7 @@ module ActiveAdmin
       html = template.capture { __getobj__.send(:inputs_for_nested_attributes, opts, &form_block) }
       text = new_record.is_a?(String) ? new_record : I18n.t("active_admin.has_many_new", model: assoc_name.human)
 
-      template.link_to text, "#", class: "button has_many_add", data: {
+      template.link_to text, "#", class: "has-many-add", data: {
         html: CGI.escapeHTML(html).html_safe, placeholder: placeholder
       }
     end
@@ -178,7 +174,8 @@ module ActiveAdmin
       template.content_tag(
         already_in_an_inputs_block ? :li : :div,
         html,
-        class: "has_many_container #{assoc}",
+        class: "has-many-container",
+        "data-has-many-association" => assoc,
         "data-sortable" => sortable_column,
         "data-sortable-start" => sortable_start)
     end

data/lib/active_admin/helpers/optional_display.rb

@@ -23,7 +23,7 @@ module ActiveAdmin
       when Symbol, String
         render_context.public_send condition
       when Proc
-        render_context.instance_exec &condition
+        render_context.instance_exec(&condition)
       else
         true
       end

data/lib/active_admin/inputs/filters/base/search_method_select.rb

@@ -26,21 +26,19 @@ module ActiveAdmin
             attr_reader :filters
 
             def filter(*filters)
-              (@filters ||= []).push *filters
+              (@filters ||= []).push(*filters)
             end
           end
 
-          def wrapper_html_options
-            opts = super
-            (opts[:class] ||= "") << " select_and_search" unless seems_searchable?
-            opts
-          end
-
           def to_html
             input_wrapping do
-              label_html << # your label
-              select_html << # the dropdown that holds the available search methods
-              input_html # your input field
+              [
+                label_html, # your label
+                '<div class="filters-form-input-group">',
+                select_html, # the dropdown that holds the available search methods
+                input_html, # your input field
+                '</div>'
+              ].join("\n").html_safe
             end
           end
 
@@ -49,7 +47,7 @@ module ActiveAdmin
           end
 
           def select_html
-            template.select_tag "", template.options_for_select(filter_options, current_filter)
+            template.select_tag "", template.options_for_select(filter_options, current_filter), "data-search-methods": ""
           end
 
           def filters
@@ -65,7 +63,7 @@ module ActiveAdmin
 
           def filter_options
             filters.collect do |filter|
-              [I18n.t("ransack.predicates.#{filter}").capitalize, "#{method}_#{filter}"]
+              [Ransack::Translate.predicate(filter).capitalize, "#{method}_#{filter}"]
             end
           end
 

data/lib/active_admin/inputs/filters/base.rb

@@ -26,7 +26,7 @@ module ActiveAdmin
 
         def wrapper_html_options
           opts = super
-          (opts[:class] ||= "") << " filter_form_field filter_#{as}"
+          (opts[:class] ||= "") << " filters-form-field"
           opts
         end
 

data/lib/active_admin/inputs/filters/check_boxes_input.rb

@@ -10,7 +10,7 @@ module ActiveAdmin
         end
 
         def selected_values
-          @object.public_send("#{searchable_method_name}_in") || []
+          @object.public_send(:"#{searchable_method_name}_in") || []
         end
 
         def searchable_method_name
@@ -21,11 +21,6 @@ module ActiveAdmin
           end
         end
 
-        # Add whitespace before label
-        def choice_label(choice)
-          " " + super
-        end
-
         # Don't wrap in UL tag
         def choices_group_wrapping(&block)
           template.capture(&block)

data/lib/active_admin/inputs/filters/date_range_input.rb

@@ -8,8 +8,10 @@ module ActiveAdmin
         def to_html
           input_wrapping do
             [ label_html,
-              builder.text_field(gt_input_name, input_html_options_for(gt_input_name, gt_input_placeholder)),
-              builder.text_field(lt_input_name, input_html_options_for(lt_input_name, lt_input_placeholder)),
+              '<div class="filters-form-input-group">',
+              builder.date_field(gt_input_name, input_html_options_for(gt_input_name, gt_input_placeholder)),
+              builder.date_field(lt_input_name, input_html_options_for(lt_input_name, lt_input_placeholder)),
+              '</div>'
             ].join("\n").html_safe
           end
         end

data/lib/active_admin/inputs.rb

@@ -3,15 +3,12 @@ module ActiveAdmin
   module Inputs
     extend ActiveSupport::Autoload
 
-    autoload :DatepickerInput
-
     module Filters
       extend ActiveSupport::Autoload
 
       autoload :Base
       autoload :StringInput
       autoload :TextInput
-      autoload :DatePickerInput
       autoload :DateRangeInput
       autoload :NumericInput
       autoload :SelectInput

data/lib/active_admin/localizers.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
-require "active_admin/localizers/resource_localizer"
+require_relative "localizers/resource_localizer"
 
 module ActiveAdmin
   module Localizers

data/lib/active_admin/menu.rb

@@ -49,10 +49,7 @@ module ActiveAdmin
       #
       def add(options)
         options = options.dup # Make sure parameter is not modified
-        parent_chain = Array.wrap(options.delete(:parent))
-
-        item = if parent = parent_chain.shift
-                 options[:parent] = parent_chain if parent_chain.any?
+        item = if parent = options.delete(:parent)
                  (self[parent] || add(label: parent)).add options
                else
                  _add options.merge parent: self
@@ -65,7 +62,7 @@ module ActiveAdmin
 
       # Whether any children match the given item.
       def include?(item)
-        @children.values.include?(item) || @children.values.any? { |child| child.include?(item) }
+        @children.value?(item)
       end
 
       # Used in the UI to visually distinguish which menu item is selected.
@@ -73,8 +70,14 @@ module ActiveAdmin
         self == item || include?(item)
       end
 
-      def items
-        @children.values
+      # Returns sorted array of menu items that should be displayed in this context.
+      # Sorts by priority first, then alphabetically by label if needed.
+      def items(context = nil)
+        @children.values.select { |i| i.display?(context) }.sort do |a, b|
+          result = a.priority <=> b.priority
+          result = a.label(context) <=> b.label(context) if result == 0
+          result
+        end
       end
 
       attr_reader :children

data/lib/active_admin/menu_collection.rb

@@ -25,7 +25,7 @@ module ActiveAdmin
     end
 
     def exists?(menu_name)
-      @menus.keys.include? menu_name
+      @menus.key?(menu_name)
     end
 
     def fetch(menu_name)

data/lib/active_admin/menu_item.rb

@@ -1,7 +1,10 @@
 # frozen_string_literal: true
+require_relative "view_helpers/method_or_proc_helper"
+
 module ActiveAdmin
   class MenuItem
     include Menu::MenuNode
+    include MethodOrProcHelper
 
     attr_reader :html_options, :parent, :priority
 
@@ -61,10 +64,28 @@ module ActiveAdmin
       @id ||= normalize_id @dirty_id
     end
 
-    attr_reader :label
-    attr_reader :url
+    def label(context = nil)
+      render_in_context(context, @label)
+    end
+
+    def url(context = nil)
+      render_in_context(context, @url)
+    end
 
     # Don't display if the :if option passed says so
-    attr_reader :should_display
+    # Don't display if the link isn't real, we have children, and none of the children are being displayed.
+    def display?(context = nil)
+      return false unless render_in_context(context, @should_display)
+      return false if !real_url?(context) && @children.any? && !items(context).any?
+      true
+    end
+
+    private
+
+    # URL is not nil, empty, or '#'
+    def real_url?(context = nil)
+      url = url context
+      url.present? && url != '#'
+    end
   end
 end