motor-admin 0.1.49 → 0.1.50

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11b39999a22bc50fe6e3dbcd11765207acaf301b61c0e914afe3a7e553216446
-  data.tar.gz: 4d39e9ae7ce4df7661008d1db9346f99daaeb3143ae22076ba26032a42bece36
+  metadata.gz: 9e5a06d8a42714fbc30464ef704366dc9f3b851df80bd8c9a206a2ed75a6fd23
+  data.tar.gz: 13d46e2d091be7a9b6e9832ab1e036c08734bbe70b0b92b64f1b41dd0f355a17
 SHA512:
-  metadata.gz: 784ae2079cafceacf3ec1a8cf8b8796e515afe5411dd065f6b976ed25d7f0041de3603aa3ca1ad768e7a4c5d1259fc25e2bdd459db5003edb46db5911f2d07dc
-  data.tar.gz: cf5177b94de2da402d9122c339d379a95a9b71d52bcdbb4c807710243dc80c0d5b4d05673d4ed5c5385f3c42e5f6e6c39553df12c1de35e27876c7b69d872542
+  metadata.gz: 544edcef7fd6e97e1285fff69f1984a0991dc72a1768ebace132faad1a1b6609e68dd7c0ae4e6d573036c171a1b364a65fc52a9f63e653f51e4f33b8b86281ec
+  data.tar.gz: 2cb5fb5c3c47710cfea97550e0f0ac67c2463eb3d71a5a62a8ce6fbff62763e04274d92b6c4c6bc51e2f14e2a4fd84a6d9f290b7586fca407b85a1aa254fa47d

data/app/controllers/motor/assets_controller.rb

@@ -2,7 +2,7 @@
 
 module Motor
   class AssetsController < ActionController::Metal
-    CACHE_STORE = ActiveSupport::Cache::MemoryStore.new
+    CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(coder: ActiveSupport::Cache::NullCoder)
 
     GZIP_TYPES = [
       'application/javascript',

data/app/controllers/motor/icons_controller.rb

@@ -2,7 +2,7 @@
 
 module Motor
   class IconsController < ApiBaseController
-    CACHE_STORE = ActiveSupport::Cache::MemoryStore.new
+    CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(coder: ActiveSupport::Cache::NullCoder)
 
     def index
       data = CACHE_STORE.fetch('icons') do

data/app/controllers/motor/ui_controller.rb

@@ -7,14 +7,20 @@ module Motor
     helper_method :current_user
 
     def index
-      Motor.reload! if Motor.development?
-
-      Motor::Configs::SyncFromFile.call
+      render_ui
+    end
 
-      render :show
+    def new
+      render_ui
     end
 
     def show
+      render_ui
+    end
+
+    private
+
+    def render_ui
       Motor.reload! if Motor.development?
 
       Motor::Configs::SyncFromFile.call

data/app/models/motor/alert.rb

@@ -11,6 +11,7 @@ module Motor
     has_many :taggable_tags, as: :taggable, dependent: :destroy
     has_many :tags, through: :taggable_tags, class_name: 'Motor::Tag'
 
+    attribute :preferences, default: -> { HashWithIndifferentAccess.new }
     serialize :preferences, HashSerializer
 
     scope :active, -> { where(deleted_at: nil) }

data/app/models/motor/dashboard.rb

@@ -9,6 +9,7 @@ module Motor
     has_many :taggable_tags, as: :taggable, dependent: :destroy
     has_many :tags, through: :taggable_tags, class_name: 'Motor::Tag'
 
+    attribute :preferences, default: -> { HashWithIndifferentAccess.new }
     serialize :preferences, HashSerializer
 
     scope :active, -> { where(deleted_at: nil) }

data/app/models/motor/form.rb

@@ -9,6 +9,7 @@ module Motor
     has_many :taggable_tags, as: :taggable, dependent: :destroy
     has_many :tags, through: :taggable_tags, class_name: 'Motor::Tag'
 
+    attribute :preferences, default: -> { HashWithIndifferentAccess.new }
     serialize :preferences, HashSerializer
 
     scope :active, -> { where(deleted_at: nil) }

data/app/models/motor/query.rb

@@ -10,6 +10,7 @@ module Motor
     has_many :tags, through: :taggable_tags, class_name: 'Motor::Tag'
     has_many :alerts, dependent: :destroy
 
+    attribute :preferences, default: -> { HashWithIndifferentAccess.new }
     serialize :preferences, HashSerializer
 
     scope :active, -> { where(deleted_at: nil) }

data/app/models/motor/resource.rb

@@ -4,6 +4,7 @@ module Motor
   class Resource < ::Motor::ApplicationRecord
     audited
 
+    attribute :preferences, default: -> { HashWithIndifferentAccess.new }
     serialize :preferences, HashSerializer
   end
 end

data/config/routes.rb

@@ -2,7 +2,7 @@
 
 Motor::Admin.routes.draw do
   namespace :motor, path: '' do
-    scope 'api', as: 'api' do
+    scope 'api', as: :api do
       resources :run_queries, only: %i[show create]
       resources :send_alerts, only: %i[create]
       resources :queries, only: %i[index show create update destroy]
@@ -34,16 +34,15 @@ Motor::Admin.routes.draw do
 
     get '/', to: 'ui#show'
 
-    scope as: 'ui' do
+    scope as: :ui do
+      get '/data(/*path)', to: 'ui#index', as: :data
+
       with_options controller: 'ui' do
-        resources :data, only: %i[index show],
-                         param: 'path',
-                         constraints: { path: /.+/ }
         resources :reports, only: %i[index show]
-        resources :queries, only: %i[index show]
-        resources :dashboards, only: %i[index show]
-        resources :alerts, only: %i[index show]
-        resources :forms, only: %i[index show]
+        resources :queries, only: %i[index show new]
+        resources :dashboards, only: %i[index show new]
+        resources :alerts, only: %i[index show new]
+        resources :forms, only: %i[index show new]
       end
     end
   end

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

@@ -1,62 +1,65 @@
 class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
   def self.up
     create_table :motor_queries do |t|
-      t.column :name, :string, null: false
-      t.column :description, :string
-      t.column :sql_body, :string, null: false
-      t.column :preferences, :string, null: false, default: '{}'
-      t.column :author_id, :integer
-      t.column :author_type, :string
+      t.column :name, :text, null: false
+      t.column :description, :text
+      t.column :sql_body, :text, null: false
+      t.column :preferences, :text, null: false
+      t.column :author_id, :bigint
+      t.column :author_type, :text
       t.column :deleted_at, :datetime
 
       t.timestamps
 
       t.index :updated_at
-      t.index 'lower(name)',
-              name: 'motor_queries_lower_name_unique_index',
+      t.index 'name',
+              name: 'motor_queries_name_unique_index',
               unique: true,
-              where: 'deleted_at IS NULL'
+              where: 'deleted_at IS NULL',
+              length: { name: 255 }
     end
 
     create_table :motor_dashboards do |t|
-      t.column :title, :string, null: false
-      t.column :description, :string
-      t.column :preferences, :string, null: false, default: '{}'
-      t.column :author_id, :integer
-      t.column :author_type, :string
+      t.column :title, :text, null: false
+      t.column :description, :text
+      t.column :preferences, :text, null: false
+      t.column :author_id, :bigint
+      t.column :author_type, :text
       t.column :deleted_at, :datetime
 
       t.timestamps
 
       t.index :updated_at
-      t.index 'lower(title)',
-              name: 'motor_dashboards_lower_title_unique_index',
+      t.index 'title',
+              name: 'motor_dashboards_title_unique_index',
               unique: true,
-              where: 'deleted_at IS NULL'
+              where: 'deleted_at IS NULL',
+              length: { title: 255 }
     end
 
     create_table :motor_forms do |t|
-      t.column :name, :string, null: false
-      t.column :description, :string
-      t.column :api_path, :string, null: false
-      t.column :http_method, :string, null: false
-      t.column :preferences, :string, null: false, default: '{}'
-      t.column :author_id, :integer
-      t.column :author_type, :string
+      t.column :name, :text, null: false
+      t.column :description, :text
+      t.column :api_path, :text, null: false
+      t.column :http_method, :text, null: false
+      t.column :preferences, :text, null: false
+      t.column :author_id, :bigint
+      t.column :author_type, :text
       t.column :deleted_at, :datetime
 
       t.timestamps
 
       t.index :updated_at
-      t.index 'lower(name)',
-              name: 'motor_forms_lower_name_unique_index',
+      t.index 'name',
+              name: 'motor_forms_name_unique_index',
               unique: true,
-              where: 'deleted_at IS NULL'
+              where: 'deleted_at IS NULL',
+              length: { name: 255 }
     end
 
     create_table :motor_resources do |t|
-      t.column :name, :string, null: false, index: { unique: true }
-      t.column :preferences, :string, null: false, default: '{}'
+      t.column :name, :text, null: false, index: { unique: true, length: 255 }
+      t.column :preferences, :text, null: false
 
       t.timestamps
 
@@ -64,8 +67,8 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
     end
 
     create_table :motor_configs do |t|
-      t.column :key, :string, null: false, index: { unique: true }
-      t.column :value, :string, null: false, default: '{}'
+      t.column :key, :text, null: false, index: { unique: true, length: 255 }
+      t.column :value, :text, null: false
 
       t.timestamps
 
@@ -74,74 +77,80 @@ class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Mi
 
     create_table :motor_alerts do |t|
       t.references :query, null: false, foreign_key: { to_table: :motor_queries }, index: true
-      t.column :name, :string, null: false
-      t.column :description, :string
-      t.column :to_emails, :string, null: false
+      t.column :name, :text, null: false
+      t.column :description, :text
+      t.column :to_emails, :text, null: false
       t.column :is_enabled, :boolean, null: false, default: true
-      t.column :preferences, :string, null: false, default: '{}'
-      t.column :author_id, :integer
-      t.column :author_type, :string
+      t.column :preferences, :text, null: false
+      t.column :author_id, :bigint
+      t.column :author_type, :text
       t.column :deleted_at, :datetime
 
       t.timestamps
 
       t.index :updated_at
-      t.index 'lower(name)',
-              name: 'motor_alerts_lower_name_unique_index',
+      t.index 'name',
+              name: 'motor_alerts_name_unique_index',
               unique: true,
-              where: 'deleted_at IS NULL'
+              where: 'deleted_at IS NULL',
+              length: { name: 255 }
     end
 
     create_table :motor_alert_locks do |t|
       t.references :alert, null: false, foreign_key: { to_table: :motor_alerts }
-      t.column :lock_timestamp, :string, null: false
+      t.column :lock_timestamp, :text, null: false
 
       t.timestamps
 
-      t.index %i[alert_id lock_timestamp], unique: true
+      t.index %i[alert_id lock_timestamp], unique: true, length: { lock_timestamp: 255 }
     end
 
     create_table :motor_tags do |t|
-      t.column :name, :string, null: false
+      t.column :name, :text, null: false
 
       t.timestamps
 
-      t.index 'lower(name)',
-              name: 'motor_tags_lower_name_unique_index',
-              unique: true
+      t.index 'name',
+              name: 'motor_tags_name_unique_index',
+              unique: true,
+              length: { name: 255 }
     end
 
     create_table :motor_taggable_tags do |t|
       t.references :tag, null: false, foreign_key: { to_table: :motor_tags }, index: true
-      t.column :taggable_id, :integer, null: false
-      t.column :taggable_type, :string, null: false
+      t.column :taggable_id, :bigint, null: false
+      t.column :taggable_type, :text, null: false
 
       t.index %i[taggable_id taggable_type tag_id],
               name: 'motor_polymorphic_association_tag_index',
-              unique: true
+              unique: true,
+              length: { taggable_type: 255 }
     end
 
     create_table :motor_audits 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 :auditable_id, :bigint
+      t.column :auditable_type, :text
+      t.column :associated_id, :bigint
+      t.column :associated_type, :text
+      t.column :user_id, :bigint
+      t.column :user_type, :text
+      t.column :username, :text
+      t.column :action, :text
       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 :version, :bigint, default: 0
+      t.column :comment, :text
+      t.column :remote_address, :text
+      t.column :request_uuid, :text
       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, %i[auditable_type auditable_id version], name: 'motor_auditable_index',
+                                                                      length: { auditable_type: 255 }
+    add_index :motor_audits, %i[associated_type associated_id], name: 'motor_auditable_associated_index',
+                                                                length: { associated_type: 255 }
+    add_index :motor_audits, %i[user_id user_type], name: 'motor_auditable_user_index',
+                                                    length: { user_type: 255 }
+    add_index :motor_audits, :request_uuid, length: { request_uuid: 255 }
     add_index :motor_audits, :created_at
   end
 

data/lib/motor.rb

@@ -40,7 +40,6 @@ module Motor
       (defined?(::Puma) && File.basename($PROGRAM_NAME) == 'puma') ||
       defined?(::Unicorn::HttpServer) ||
       defined?(::Mongrel::HttpServer) ||
-      defined?(::WEBrick::VERSION) ||
       defined?(JRuby::Rack::VERSION) ||
       defined?(::Trinidad::Server)
   end

data/lib/motor/active_record_utils.rb

@@ -1,6 +1,18 @@
 # frozen_string_literal: true
 
-module ActiveRecordUtils
+module Motor
+  module ActiveRecordUtils
+    module_function
+
+    def reset_id_sequence!(model)
+      case ActiveRecord::Base.connection.class.name
+      when 'ActiveRecord::ConnectionAdapters::PostgreSQLAdapter'
+        ActiveRecord::Base.connection.reset_pk_sequence!(model.table_name)
+      else
+        ActiveRecord::Base.connection.reset_sequence!(model.table_name, 'id')
+      end
+    end
+  end
 end
 
 require_relative './active_record_utils/types'

data/lib/motor/alerts/persistance.rb

@@ -28,7 +28,7 @@ module Motor
       end
 
       def create_from_params!(params, current_user = nil)
-        raise NameAlreadyExists if Alert.exists?(['lower(name) = ?', params[:name].to_s.downcase])
+        raise NameAlreadyExists if Alert.exists?(name: params[:name])
 
         alert = build_from_params(params, current_user)
 
@@ -77,7 +77,7 @@ module Motor
       end
 
       def archive_with_existing_name(alert)
-        Motor::Alert.where(['lower(name) = ? AND id != ?', alert.name.to_s.downcase, alert.id])
+        Motor::Alert.where(['name = ? AND id != ?', alert.name, alert.id])
                     .update_all(deleted_at: Time.current)
       end
 
@@ -87,9 +87,9 @@ module Motor
 
       def name_already_exists?(alert)
         if alert.new_record?
-          Alert.exists?(['lower(name) = ?', alert.name.to_s.downcase])
+          Alert.exists?(name: alert.name)
         else
-          Alert.exists?(['lower(name) = ? AND id != ?', alert.name.to_s.downcase, alert.id])
+          Alert.exists?(['name = ? AND id != ?', alert.name, alert.id])
         end
       end
     end

data/lib/motor/alerts/scheduled_alerts_cache.rb

@@ -3,7 +3,8 @@
 module Motor
   module Alerts
     module ScheduledAlertsCache
-      CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(size: 5.megabytes)
+      CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(size: 5.megabytes,
+                                                          coder: ActiveSupport::Cache::NullCoder)
 
       module_function
 

data/lib/motor/alerts/scheduler.rb

@@ -3,7 +3,7 @@
 module Motor
   module Alerts
     module Scheduler
-      SCHEDULER_INTERVAL = 10.seconds
+      SCHEDULER_INTERVAL = 1.minute
       CHECK_BEHIND_DURATION = 15.minutes
 
       SCHEDULER_TASK = Concurrent::TimerTask.new(

data/lib/motor/api_query/build_json.rb

@@ -10,12 +10,12 @@ module Motor
 
         rel = rel.preload_associations_lazily if rel.is_a?(ActiveRecord::Relation)
 
-        json_params = {}
+        json_params = {}.with_indifferent_access
 
         assign_include_params(json_params, rel, params)
         assign_fields_params(json_params, rel, params)
 
-        rel.as_json(json_params.with_indifferent_access)
+        rel.as_json(json_params)
       end
 
       def assign_include_params(json_params, _rel, api_params)
@@ -51,11 +51,11 @@ module Motor
         params[:fields].each do |key, fields|
           fields = fields.split(',') if fields.is_a?(String)
 
-          merge_fields_params!(key, fields, json_params, model)
+          merge_fields_params!(json_params, key, fields, model)
         end
       end
 
-      def merge_fields_params!(key, fields, json_params, model)
+      def merge_fields_params!(json_params, key, fields, model)
         model_name = model.name.underscore
 
         if key == model_name || model_name.split('/').last == key

data/lib/motor/api_query/paginate.rb

@@ -11,6 +11,7 @@ module Motor
         params ||= {}
 
         rel = rel.limit([MAX_PER_PAGE, (params[:limit] || MAX_PER_PAGE).to_i].min)
+
         rel.offset(params[:offset].to_i)
       end
     end

data/lib/motor/build_schema/load_from_rails.rb

@@ -12,6 +12,8 @@ module Motor
         models.map do |model|
           Object.const_get(model.name)
 
+          next unless ActiveRecord::Base.connection.table_exists?(model.table_name)
+
           schema = build_model_schema(model)
 
           if model.respond_to?(:devise_modules)
@@ -20,7 +22,7 @@ module Motor
 
           schema
         rescue StandardError, NotImplementedError => e
-          Rails.logger.error(e) if model.name != 'Audited::Audit'
+          Rails.logger.error(e)
 
           next
         end.compact
@@ -34,6 +36,7 @@ module Motor
         models -= Motor::ApplicationRecord.descendants
         models -= [Motor::Audit]
         models -= [ActiveRecord::SchemaMigration] if defined?(ActiveRecord::SchemaMigration)
+        models -= [ActiveRecord::InternalMetadata] if defined?(ActiveRecord::InternalMetadata)
         models -= [ActiveStorage::Blob] if defined?(ActiveStorage::Blob)
         models -= [ActiveStorage::VariantRecord] if defined?(ActiveStorage::VariantRecord)
 
@@ -101,7 +104,7 @@ module Motor
           name: column.name,
           display_name: Utils.humanize_column_name(column.name),
           column_type: is_enum ? 'string' : UNIFIED_TYPES[column.type.to_s] || column.type.to_s,
-          is_array: column.array?,
+          is_array: column.respond_to?(:array?) && column.array?,
           access_type: COLUMN_NAME_ACCESS_TYPES.fetch(column.name, ColumnAccessTypes::READ_WRITE),
           default_value: default_attrs[column.name],
           validators: fetch_validators(model, column.name),

data/lib/motor/configs/build_ui_app_tag.rb

@@ -7,7 +7,8 @@ module Motor
         if Motor.development?
           ActiveSupport::Cache::NullStore.new
         else
-          ActiveSupport::Cache::MemoryStore.new(size: 5.megabytes)
+          ActiveSupport::Cache::MemoryStore.new(size: 5.megabytes,
+                                                coder: ActiveSupport::Cache::NullCoder)
         end
 
       module_function

data/lib/motor/configs/load_from_cache.rb

@@ -3,7 +3,8 @@
 module Motor
   module Configs
     module LoadFromCache
-      CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(size: 10.megabytes)
+      CACHE_STORE = ActiveSupport::Cache::MemoryStore.new(size: 10.megabytes,
+                                                          coder: ActiveSupport::Cache::NullCoder)
 
       module_function
 
@@ -63,18 +64,22 @@ module Motor
       end
 
       def load_cache_keys
-        ActiveRecord::Base.connection.execute(
-          "(#{
-            [
-              Motor::Config.select("'configs', MAX(updated_at)").to_sql,
-              Motor::Resource.select("'resources', MAX(updated_at)").to_sql,
-              Motor::Dashboard.select("'dashboards', MAX(updated_at)").to_sql,
-              Motor::Alert.select("'alerts', MAX(updated_at)").to_sql,
-              Motor::Query.select("'queries', MAX(updated_at)").to_sql,
-              Motor::Form.select("'forms', MAX(updated_at)").to_sql
-            ].join(') UNION (')
-          })"
-        ).to_a.map(&:values).to_h.with_indifferent_access
+        result = ActiveRecord::Base.connection.execute(cache_keys_sql).to_a
+
+        result = result.map(&:values) if result.first.is_a?(Hash)
+
+        result.to_h.with_indifferent_access
+      end
+
+      def cache_keys_sql
+        [
+          Motor::Config.select("'configs', MAX(updated_at)").to_sql,
+          Motor::Resource.select("'resources', MAX(updated_at)").to_sql,
+          Motor::Dashboard.select("'dashboards', MAX(updated_at)").to_sql,
+          Motor::Alert.select("'alerts', MAX(updated_at)").to_sql,
+          Motor::Query.select("'queries', MAX(updated_at)").to_sql,
+          Motor::Form.select("'forms', MAX(updated_at)").to_sql
+        ].join(' UNION ')
       end
     end
   end

data/lib/motor/configs/sync_from_hash.rb

@@ -87,7 +87,7 @@ module Motor
 
         archive_taggable_items(records - processed_records, configs_timestamp)
 
-        ActiveRecord::Base.connection.reset_pk_sequence!(records.klass.table_name)
+        ActiveRecordUtils.reset_id_sequence!(records.klass)
       end
 
       def update_taggable_items(records, config_items, update_proc)

data/lib/motor/configs/write_to_file.rb

@@ -9,6 +9,7 @@ module Motor
       module_function
 
       def call
+        return if Rails.env.test?
         return if THREAD_POOL.queue_length.positive?
 
         THREAD_POOL.post do

data/lib/motor/dashboards/persistance.rb

@@ -16,7 +16,7 @@ module Motor
       end
 
       def create_from_params!(params, current_user = nil)
-        raise TitleAlreadyExists if Dashboard.exists?(['lower(title) = ?', params[:title].to_s.downcase])
+        raise TitleAlreadyExists if Dashboard.exists?(title: params[:title])
 
         dashboard = build_from_params(params, current_user)
 
@@ -57,15 +57,15 @@ module Motor
       end
 
       def archive_with_existing_name(dashboard)
-        Motor::Dashboard.where(['lower(title) = ? AND id != ?', dashboard.title.to_s.downcase, dashboard.id])
+        Motor::Dashboard.where(['title = ? AND id != ?', dashboard.title, dashboard.id])
                         .update_all(deleted_at: Time.current)
       end
 
       def title_already_exists?(dashboard)
         if dashboard.new_record?
-          Motor::Dashboard.exists?(['lower(title) = ?', dashboard.title.to_s.downcase])
+          Motor::Dashboard.exists?(title: dashboard.title)
         else
-          Motor::Dashboard.exists?(['lower(title) = ? AND id != ?', dashboard.title.to_s.downcase, dashboard.id])
+          Motor::Dashboard.exists?(['title = ? AND id != ?', dashboard.title, dashboard.id])
         end
       end
     end

data/lib/motor/forms/persistance.rb

@@ -16,7 +16,7 @@ module Motor
       end
 
       def create_from_params!(params, current_user = nil)
-        raise NameAlreadyExists if Form.exists?(['lower(name) = ?', params[:name].to_s.downcase])
+        raise NameAlreadyExists if Form.exists?(name: params[:name])
 
         form = build_from_params(params, current_user)
 
@@ -57,15 +57,15 @@ module Motor
       end
 
       def archive_with_existing_name(form)
-        Motor::Form.where(['lower(name) = ? AND id != ?', form.name.to_s.downcase, form.id])
+        Motor::Form.where(['name = ? AND id != ?', form.name, form.id])
                    .update_all(deleted_at: Time.current)
       end
 
       def name_already_exists?(form)
         if form.new_record?
-          Motor::Form.exists?(['lower(name) = ?', form.name.to_s.downcase])
+          Motor::Form.exists?(['name = ?', form.name])
         else
-          Motor::Form.exists?(['lower(name) = ? AND id != ?', form.name.to_s.downcase, form.id])
+          Motor::Form.exists?(['name = ? AND id != ?', form.name, form.id])
         end
       end
     end

data/lib/motor/queries/persistance.rb

@@ -16,7 +16,7 @@ module Motor
       end
 
       def create_from_params!(params, current_user = nil)
-        raise NameAlreadyExists if Query.exists?(['lower(name) = ?', params[:name].to_s.downcase])
+        raise NameAlreadyExists if Query.exists?(name: params[:name])
 
         query = build_from_params(params, current_user)
 
@@ -57,15 +57,15 @@ module Motor
       end
 
       def archive_with_existing_name(query)
-        Motor::Query.where(['lower(name) = ? AND id != ?', query.name.to_s.downcase, query.id])
+        Motor::Query.where(['name = ? AND id != ?', query.name, query.id])
                     .update_all(deleted_at: Time.current)
       end
 
       def name_already_exists?(query)
         if query.new_record?
-          Query.exists?(['lower(name) = ?', query.name.to_s.downcase])
+          Query.exists?(name: query.name)
         else
-          Query.exists?(['lower(name) = ? AND id != ?', query.name.to_s.downcase, query.id])
+          Query.exists?(['name = ? AND id != ?', query.name, query.id])
         end
       end
     end

data/lib/motor/queries/run_query.rb

@@ -35,8 +35,8 @@ module Motor
 
         ActiveRecord::Base.transaction do
           result =
-            case ActiveRecord::Base.connection
-            when ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
+            case ActiveRecord::Base.connection.class.name
+            when 'ActiveRecord::ConnectionAdapters::PostgreSQLAdapter'
               PostgresqlExecQuery.call(ActiveRecord::Base.connection, statement)
             else
               ActiveRecord::Base.connection.exec_query(*statement)

data/lib/motor/tags.rb

@@ -10,7 +10,7 @@ module Motor
       tags.each do |tag_name|
         next if taggable.taggable_tags.find { |tt| tt.tag.name.casecmp(tag_name).zero? }
 
-        tag = Tag.where('lower(name) = ?', tag_name.downcase).take || Tag.new(name: tag_name)
+        tag = Tag.find_or_initialize_by(name: tag_name)
 
         taggable.taggable_tags.new(tag: tag)
       end

data/lib/motor/version.rb

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

data/ui/dist/manifest.json

@@ -2068,11 +2068,11 @@
   "mail-opened.svg": "icons/mail-opened.svg",
   "mail.svg": "icons/mail.svg",
   "mailbox.svg": "icons/mailbox.svg",
-  "main-918b506ecd64910a5845.css.gz": "main-918b506ecd64910a5845.css.gz",
-  "main-918b506ecd64910a5845.js.LICENSE.txt": "main-918b506ecd64910a5845.js.LICENSE.txt",
-  "main-918b506ecd64910a5845.js.gz": "main-918b506ecd64910a5845.js.gz",
-  "main.css": "main-918b506ecd64910a5845.css",
-  "main.js": "main-918b506ecd64910a5845.js",
+  "main-55914a490baf8f7eba59.css.gz": "main-55914a490baf8f7eba59.css.gz",
+  "main-55914a490baf8f7eba59.js.LICENSE.txt": "main-55914a490baf8f7eba59.js.LICENSE.txt",
+  "main-55914a490baf8f7eba59.js.gz": "main-55914a490baf8f7eba59.js.gz",
+  "main.css": "main-55914a490baf8f7eba59.css",
+  "main.js": "main-55914a490baf8f7eba59.js",
   "man.svg": "icons/man.svg",
   "manual-gearbox.svg": "icons/manual-gearbox.svg",
   "map-2.svg": "icons/map-2.svg",

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: motor-admin
 version: !ruby/object:Gem::Version
-  version: 0.1.49
+  version: 0.1.50
 platform: ruby
 authors:
 - Pete Matsyburka
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-25 00:00:00.000000000 Z
+date: 2021-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord-filter
@@ -1481,8 +1481,8 @@ files:
 - ui/dist/icons/zoom-money.svg.gz
 - ui/dist/icons/zoom-out.svg.gz
 - ui/dist/icons/zoom-question.svg.gz
-- ui/dist/main-918b506ecd64910a5845.css.gz
-- ui/dist/main-918b506ecd64910a5845.js.gz
+- ui/dist/main-55914a490baf8f7eba59.css.gz
+- ui/dist/main-55914a490baf8f7eba59.js.gz
 - ui/dist/manifest.json
 homepage:
 licenses: