madmin 1.2.0 → 1.2.1

data/app/views/madmin/fields/attachment/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.file_field field.to_param %>

data/app/views/madmin/fields/attachment/_show.html.erb

@@ -1,3 +1,9 @@
 <% if (attachment = field.value(record)) && attachment.attached? %>
-  <%= link_to attachment.filename, attachment, target: :_blank %>
+  <% if attachment.variable? %>
+    <%= link_to attachment, target: :_blank do %>
+      <%= image_tag attachment, class: "max-h-32" %>
+    <% end %>
+  <% else %>
+    <%= link_to attachment.filename, attachment, target: :_blank, class: "text-indigo-500 underline" %>
+  <% end %>
 <% end %>

data/app/views/madmin/fields/attachments/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.file_field field.attribute_name, multiple: true %>

data/app/views/madmin/fields/attachments/_show.html.erb

@@ -1,7 +1,11 @@
 <% if (attachments = field.value(record)) && attachments.attached? %>
   <% attachments.each do |attachment| %>
-    <div>
-      <%= link_to attachment.filename, attachment, target: :_blank %>
-    </div>
+    <% if attachment.variable? %>
+      <%= link_to attachment, target: :_blank do %>
+        <%= image_tag attachment, class: "max-h-32" %>
+      <% end %>
+    <% else %>
+      <%= link_to attachment.filename, attachment, target: :_blank, class: "text-indigo-500 underline" %>
+    <% end %>
   <% end %>
 <% end %>

data/app/views/madmin/fields/belongs_to/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
-<%= form.select field.to_param, field.options_for_select(record), { prompt: true }, { class: "form-select", data: { controller: "slimselect", data: { controller: "slimselect" } } } %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
+<%= form.select field.to_param, field.options_for_select(record), { prompt: true }, { class: "form-select", data: { controller: "select", select_url_value: field.index_path } } %>

data/app/views/madmin/fields/belongs_to/_show.html.erb

@@ -1,3 +1,3 @@
 <% if (object = field.value(record)) %>
-  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object), class: "text-indigo-500 underline" %>
 <% end %>

data/app/views/madmin/fields/boolean/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.check_box field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/date/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.text_field field.attribute_name, { class: "form-select", data: { controller: "flatpickr" } } %>

data/app/views/madmin/fields/date_time/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.text_field field.attribute_name, data: { controller: "flatpickr", flatpickr_enable_time: true } %>

data/app/views/madmin/fields/decimal/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.number_field field.attribute_name, step: :any, class: "form-input" %>

data/app/views/madmin/fields/enum/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
-<%= form.select field.attribute_name, field.options_for_select(record), { prompt: true }, { class: "form-select" } %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
+<%= form.select field.attribute_name, field.options_for_select(record), { prompt: true }, { class: "form-select", data: { controller: "select" } } %>

data/app/views/madmin/fields/float/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.number_field field.attribute_name, step: :any, class: "form-input" %>

data/app/views/madmin/fields/has_many/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
-<%= form.select "#{field.attribute_name.to_s.singularize}_ids", field.options_for_select(record), { prompt: true }, { multiple: true, class: "form-select", data: { controller: "slimselect" } } %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
+<%= form.select "#{field.attribute_name.to_s.singularize}_ids", field.options_for_select(record), { prompt: true }, { multiple: true, class: "form-select", data: { controller: "select", select_url_value: field.index_path } } %>

data/app/views/madmin/fields/has_many/_show.html.erb

@@ -1,5 +1,5 @@
 <% field.value(record).each do |object| %>
   <div>
-    <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+    <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object), class: "text-indigo-500 underline" %>
   </div>
 <% end %>

data/app/views/madmin/fields/has_one/_form.html.erb

@@ -1,3 +1,4 @@
-HAS ONE
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= field.value(record) %>

data/app/views/madmin/fields/integer/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.number_field field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/json/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.text_area field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/nested_has_many/_form.html.erb

@@ -1,4 +1,6 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block w-32 flex-shrink-0">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 
 <div class="container space-y-8" data-controller="nested-form">
   <template data-target="nested-form.template">

data/app/views/madmin/fields/nested_has_many/_show.html.erb

@@ -1,5 +1,5 @@
 <% field.value(record).each do |object| %>
   <div>
-    <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+    <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object), class: "text-indigo-500 underline" %>
   </div>
 <% end %>

data/app/views/madmin/fields/password/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.password_field field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/polymorphic/_form.html.erb

@@ -1,5 +1,7 @@
 <%= form.fields_for field.attribute_name do |pf| %>
-  <%= pf.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+  <div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+    <%= render "madmin/shared/label", form: pf, field: field %>
+  </div>
   <%= pf.select :value, field.options_for_select(record).map(&:to_global_id), { selected: field.value(record)&.to_global_id, prompt: true }, { class: "form-select", data: { controller: "slimselect" } } %>
   <%= pf.hidden_field :type, value: "polymorphic" %>
 <% end %>

data/app/views/madmin/fields/polymorphic/_show.html.erb

@@ -1,3 +1,3 @@
 <% if (object = field.value(record)) %>
-  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object), class: "text-indigo-500 underline" %>
 <% end %>

data/app/views/madmin/fields/rich_text/_form.html.erb

@@ -1,4 +1,6 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <div class="flex-1">
   <%= form.rich_text_area field.attribute_name, class: "form-input block" %>
 </div>

data/app/views/madmin/fields/string/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.text_field field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/text/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.text_area field.attribute_name, class: "form-input" %>

data/app/views/madmin/fields/time/_form.html.erb

@@ -1,2 +1,4 @@
-<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<div class="block md:inline-block md:w-32 flex-shrink-0 text-gray-700">
+  <%= render "madmin/shared/label", form: form, field: field %>
+</div>
 <%= form.time_select field.attribute_name, {}, { class: "form-select" } %>

data/app/views/madmin/shared/_label.html.erb

@@ -0,0 +1,4 @@
+<%= form.label field.attribute_name %>
+<% if field.required? %>
+  <span class="text-red-500">*</span>
+<% end %>

data/lib/generators/madmin/field/field_generator.rb

@@ -0,0 +1,31 @@
+module Madmin
+  module Generators
+    class FieldGenerator < Rails::Generators::NamedBase
+      include Madmin::GeneratorHelpers
+
+      source_root File.expand_path("../templates", __FILE__)
+
+      def eager_load
+        Rails.application.eager_load!
+      end
+
+      def generate_field
+        template "field.rb", "app/madmin/fields/#{file_path}_field.rb"
+        copy_resource_template "_form"
+        copy_resource_template "_index"
+        copy_resource_template "_show"
+      end
+
+      private
+
+      def copy_resource_template(template_name)
+        template_file = "#{template_name}.html.erb"
+
+        copy_file(
+          template_file,
+          "app/views/madmin/fields/#{file_path}_field/#{template_file}"
+        )
+      end
+    end
+  end
+end

data/lib/generators/madmin/field/templates/_form.html.erb

@@ -0,0 +1,2 @@
+<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<%= form.text_field field.attribute_name, class: "form-input" %>

data/lib/generators/madmin/field/templates/_index.html.erb

@@ -0,0 +1 @@
+<%= field.value(record) %>

data/lib/generators/madmin/field/templates/_show.html.erb

@@ -0,0 +1 @@
+<%= field.value(record) %>

data/lib/generators/madmin/field/templates/field.rb.tt

@@ -0,0 +1,26 @@
+class <%= class_name %>Field < Madmin::Field
+  # def value(record)
+  #   record.public_send(attribute_name)
+  # end
+
+  # def to_partial_path(name)
+  #   unless %w[index show form].include? name
+  #     raise ArgumentError, "`partial` must be 'index', 'show', or 'form'"
+  #  end
+  #
+  #   "/madmin/fields/#{self.class.field_type}/#{name}"
+  # end
+
+  # def to_param
+  #   attribute_name
+  # end
+
+  # # Used for checking visibility of attribute on an view
+  # def visible?(action, default: true)
+  #   options.fetch(action.to_sym, default)
+  # end
+
+  # def required?
+  #   model.validators_on(attribute_name).any? { |v| v.is_a? ActiveModel::Validations::PresenceValidator }
+  # end
+end

data/lib/generators/madmin/install/install_generator.rb

@@ -16,8 +16,13 @@ module Madmin
       end
 
       def generate_routes
+        if rails6_1_and_up?
+          route "draw :madmin", file: ROUTES_FILE[:default]
+          template("routes.rb.tt", "config/routes/madmin.rb")
+        end
+
         if route_namespace_exists?
-          route "root to: \"dashboard#show\"", indentation: 4, sentinel: /namespace :madmin do\s*\n/m
+          route "root to: \"dashboard#show\"", indentation: separated_routes_file? ? 2 : 4, sentinel: /namespace :madmin do\s*\n/m
         else
           route "root to: \"dashboard#show\"", namespace: [:madmin]
         end

data/lib/generators/madmin/install/templates/routes.rb.tt

@@ -0,0 +1,3 @@
+# Below are the routes for madmin
+namespace :madmin do
+end

data/lib/generators/madmin/resource/resource_generator.rb

@@ -20,7 +20,7 @@ module Madmin
 
       def generate_route
         if route_namespace_exists?
-          route "resources :#{plural_name}", namespace: class_path, indentation: 4, sentinel: /namespace :madmin do\s*\n/m
+          route "resources :#{plural_name}", namespace: class_path, indentation: separated_routes_file? ? 2 : 4, sentinel: /namespace :madmin do\s*\n/m
         else
           route "resources :#{plural_name}", namespace: [:madmin] + class_path
         end

data/lib/generators/madmin/resource/templates/controller.rb.tt

@@ -1,4 +1,10 @@
 module Madmin
   class <%= class_name.pluralize %>Controller < Madmin::ResourceController
+<% if class_name == "ActiveStorage::Blob" -%>
+    def new
+      super
+      @record.assign_attributes(filename: "")
+    end
+<% end -%>
   end
 end

data/lib/madmin/engine.rb

@@ -6,7 +6,7 @@ module Madmin
     end
 
     config.to_prepare do
-      Madmin.resources = []
+      Madmin.reset_resources!
     end
   end
 end

data/lib/madmin/field.rb

@@ -36,5 +36,9 @@ module Madmin
     def required?
       model.validators_on(attribute_name).any? { |v| v.is_a? ActiveModel::Validations::PresenceValidator }
     end
+
+    def searchable?
+      false
+    end
   end
 end

data/lib/madmin/fields/belongs_to.rb

@@ -2,18 +2,21 @@ module Madmin
   module Fields
     class BelongsTo < Field
       def options_for_select(record)
-        association = record.class.reflect_on_association(attribute_name)
-        klass = association.klass
-        resource = nil
-        klass.all.map do |r|
-          resource ||= Madmin.resource_for(r)
-          [resource.display_name(r), r.id]
+        if (record = record.send(attribute_name))
+          resource = Madmin.resource_for(record)
+          [[resource.display_name(record), record.id]]
+        else
+          []
         end
       end
 
       def to_param
         "#{attribute_name}_id"
       end
+
+      def index_path
+        Madmin.resource_by_name(model.reflect_on_association(attribute_name).klass).index_path(format: :json)
+      end
     end
   end
 end

data/lib/madmin/fields/has_many.rb

@@ -2,18 +2,22 @@ module Madmin
   module Fields
     class HasMany < Field
       def options_for_select(record)
-        association = record.class.reflect_on_association(attribute_name)
-        klass = association.klass
-        resource = nil
-        klass.all.map do |r|
-          resource ||= Madmin.resource_for(r)
-          [resource.display_name(r), r.id]
+        if (records = record.send(attribute_name))
+          return [] unless records.first
+          resource = Madmin.resource_for(records.first)
+          records.map { |record| [resource.display_name(record), record.id] }
+        else
+          []
         end
       end
 
       def to_param
         {"#{attribute_name.to_s.singularize}_ids".to_sym => []}
       end
+
+      def index_path
+        Madmin.resource_by_name(model.reflect_on_association(attribute_name).klass).index_path(format: :json)
+      end
     end
   end
 end

data/lib/madmin/fields/string.rb

@@ -1,6 +1,9 @@
 module Madmin
   module Fields
     class String < Field
+      def searchable?
+        options.fetch(:searchable, model.column_names.include?(attribute_name.to_s))
+      end
     end
   end
 end

data/lib/madmin/fields/text.rb

@@ -1,6 +1,9 @@
 module Madmin
   module Fields
     class Text < Field
+      def searchable?
+        options.fetch(:searchable, model.column_names.include?(attribute_name.to_s))
+      end
     end
   end
 end

data/lib/madmin/generator_helpers.rb

@@ -1,24 +1,30 @@
 module Madmin
   module GeneratorHelpers
+    ROUTES_FILE = {default: "config/routes.rb", separated: "config/routes/madmin.rb"}.freeze
+
     def call_generator(generator, *args)
       Rails::Generators.invoke(generator, args, generator_options)
     end
 
     def route_namespace_exists?
-      File.readlines(Rails.root.join("config/routes.rb")).grep(/namespace :madmin/).size > 0
+      File.readlines(Rails.root.join(default_routes_file)).grep(/namespace :madmin/).size > 0
+    end
+
+    def rails6_1_and_up?
+      Gem.loaded_specs["rails"].version >= Gem::Version.new(6.1)
     end
 
     # Method copied from Rails 6.1 master
-    def route(routing_code, namespace: nil, sentinel: nil, indentation: 2)
+    def route(routing_code, namespace: nil, sentinel: nil, indentation: 2, file: default_routes_file)
       routing_code = Array(namespace).reverse.reduce(routing_code) { |code, ns|
         "namespace :#{ns} do\n#{indent(code, 2)}\nend"
       }
 
       log :route, routing_code
-      sentinel ||= /\.routes\.draw do\s*\n/m
+      sentinel ||= default_sentinel(file)
 
       in_root do
-        inject_into_file "config/routes.rb", optimize_indentation(routing_code, indentation), after: sentinel, verbose: false, force: false
+        inject_into_file file, optimize_indentation(routing_code, indentation), after: sentinel, verbose: false, force: false
       end
     end
 
@@ -30,6 +36,18 @@ module Madmin
 
     private
 
+    def separated_routes_file?
+      default_routes_file.eql?(ROUTES_FILE[:separated])
+    end
+
+    def default_sentinel(file)
+      file.eql?(ROUTES_FILE[:default]) ? /\.routes\.draw do\s*\n/m : /namespace :madmin do\s*\n/m
+    end
+
+    def default_routes_file
+      rails6_1_and_up? ? ROUTES_FILE[:separated] : ROUTES_FILE[:default]
+    end
+
     def generator_options
       {behavior: behavior}
     end