motor-admin 0.1.32 → 0.1.37

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9db6c3263064796fde708305f78fbc33db762534890eeeb79ec13e2954b8b54b
-  data.tar.gz: 462878ba2c5c21c77af910b27be67a1a8b7518ba73fd2b239c3309f88e9ec49a
+  metadata.gz: c003af4059ae5bd3a60d30031526c1ed2aefd267c7e1f93f33e5bac94eaff5c7
+  data.tar.gz: aa7403bca65f0d936b86bcc58f1fe88b4081439d74843f1ca1c3d25fd4da8a81
 SHA512:
-  metadata.gz: 457cfc674fa8845a9a1e21e6177a38a9159c92ad021ffb6af2504eb1df326cf5325efad85efed341082e9c3e3a863aba408d456626f5da167671e60af93f3416
-  data.tar.gz: e13a4a74752db108786a9625d3ea3e851e6bdcc4a7b6ac55f041d3cd753b5993f61e8d825038846603718483733888258e734d2fbc204b27b0149b2c751d0b46
+  metadata.gz: a195dd04aceadc74e1d7fa89dc0100c15fc045c0fad238230567069801698d341d6a9e598b861ae9e802e61dd5a48915d15c74b88a1228deae5a985fad283762
+  data.tar.gz: 7d6dc23a2750dc941b20d83170dc3f2eb3018d02897b2500b99831285ac139fcde46ebfa7ae044be856504bac6f09e1d11e70c02a443eba6d70278b959784f74

data/app/controllers/concerns/motor/load_and_authorize_dynamic_resource.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module Motor
+  module LoadAndAuthorizeDynamicResource
+    extend ActiveSupport::Concern
+
+    INSTANCE_VARIABLE_NAME = 'resource'
+
+    included do
+      before_action :load_and_authorize_resource
+      before_action :load_and_authorize_association
+    end
+
+    def resource_class
+      @resource_class ||=
+        Motor::BuildSchema::Utils.classify_slug(resource_name_prefix + params[:resource])
+    end
+
+    def resource_name_prefix
+      ''
+    end
+
+    def load_and_authorize_resource
+      options = {
+        class: resource_class,
+        parent: false,
+        instance_name: INSTANCE_VARIABLE_NAME
+      }
+
+      if params[:resource_id].present?
+        options = options.merge(
+          parent: true,
+          id_param: :resource_id
+        )
+      end
+
+      CanCan::ControllerResource.new(
+        self,
+        options
+      ).load_and_authorize_resource
+    rescue ActiveRecord::RecordNotFound
+      head :not_found
+    rescue StandardError => e
+      render json: { errors: [e.message] }, status: :unprocessable_entity
+    end
+
+    def load_and_authorize_association
+      return if params[:association].blank?
+
+      association = resource_class.reflections[params[:association]]
+
+      if association
+        CanCan::ControllerResource.new(
+          self,
+          class: association.klass,
+          parent: false,
+          through: :resource,
+          through_association: params[:association].to_sym,
+          instance_name: INSTANCE_VARIABLE_NAME
+        ).load_and_authorize_resource
+      else
+        render json: { message: 'Unknown association' }, status: :not_found
+      end
+    rescue ActiveRecord::RecordNotFound
+      head :not_found
+    rescue StandardError => e
+      render json: { errors: [e.message] }, status: :unprocessable_entity
+    end
+  end
+end

data/app/controllers/motor/audits_controller.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Motor
+  class AuditsController < ApiBaseController
+    load_and_authorize_resource :audit
+
+    def index
+      audits = Motor::ApiQuery.call(@audits, params)
+
+      render json: {
+        data: Motor::ApiQuery::BuildJson.call(audits, params),
+        meta: Motor::ApiQuery::BuildMeta.call(audits, params)
+      }
+    end
+  end
+end

data/app/controllers/motor/data_controller.rb

@@ -2,14 +2,10 @@
 
 module Motor
   class DataController < ApiBaseController
-    include Motor::WrapIoParams
-
-    INSTANCE_VARIABLE_NAME = 'resource'
-
     wrap_parameters :data, except: %i[include fields]
 
-    before_action :load_and_authorize_resource
-    before_action :load_and_authorize_association
+    include Motor::WrapIoParams
+    include Motor::LoadAndAuthorizeDynamicResource
 
     def index
       @resources = Motor::ApiQuery.call(@resources, params)
@@ -64,53 +60,6 @@ module Motor
 
     private
 
-    def resource_class
-      @resource_class ||= Motor::BuildSchema::Utils.classify_slug(params[:resource])
-    end
-
-    def load_and_authorize_resource
-      options = {
-        class: resource_class,
-        parent: false,
-        instance_name: INSTANCE_VARIABLE_NAME
-      }
-
-      if params[:resource_id].present?
-        options = options.merge(
-          parent: true,
-          id_param: :resource_id
-        )
-      end
-
-      CanCan::ControllerResource.new(
-        self,
-        options
-      ).load_and_authorize_resource
-    rescue StandardError => e
-      render json: { errors: [e.message] }, status: :unprocessable_entity
-    end
-
-    def load_and_authorize_association
-      return if params[:association].blank?
-
-      association = resource_class.reflections[params[:association]]
-
-      if association
-        CanCan::ControllerResource.new(
-          self,
-          class: association.klass,
-          parent: false,
-          through: :resource,
-          through_association: params[:association].to_sym,
-          instance_name: INSTANCE_VARIABLE_NAME
-        ).load_and_authorize_resource
-      else
-        render json: { message: 'Unknown association' }, status: :not_found
-      end
-    rescue StandardError => e
-      render json: { errors: [e.message] }, status: :unprocessable_entity
-    end
-
     def resource_params
       if params[:data].present?
         params.require(:data).except(resource_class.primary_key).permit!

data/app/models/motor/alert.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Alert < ::Motor::ApplicationRecord
+    audited
+
     belongs_to :query
     belongs_to :author, polymorphic: true, optional: true
 

data/app/models/motor/application_record.rb

@@ -4,5 +4,15 @@ module Motor
   class ApplicationRecord < ActiveRecord::Base
     self.abstract_class = true
     self.table_name_prefix = 'motor_'
+
+    def self.audited(*args)
+      default_class = Audited.audit_class
+
+      Audited.audit_class = Motor::Audit
+
+      super
+    ensure
+      Audited.audit_class = default_class
+    end
   end
 end

data/app/models/motor/audit.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module Motor
+  class Audit < Audited::Audit
+    self.table_name = 'motor_audits'
+
+    serialize :audited_changes, HashSerializer
+  end
+end

data/app/models/motor/config.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Config < ::Motor::ApplicationRecord
+    audited
+
     serialize :value, HashSerializer
   end
 end

data/app/models/motor/dashboard.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Dashboard < ::Motor::ApplicationRecord
+    audited
+
     belongs_to :author, polymorphic: true, optional: true
 
     has_many :taggable_tags, as: :taggable

data/app/models/motor/form.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Form < ::Motor::ApplicationRecord
+    audited
+
     belongs_to :author, polymorphic: true, optional: true
 
     has_many :taggable_tags, as: :taggable

data/app/models/motor/query.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Query < ::Motor::ApplicationRecord
+    audited
+
     belongs_to :author, polymorphic: true, optional: true
 
     has_many :taggable_tags, as: :taggable
@@ -10,5 +12,13 @@ module Motor
     serialize :preferences, HashSerializer
 
     scope :active, -> { where(deleted_at: nil) }
+
+    def result(variables_hash = {})
+      result = Motor::Queries::RunQuery.call(self, variables_hash: variables_hash)
+      column_names = result.columns.pluck(:name)
+
+      result.data.map { |row| column_names.zip(row).to_h }
+    end
+    alias run result
   end
 end

data/app/models/motor/resource.rb

@@ -2,6 +2,8 @@
 
 module Motor
   class Resource < ::Motor::ApplicationRecord
+    audited
+
     serialize :preferences, HashSerializer
   end
 end

data/config/routes.rb

@@ -15,6 +15,7 @@ Motor::Admin.routes.draw do
       resources :alerts, only: %i[index show create update destroy]
       resources :active_storage_attachments, only: %i[create], path: 'data/active_storage__attachments'
       resource :schema, only: %i[show update]
+      resources :audits, only: %i[index]
       resources :resources, path: '/data/:resource',
                             only: %i[index show update create destroy],
                             controller: 'data' do

data/lib/generators/motor/templates/install.rb

@@ -13,9 +13,9 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
 
       t.index :updated_at
       t.index 'lower(name)',
-        name: 'motor_queries_lower_name_unique_index',
-        unique: true,
-        where: 'deleted_at IS NULL'
+              name: 'motor_queries_lower_name_unique_index',
+              unique: true,
+              where: 'deleted_at IS NULL'
     end
 
     create_table :motor_dashboards, force: true do |t|
@@ -30,9 +30,9 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
 
       t.index :updated_at
       t.index 'lower(title)',
-        name: 'motor_dashboards_lower_title_unique_index',
-        unique: true,
-        where: 'deleted_at IS NULL'
+              name: 'motor_dashboards_lower_title_unique_index',
+              unique: true,
+              where: 'deleted_at IS NULL'
     end
 
     create_table :motor_forms, force: true do |t|
@@ -49,9 +49,9 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
 
       t.index :updated_at
       t.index 'lower(name)',
-        name: 'motor_forms_lower_name_unique_index',
-        unique: true,
-        where: 'deleted_at IS NULL'
+              name: 'motor_forms_lower_name_unique_index',
+              unique: true,
+              where: 'deleted_at IS NULL'
     end
 
     create_table :motor_resources, force: true do |t|
@@ -87,9 +87,9 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
 
       t.index :updated_at
       t.index 'lower(name)',
-        name: 'motor_alerts_lower_name_unique_index',
-        unique: true,
-        where: 'deleted_at IS NULL'
+              name: 'motor_alerts_lower_name_unique_index',
+              unique: true,
+              where: 'deleted_at IS NULL'
     end
 
     create_table :motor_alert_locks, force: true do |t|
@@ -107,8 +107,8 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
       t.timestamps
 
       t.index 'lower(name)',
-        name: 'motor_tags_lower_name_unique_index',
-        unique: true
+              name: 'motor_tags_lower_name_unique_index',
+              unique: true
     end
 
     create_table :motor_taggable_tags, force: true do |t|
@@ -117,12 +117,36 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
       t.column :taggable_type, :string, null: false
 
       t.index %i[taggable_id taggable_type tag_id],
-        name: 'motor_polymorphic_association_tag_index',
-        unique: true
+              name: 'motor_polymorphic_association_tag_index',
+              unique: true
     end
+
+    create_table :motor_audits, force: true do |t|
+      t.column :auditable_id, :integer
+      t.column :auditable_type, :string
+      t.column :associated_id, :integer
+      t.column :associated_type, :string
+      t.column :user_id, :integer
+      t.column :user_type, :string
+      t.column :username, :string
+      t.column :action, :string
+      t.column :audited_changes, :text
+      t.column :version, :integer, default: 0
+      t.column :comment, :string
+      t.column :remote_address, :string
+      t.column :request_uuid, :string
+      t.column :created_at, :datetime
+    end
+
+    add_index :motor_audits, %i[auditable_type auditable_id version], name: 'motor_auditable_index'
+    add_index :motor_audits, %i[associated_type associated_id], name: 'motor_auditable_associated_index'
+    add_index :motor_audits, %i[user_id user_type], name: 'motor_auditable_user_index'
+    add_index :motor_audits, :request_uuid
+    add_index :motor_audits, :created_at
   end
 
   def self.down
+    drop_table :motor_audits
     drop_table :motor_alert_locks
     drop_table :motor_alerts
     drop_table :motor_taggable_tags

data/lib/motor.rb

@@ -6,6 +6,7 @@ require 'js_regex'
 require 'fugit'
 require 'csv'
 require 'active_record/filter'
+require 'audited'
 
 module Motor
   PATH = Pathname.new(__dir__)
@@ -45,7 +46,6 @@ end
 
 require 'motor/version'
 require 'motor/admin'
-# require 'motor/api'
 require 'motor/assets'
 require 'motor/build_schema'
 require 'motor/api_query'

data/lib/motor/admin.rb

@@ -15,9 +15,16 @@ module Motor
             end
 
           puts
-          puts "⚡ Motor Admin is starting under #{url}"
+          puts "⚡ Motor::Admin is starting under #{url}"
+        else
           puts
+          puts '⚠️  Motor::Admin is not mounted.'
+          puts 'Add the following line to your config/routes.rb:'
+          puts
+          puts "    mount Motor::Admin => '/admin'"
         end
+
+        puts
       end
     end
 

data/lib/motor/api_query/sort.rb

@@ -13,7 +13,7 @@ module Motor
         arel_order = build_arel_order(rel.klass, param)
         join_params = build_join_params(rel.klass, param)
 
-        rel.order(arel_order).left_joins(join_params)
+        rel.reorder(arel_order).left_joins(join_params)
       end
 
       def build_join_params(_model, param)

data/lib/motor/build_schema.rb

@@ -48,8 +48,8 @@ module Motor
 
     DEFAULT_TABS = [
       {
-        name: 'summary',
-        display_name: 'Summary',
+        name: 'details',
+        display_name: 'Details',
         tab_type: 'default',
         preferences: {},
         visible: true

data/lib/motor/build_schema/load_from_rails.rb

@@ -11,7 +11,7 @@ module Motor
         models.map do |model|
           build_model_schema(model)
         rescue StandardError, NotImplementedError => e
-          Rails.logger.error(e)
+          Rails.logger.error(e) if model.name != 'Audited::Audit'
 
           next
         end.compact
@@ -24,6 +24,7 @@ module Motor
         models = models.reject(&:abstract_class)
 
         models -= Motor::ApplicationRecord.descendants
+        models -= [Motor::Audit]
         models -= [ActiveRecord::SchemaMigration] if defined?(ActiveRecord::SchemaMigration)
         models -= [ActiveStorage::Blob] if defined?(ActiveStorage::Blob)
         models -= [ActiveStorage::VariantRecord] if defined?(ActiveStorage::VariantRecord)

data/lib/motor/tags.rb

@@ -21,7 +21,8 @@ module Motor
     end
 
     def remove_missing_tags(taggable, tags)
-      tags_to_remove = taggable.tags.reject { |tt| tt.name.downcase.in?(tags) }
+      downcase_tags = tags.map(&:downcase)
+      tags_to_remove = taggable.tags.reject { |tt| tt.name.downcase.in?(downcase_tags) }
 
       taggable.tags -= tags_to_remove
 

data/lib/motor/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Motor
-  VERSION = '0.1.32'
+  VERSION = '0.1.37'
 end

data/ui/dist/manifest.json

@@ -5,9 +5,9 @@
   "fonts/ionicons.ttf?v=3.0.0-alpha.3": "fonts/ionicons.ttf",
   "fonts/ionicons.woff2?v=3.0.0-alpha.3": "fonts/ionicons.woff2",
   "fonts/ionicons.woff?v=3.0.0-alpha.3": "fonts/ionicons.woff",
-  "main-eea896c6f0ede15495f0.css.gz": "main-eea896c6f0ede15495f0.css.gz",
-  "main-eea896c6f0ede15495f0.js.LICENSE.txt": "main-eea896c6f0ede15495f0.js.LICENSE.txt",
-  "main-eea896c6f0ede15495f0.js.gz": "main-eea896c6f0ede15495f0.js.gz",
-  "main.css": "main-eea896c6f0ede15495f0.css",
-  "main.js": "main-eea896c6f0ede15495f0.js"
+  "main-358ea31cd7020f915067.css.gz": "main-358ea31cd7020f915067.css.gz",
+  "main-358ea31cd7020f915067.js.LICENSE.txt": "main-358ea31cd7020f915067.js.LICENSE.txt",
+  "main-358ea31cd7020f915067.js.gz": "main-358ea31cd7020f915067.js.gz",
+  "main.css": "main-358ea31cd7020f915067.css",
+  "main.js": "main-358ea31cd7020f915067.js"
 }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: motor-admin
 version: !ruby/object:Gem::Version
-  version: 0.1.32
+  version: 0.1.37
 platform: ruby
 authors:
 - Pete Matsyburka
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-11 00:00:00.000000000 Z
+date: 2021-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord-filter
@@ -38,6 +38,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.6.0
+- !ruby/object:Gem::Dependency
+  name: audited
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.9'
 - !ruby/object:Gem::Dependency
   name: cancancan
   requirement: !ruby/object:Gem::Requirement
@@ -132,12 +146,14 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- app/controllers/concerns/motor/load_and_authorize_dynamic_resource.rb
 - app/controllers/concerns/motor/wrap_io_params.rb
 - app/controllers/motor/active_storage_attachments_controller.rb
 - app/controllers/motor/alerts_controller.rb
 - app/controllers/motor/api_base_controller.rb
 - app/controllers/motor/application_controller.rb
 - app/controllers/motor/assets_controller.rb
+- app/controllers/motor/audits_controller.rb
 - app/controllers/motor/configs_controller.rb
 - app/controllers/motor/dashboards_controller.rb
 - app/controllers/motor/data_controller.rb
@@ -157,6 +173,7 @@ files:
 - app/models/motor/alert.rb
 - app/models/motor/alert_lock.rb
 - app/models/motor/application_record.rb
+- app/models/motor/audit.rb
 - app/models/motor/config.rb
 - app/models/motor/dashboard.rb
 - app/models/motor/form.rb
@@ -214,8 +231,8 @@ files:
 - lib/motor/ui_configs.rb
 - lib/motor/version.rb
 - ui/dist/fonts/ionicons.woff2
-- ui/dist/main-eea896c6f0ede15495f0.css.gz
-- ui/dist/main-eea896c6f0ede15495f0.js.gz
+- ui/dist/main-358ea31cd7020f915067.css.gz
+- ui/dist/main-358ea31cd7020f915067.js.gz
 - ui/dist/manifest.json
 homepage:
 licenses: