madmin 0.1.0 → 0.1.1

data/app/views/madmin/application/new.html.erb

@@ -0,0 +1,3 @@
+<h1><%= link_to resource.friendly_name.pluralize, resource.index_path %> / New <%= resource.friendly_name %></h1>
+
+<%= render partial: "form", locals: { record: @record, resource: resource } %>

data/app/views/madmin/application/show.html.erb

@@ -0,0 +1,24 @@
+<div class="flex justify-between">
+  <h1><%= link_to resource.friendly_name.pluralize, resource.index_path %> / <%= resource.model.name.singularize %> #<%= @record.id %></h1>
+
+  <div class="flex px-4">
+    <%= link_to "Edit", resource.edit_path(@record), class: "mr-2" %>
+    <%= button_to "Delete", resource.show_path(@record), method: :delete, data: { confirm: "Are you sure?" }, class: "inline-block" %>
+  </div>
+</div>
+
+<% resource.attributes.each do |attribute| %>
+  <% next if attribute[:field].nil? %>
+  <% next unless attribute[:field].visible?(action_name) %>
+  <% field = attribute[:field] %>
+
+  <div class="flex py-2">
+    <div class="w-32 flex-shrink-0 text-gray-700 uppercase tracking-wide text-sm">
+      <%= field.attribute_name.to_s.titleize %>
+    </div>
+
+    <div>
+      <%= render partial: field.to_partial_path("show"), locals: { field: field, record: @record } %>
+    </div>
+  </div>
+<% end %>

data/app/views/madmin/dashboard/show.html.erb

@@ -0,0 +1 @@
+<h1>Madmin</h1>

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

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

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

@@ -0,0 +1,3 @@
+<% if (attachment = field.value(record)) && attachment.attached? %>
+  <%= link_to attachment.filename, attachment, target: :_blank %>
+<% end %>

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

@@ -0,0 +1,3 @@
+<% if (attachment = field.value(record)) && attachment.attached? %>
+  <%= link_to attachment.filename, attachment, target: :_blank %>
+<% end %>

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

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

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

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

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

@@ -0,0 +1,7 @@
+<% if (attachments = field.value(record)) && attachments.attached? %>
+  <% attachments.each do |attachment| %>
+    <div>
+      <%= link_to attachment.filename, attachment, target: :_blank %>
+    </div>
+  <% end %>
+<% end %>

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

@@ -1,14 +1,2 @@
-<div class="form-group">
-  <%= form.label field.key %>
-  <%=
-    form.select field.association_foreign_key,
-                options_from_collection_for_select(
-                  field.association_collection,
-                  :id,
-                  field.association_display_value,
-                  field.association_id_or_blank_for(resource)
-                ),
-                { include_blank: true },
-                class: 'form-control'
-  %>
-</div>
+<%= 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/app/views/madmin/fields/belongs_to/_index.html.erb

@@ -1,7 +1,2 @@
-<%=
-  link_to field.association_value_for(resource),
-          resource_path(
-            field.association_slug,
-            field.association_id_for(resource)
-          )
-%>
+<% object = field.value(record) %>
+<%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>

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

@@ -1,8 +1,2 @@
-<div class="row">
-  <div class="col-3">
-    <%= field.label %>
-  </div>
-  <div class="col-9">
-    <%= field.association_value_for(resource) %>
-  </div>
-</div>
+<% object = field.value(record) %>
+<%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

@@ -0,0 +1,3 @@
+ENUM
+<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<%= field.value(record) %>

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

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

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

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

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

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

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

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

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

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

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

@@ -0,0 +1,2 @@
+<%= 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/app/views/madmin/fields/has_many/_index.html.erb

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

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

@@ -1,15 +1,5 @@
-<div class="mt-3">
-  <div class="row">
-    <div class="col">
-      <h3><%= field.label %></h3>
-      <%= link_to "Add", new_resource_path(field.association_slug, field.association_param => { field.association_foreign_key => resource.id }) %>
-      <%=
-        render "madmin/resources/index/content",
-               headers: [],
-               collection: field.value_for(resource),
-               madmin_resource: madmin_resource_for(model: field.association_class),
-               show_new_button: false
-      %>
-    </div>
+<% field.value(record).each do |object| %>
+  <div>
+    <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
   </div>
-</div>
+<% end %>

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

@@ -0,0 +1,3 @@
+HAS ONE
+<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<%= field.value(record) %>

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

@@ -0,0 +1,3 @@
+<% if (object = field.value(record)) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+<% end %>

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

@@ -1,12 +1,3 @@
-<div class="mt-3">
-  <div class="row">
-    <div class="col">
-      <h3><%= field.name %></h3>
-      <% if !field.value %>
-        <%= link_to "Add", new_resource_path(madmin_resource_for(model: field.foreign_class).slug, ActiveModel::Naming.param_key(field.foreign_class) => { field.foreign_key => field.resource.id }) %>
-      <% else %>
-        <%= render "madmin/resources/index/content", headers: [], collection: [Madmin::ResourceDecorator.new(field.value)], show_new_button: false %>
-      <% end %>
-    </div>
-  </div>
-</div>
+<% if (object = field.value(record)) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+<% end %>

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

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

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

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

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

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

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

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

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

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

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

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

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

@@ -1,32 +1,5 @@
-<div class="form-group">
-  <%= form.label field.key %>
-  <%=
-    form.select field.polymorphic_type_param,
-                polymorphic_models(field.key).map { |m| [m.name, m.name, { "data-slug" => madmin_resource_for(model: m).slug }] },
-                { include_blank: true },
-                id: field.polymorphic_type_param,
-                class: "form-control"
-  %>
-  <% if field.polymorphic_relationship_exists?(form.object) %>
-    <%=
-      form.select field.polymorphic_id_param,
-                  polymorphic_options_for_selected_type(form: form, field: field),
-                  {},
-                  id: field.polymorphic_id_param,
-                  class: "form-control"
-    %>
-  <% else %>
-    <%=
-      form.select field.polymorphic_id_param,
-                  [],
-                  {},
-                  id: field.polymorphic_id_param,
-                  class: "d-none form-control"
-    %>
-  <% end %>
-</div>
-
-<script>
-  window.polymorphicFields = window.polymorphicFields || [];
-  window.polymorphicFields.push('<%= field.key %>')
-</script>
+<%= form.fields_for field.attribute_name do |pf| %>
+  <%= pf.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+  <%= pf.select :value, field.options_for_select(record).map(&:to_global_id), { selected: field.value(record)&.to_global_id, prompt: true }, { class: "form-select" } %>
+  <%= pf.hidden_field :type, value: "polymorphic" %>
+<% end %>

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

@@ -1 +1,3 @@
-<%= link_to field.polymorphic_value_for(resource), resource_path(field.polymorphic_slug_for(resource), field.polymorphic_id_for(resource)) %>
+<% if (object = field.value(record)) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+<% end %>

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

@@ -1,14 +1,3 @@
-<div class="row">
-  <div class="col-3">
-    <%= field.label %>
-  </div>
-  <div class="col-9">
-    <%=
-      link_to field.polymorphic_value_for(resource),
-              resource_path(
-                field.polymorphic_slug_for(resource),
-                field.polymorphic_id_for(resource)
-              )
-    %>
-  </div>
-</div>
+<% if (object = field.value(record)) %>
+  <%= link_to Madmin.resource_for(object).display_name(object), Madmin.resource_for(object).show_path(object) %>
+<% end %>

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

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

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

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

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

@@ -0,0 +1,3 @@
+<div class="prose">
+  <%= field.value(record) %>
+</div>

data/app/views/madmin/fields/string/_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/app/views/madmin/fields/string/_index.html.erb

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

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

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

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

@@ -1,4 +1,2 @@
-<div class="form-group">
-  <%= form.label field.key, field.label %>
-  <%= form.text_field field.key, class: "form-control" %>
-</div>
+<%= form.label field.attribute_name, class: "inline-block w-32 flex-shrink-0" %>
+<%= form.text_area field.attribute_name, class: "form-input" %>

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

@@ -1 +1 @@
-<%= field.value_for(resource) %>
+<%= field.value(record) %>

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

@@ -1,8 +1 @@
-<div class="row">
-  <div class="col-3">
-    <%= field.label %>
-  </div>
-  <div class="col-9">
-    <%= field.value_for(resource) %>
-  </div>
-</div>
+<%= field.value(record) %>

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

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

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

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

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

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

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

@@ -1,4 +1,3 @@
-require "rails/generators/base"
 require "madmin/generator_helpers"
 
 module Madmin
@@ -6,25 +5,38 @@ module Madmin
     class InstallGenerator < Rails::Generators::Base
       include Madmin::GeneratorHelpers
 
-      def generate_resources
-        resources.each do |model|
-          call_generator "madmin:resource", model.to_s
+      def eager_load
+        Rails.application.eager_load!
+      end
+
+      def generate_routes
+        if route_namespace_exists?
+          route "root to: \"dashboard#show\"", indentation: 4, sentinel: /namespace :madmin do\s*\n/m
+        else
+          route "root to: \"dashboard#show\"", namespace: [:madmin]
         end
       end
 
-      def install_route
-        inject_into_file(
-          Rails.root.join("config/routes.rb"),
-          after: "Rails.application.routes.draw do\n"
-        ) { "  namespace :madmin do\n  end\n  mount Madmin::Engine => \"/madmin\"\n" }
+      def generate_resources
+        generateable_models.each do |model|
+          if model.table_exists?
+            call_generator "madmin:resource", model.to_s
+          else
+            puts "Skipping #{model} because database table does not exist"
+          end
+        end
       end
 
       private
 
-      def resources
-        Rails.application.eager_load! unless Rails.application.config.cache_classes
+      def generateable_models
+        active_record_models.reject do |model|
+          model.abstract_class? || model == ActiveRecord::Base
+        end
+      end
 
-        ActiveRecord::Base.descendants.reject(&:abstract_class?)
+      def active_record_models
+        ObjectSpace.each_object(ActiveRecord::Base.singleton_class)
       end
     end
   end

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

@@ -1,75 +1,111 @@
-require "rails/generators/named_base"
-require "madmin/generator_helpers"
-
 module Madmin
   module Generators
     class ResourceGenerator < Rails::Generators::NamedBase
-      ATTRIBUTE_TYPE_MAPPING = {
-        boolean: "Field::CheckBox",
-        date: "Field::DateTime",
-        datetime: "Field::DateTime",
-        enum: "Field::Text",
-        float: "Field::Number",
-        integer: "Field::Number",
-        time: "Field::DateTime",
-        text: "Field::TextArea",
-        string: "Field::Text",
-      }
+      include Madmin::GeneratorHelpers
 
       source_root File.expand_path("../templates", __FILE__)
 
-      def create_resource_file
-        template(
-          "resource.rb.erb",
-          Rails.root.join("app/madmin/resources/#{file_name}.rb"),
-        )
+      def eager_load
+        Rails.application.eager_load!
       end
 
-      private
+      def generate_resource
+        template "resource.rb", "app/madmin/resources/#{file_path}_resource.rb"
+      end
 
-      def attributes
-        klass.attribute_types.map { |name, attr|
-          # Skip attributes related to associations
-          next if ignored_attributes.include?(name)
+      def generate_controller
+        destination = Rails.root.join("app/controllers/madmin/#{file_path.pluralize}_controller.rb")
+        template("controller.rb", destination)
+      end
 
-          [name, madmin_type_for_column(attr.type)]
-        }.compact
+      def generate_route
+        if route_namespace_exists?
+          route "resources :#{plural_name}", namespace: class_path, indentation: 4, sentinel: /namespace :madmin do\s*\n/m
+        else
+          route "resources :#{plural_name}", namespace: [:madmin] + class_path
+        end
       end
 
+      private
+
       def associations
-        klass.reflections.map do |name, association|
-          if association.polymorphic?
-            [name, "Field::Polymorphic"]
-          elsif association.belongs_to?
-            [name, "Field::BelongsTo"]
-          elsif association.has_one?
-            [name, "Field::HasOne"]
-          else
-            [name, "Field::HasMany"]
+        model.reflections.reject { |name, association|
+          # Hide these special associations
+          name.starts_with?("rich_text") ||
+            name.ends_with?("_attachment") ||
+            name.ends_with?("_attachments") ||
+            name.ends_with?("_blob") ||
+            name.ends_with?("_blobs")
+        }.keys
+      end
+
+      def attributes
+        model.attribute_names + virtual_attributes - redundant_attributes
+      end
+
+      def virtual_attributes
+        virtual = []
+
+        # Add virtual attributes for ActionText and ActiveStorage
+        model.reflections.each do |name, association|
+          if name.starts_with?("rich_text")
+            virtual << name.split("rich_text_").last
+          elsif name.ends_with?("_attachment")
+            virtual << name.split("_attachment").first
+          elsif name.ends_with?("_attachments")
+            virtual << name.split("_attachments").first
           end
         end
+
+        virtual
       end
 
-      def ignored_attributes
-        attrs = []
+      def redundant_attributes
+        redundant = []
 
-        klass.reflections.map do |name, association|
-          if association.polymorphic?
-            attrs += [association.foreign_key, association.foreign_type]
-          elsif association.belongs_to?
-            attrs += [association.foreign_key]
+        model.reflections.each do |name, association|
+          if association.has_one?
+            next
+          elsif association.collection?
+            next
+          elsif association.polymorphic?
+            redundant << "#{name}_id"
+            redundant << "#{name}_type"
+          elsif name.starts_with?("rich_text")
+            redundant << name
+          else # belongs to
+            redundant << "#{name}_id"
           end
         end
 
-        attrs
+        redundant
+      end
+
+      def model
+        @model ||= class_name.constantize
       end
 
-      def klass
-        @klass ||= Object.const_get(class_name)
+      def formatted_options_for_attribute(name)
+        options = options_for_attribute(name)
+        return if options.blank?
+
+        ", " + options.map { |key, value|
+          "#{key}: #{value}"
+        }.join(", ")
       end
 
-      def madmin_type_for_column(column_type)
-        ATTRIBUTE_TYPE_MAPPING[column_type]
+      def options_for_attribute(name)
+        if %w[id created_at updated_at].include?(name)
+          {form: false}
+
+        # Attributes without a database column
+        elsif !model.column_names.include?(name)
+          {index: false}
+
+        # Counter cache columns are typically not editable
+        elsif name.ends_with?("_count")
+          {form: false}
+        end
       end
     end
   end