wallaby-active_record 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77775377884b090f9d0c0679a0eda24caccaece57e0429f8307457d91ebce2a5
-  data.tar.gz: 23bdf49c685c9b938cfea121ab3bf44fad2f2bffeba964a4bfcfacd3595368f8
+  metadata.gz: 3326a29a32d97399d5d5e5381ca283b5fefc1601bbe212250c2753deb26d9509
+  data.tar.gz: e61f519c75994ffce103baee81a573941869487dec966e6060299ec24fe73e12
 SHA512:
-  metadata.gz: 35d9dcf0387cae97199ad0aa47628bbc4719c1f3b48cccd0e04450b87c5b54c5358e8fe5ab3a9fc960e5fa410990438a4b5d00bb7431ab0d8ade6ea7f2aa26af
-  data.tar.gz: 07c885247c19f20b71db0f4f85b07e7cb955e68d4cbdec727057dc55938ecac103e4e42f813584869df932638dd7559f2544372aead0903beb86089f75d18c04
+  metadata.gz: 05003c11197eeab16a351e202b3f7ee97a593abb0b7bf99505d516acb924950522805e6a0198fa274a635d737f1a4a9b69a8591f961304a0e4085706de86800a
+  data.tar.gz: 8f34233e937848899b6d16a7143a990cd584ccaf23b9c29700ac1cc8837214f1184b34e0fd0fd1f93eaeef15fcaab0898276d63e5529aac4da2a9aac064a378d

data/README.md

@@ -7,7 +7,8 @@
 [![Test Coverage](https://api.codeclimate.com/v1/badges/9ba0a610043a2e1a9e74/test_coverage)](https://codeclimate.com/github/wallaby-rails/wallaby-active_record/test_coverage)
 [![Inch CI](https://inch-ci.org/github/wallaby-rails/wallaby-active_record.svg?branch=master)](https://inch-ci.org/github/wallaby-rails/wallaby-active_record)
 
-Wallaby::ActiveRecord is the ActiveRecord adapter for [Wallaby::Core](https://github.com/wallaby-rails/wallaby-core).
+Wallaby::ActiveRecord is the ActiveRecord adapter that implements the [Wallaby::Core](https://github.com/wallaby-rails/wallaby-core)
+interfaces to handle ActiveRecord model/instance(s) in all the CRUD/authorization/pagination operations.
 
 ## Install
 

data/lib/adapters/wallaby/active_record.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
 
 module Wallaby
-  # ActiveRecord mode
+  # The Wallaby::ActiveRecord mode that implements the {Wallaby::Core} interfaces
+  # to handle ActiveRecord model/instance in all CRUD operations
   class ActiveRecord < Mode
   end
 end

data/lib/adapters/wallaby/active_record/cancancan_provider.rb

@@ -2,7 +2,7 @@
 
 module Wallaby
   class ActiveRecord
-    # Cancancan provider for ActiveRecord
+    # Cancancan provider for {Wallaby::ActiveRecord}
     class CancancanProvider < CancancanAuthorizationProvider
     end
   end

data/lib/adapters/wallaby/active_record/default_provider.rb

@@ -2,7 +2,7 @@
 
 module Wallaby
   class ActiveRecord
-    # Default provider for ActiveRecord
+    # Default provider for {Wallaby::ActiveRecord}
     class DefaultProvider < DefaultAuthorizationProvider
     end
   end

data/lib/adapters/wallaby/active_record/model_decorator.rb

@@ -2,24 +2,25 @@
 
 module Wallaby
   class ActiveRecord
-    # Modal decorator for ActiveRecord
+    # Modal decorator for {Wallaby::ActiveRecord}
     class ModelDecorator < ::Wallaby::ModelDecorator
-      # Data types to exclude for index page
-      INDEX_EXCLUSIVE_DATA_TYPES =
-        (['', 'medium', 'long'] * 2)
-        .zip(%w(blob text) * 3).map(&:join)
-        .concat(%w(binary citext hstore json jsonb tsvector xml)).freeze
+      # Data types to exclude for {#index_field_names}
+      INDEX_EXCLUSIVE_DATA_TYPES = %w(
+        binary citext hstore json jsonb tsvector xml
+        blob mediumblob longblob text mediumtext longtext
+      ).freeze
 
-      # Class to exclude for show page
+      # Classes to exclude for {#show_field_names}
       SHOW_EXCLUSIVE_CLASS_NAMES = %w(ActiveStorage::Attachment ActiveStorage::Blob).freeze
 
-      # Data types to exclude for form page
+      # Fields to exclude for {#form_field_names}
       FORM_EXCLUSIVE_DATA_TYPES = %w(created_at updated_at).freeze
 
-      # Origin metadata directly coming from ActiveRecord.
+      # Original metadata information of the primative and association fields
+      # pulling out from the ActiveRecord model.
       #
-      # It needs to be frozen so that we can keep the metadata integrity
-      # @example sample fields:
+      # It needs to be frozen so that we can keep the metadata intact.
+      # @example sample fields metadata:
       #   model_decorator.fields
       #   # =>
       #   {
@@ -27,28 +28,29 @@ module Wallaby
       #     id: { name: 'id', type: 'integer', label: 'Id' },
       #     # association field
       #     category: {
-      #       'name' => 'category',
-      #       'type' => 'belongs_to',
-      #       'label' => 'Category',
-      #       'is_association' => true,
-      #       'is_through' => false,
-      #       'has_scope' => false,
-      #       'foreign_key' => 'category_id',
-      #       'class' => Category
+      #       name: 'category',
+      #       type: 'belongs_to',
+      #       label: 'Category',
+      #       is_association: true,
+      #       is_through: false,
+      #       has_scope: false,
+      #       foreign_key: 'category_id',
+      #       class: Category
       #     }
       #   }
       # @return [ActiveSupport::HashWithIndifferentAccess] metadata
       def fields
+        # NOTE: Need to check the database and table's existence before building up the metadata
+        # so that the database creation and migration related task can be executed.
         @fields ||= ::ActiveSupport::HashWithIndifferentAccess.new.tap do |hash|
-          # NOTE: Need to check database and table's existence
-          # before pulling out the metadata from model.
-          # So that the database and migration related task can be executed.
-          next unless ::ActiveRecord::Base.connected? && @model_class.table_exists?
+          next unless @model_class.table_exists?
 
           hash.merge! general_fields
           hash.merge! association_fields
           hash.except!(*foreign_keys_from_associations)
         end.freeze
+      rescue ::ActiveRecord::NoDatabaseError
+        Hash.new({})
       end
 
       # A copy of {#fields} for index page
@@ -86,7 +88,8 @@ module Wallaby
           end.keys
       end
 
-      # @return [Array<String>] a list of field names for form (new/edit) page (note: complex fields are excluded).
+      # @return [Array<String>] a list of field names for form (new/edit) page
+      #   (note: timestamps fields (e.g. created_at/updated_at) and complex relation fields are excluded).
       def form_field_names
         @form_field_names ||=
           form_fields.reject do |field_name, metadata|
@@ -109,9 +112,8 @@ module Wallaby
       # To guess the title for resource.
       #
       # It will go through the fields and try to find out the one that looks
-      # like a name or text to represent this resource. Otherwise, it will fall
+      # like a name or text representing this resource. Otherwise, it will fall
       # back to primary key.
-      #
       # @param resource [Object]
       # @return [String] the title of given resource
       def guess_title(resource)

data/lib/adapters/wallaby/active_record/model_decorator/fields_builder.rb

@@ -3,7 +3,7 @@
 module Wallaby
   class ActiveRecord
     class ModelDecorator
-      # To search and build the metadata for fields
+      # To build the metadata for fields
       class FieldsBuilder
         # @param model_class [Class]
         def initialize(model_class)
@@ -22,7 +22,8 @@ module Wallaby
           end
         end
 
-        # @return [Hash<String, Hash>] a hash for general fields
+        # @return [Hash<String, Hash>] a hash for association fields
+        #   (e.g. belongs_to / has_one / has_many / has_and_belongs_to_many)
         def association_fields
           @model_class.reflections.each_with_object({}) do |(name, ref), fields|
             metadata = {

data/lib/adapters/wallaby/active_record/model_decorator/fields_builder/association_builder.rb

@@ -4,7 +4,7 @@ module Wallaby
   class ActiveRecord
     class ModelDecorator
       class FieldsBuilder
-        # To build the metadata for associations
+        # To build the metadata for associations fields
         class AssociationBuilder
           # Update the metadata
           # @param metadata [Hash]
@@ -18,7 +18,7 @@ module Wallaby
             metadata[:foreign_key] = foreign_key_for(reflection, type)
           end
 
-          private
+          protected
 
           # @param reflection [ActiveRecord::Reflection]
           # @param type [Symbol]
@@ -35,13 +35,15 @@ module Wallaby
           end
 
           # @param reflection [ActiveRecord::Reflection]
-          # @return [Boolean] whether it's a through relation
+          # @return [true] if it's a through relation
+          # @return [false] otherwise
           def through?(reflection)
             reflection.is_a? ::ActiveRecord::Reflection::ThroughReflection
           end
 
           # @param reflection [ActiveRecord::Reflection]
-          # @return [Boolean] whether it has scope
+          # @return [true] if it has scope
+          # @return [false] otherwise
           def scope?(reflection)
             reflection.scope.present?
           end

data/lib/adapters/wallaby/active_record/model_decorator/fields_builder/polymorphic_builder.rb

@@ -19,7 +19,7 @@ module Wallaby
             end
           end
 
-          private
+          protected
 
           # @param reflection [ActiveRecord::Reflection]
           # @return [Array<Class>] a list of classes for this polymorphism
@@ -38,7 +38,8 @@ module Wallaby
 
           # @param model_class [Class]
           # @param polymorphic_name [String] polymorphic name
-          # @return [Boolean] if polymorphism defined?
+          # @return [true] if polymorphism defined?
+          # @return [false] otherwise
           def polymorphic_defined?(model_class, polymorphic_name)
             polymorphic_name_sym = polymorphic_name.try(:to_sym)
             model_class.reflections.any? do |_field_name, reflection|

data/lib/adapters/wallaby/active_record/model_decorator/fields_builder/sti_builder.rb

@@ -4,44 +4,48 @@ module Wallaby
   class ActiveRecord
     class ModelDecorator
       class FieldsBuilder
-        # To build the metadata for sti column
+        # This class updates the field metadata's value of **type** and **sti_class_list**
+        # for STI (Single Table Inheritance) model
         class StiBuilder
           # @param model_class [Class]
           def initialize(model_class)
             @model_class = model_class
           end
 
-          # update the metadata
+          # Update the field metadata's value for **type** and **sti_class_list**
           # @param metadata [Hash]
           # @param column [ActiveRecord::ConnectionAdapters::Column]
           def update(metadata, column)
             return unless @model_class.inheritance_column == column.name
 
             metadata[:type] = 'sti'
-            metadata[:sti_class_list] = sti_list(find_parent_of(@model_class))
+            metadata[:sti_class_list] = sti_list(find_sti_parent_of(@model_class))
           end
 
-          private
+          protected
 
+          # Return the alphabet-order STI list
+          # by traversing the inheritance tree for given model.
           # @param klass [Class]
-          # @return [Array<Class>] a list of STI classes for this model
+          # @return [Array<Class>]
           def sti_list(klass)
-            list = klass.descendants << klass
-            list.sort_by(&:name)
+            (klass.descendants << klass).sort_by(&:name)
           end
 
+          # Find out which parent is the one that can give us the STI list.
           # @param klass [Class]
-          # @return [Class] the top parent class in the STI hierarchy
-          def find_parent_of(klass)
+          # @return [Class]
+          def find_sti_parent_of(klass)
             parent = klass
-            parent = parent.superclass until top_parent?(parent.superclass)
+            parent = parent.superclass until not_sti_parent?(parent.superclass)
             parent
           end
 
           # @param klass [Class]
-          # @return [Boolean] whether the class is ActiveRecord base class
-          def top_parent?(klass)
-            klass == ModelFinder.base || klass.try(:abstract_class?)
+          # @return [true] if klass is ActiveRecord::Base or abstract
+          # @return [false] otherwise
+          def not_sti_parent?(klass)
+            klass == ::ActiveRecord::Base || klass.try(:abstract_class?)
           end
         end
       end

data/lib/adapters/wallaby/active_record/model_decorator/title_field_finder.rb

@@ -16,15 +16,12 @@ module Wallaby
 
         # @return [String] field name that can be used as title
         def find
-          possible_title_fields = @fields.select do |_field_name, metadata|
-            TITLE_FIELD_TYPES.include? metadata[:type]
-          end
-          target_field = possible_title_fields.keys.find do |field_name|
-            TITLE_NAMES.any? { |v| field_name.to_s.index v }
-          end
-          target_field \
-            || possible_title_fields.keys.first \
-            || @model_class.primary_key
+          FieldUtils.first_field_by(
+            {
+              name: /title|name|label|string/,
+              type: 'string'
+            }, @fields
+          ) || @model_class.primary_key
         end
       end
     end

data/lib/adapters/wallaby/active_record/model_finder.rb

@@ -2,43 +2,30 @@
 
 module Wallaby
   class ActiveRecord
-    # Model finder
+    # Finder to return all the appropriate ActiveRecord models.
     class ModelFinder < ::Wallaby::ModelFinder
-      # @return [Array<Class>] a list of ActiveRecord subclasses
+      # Return a list of ActiveRecord::Base subclasses that aren't one of the following types:
+      #
+      # 1. abstract class
+      # 2. anonymous class
+      # 3. the HABTM relation class
+      # @return [Array<Class>]
       def all
-        self.class.base.descendants.reject do |model_class|
-          abstract?(model_class) || anonymous?(model_class) || habtm?(model_class)
+        ::ActiveRecord::Base.descendants.reject do |model_class|
+          defined?(::ApplicationRecord) && model_class == ::ApplicationRecord ||
+            model_class.abstract_class? ||
+            anonymous?(model_class) ||
+            model_class.name.index('HABTM')
         end.sort_by(&:to_s)
       end
 
-      # This is only for ActiveRecord
-      # @return [ApplicationRecord, ActiveRecord::Base] base ActiveRecord class
-      def self.base
-        return ::ApplicationRecord if defined? ::ApplicationRecord
-
-        ::ActiveRecord::Base
-      end
-
-      private
-
-      # Is model class abstract?
-      # @param model_class [Class]
-      # @return [Boolean]
-      def abstract?(model_class)
-        model_class.abstract_class?
-      end
+      protected
 
       # @see Wallaby::ModuleUtils.anonymous_class?
       def anonymous?(model_class)
-        ModuleUtils.anonymous_class? model_class
-      end
-
-      # Check and see if given model class is intermediate class that generated
-      # for has and belongs to many assocation
-      # @param model_class [Class]
-      # @return [Boolean]
-      def habtm?(model_class)
-        model_class.name.index('HABTM')
+        ModuleUtils.anonymous_class?(model_class).tap do |result|
+          Logger.warn "Anonymous class is detected for table #{model_class.try :table_name}" if result
+        end
       end
     end
   end

data/lib/adapters/wallaby/active_record/model_pagination_provider.rb

@@ -8,25 +8,17 @@ module Wallaby
       # @return [true] if paginatable
       # @return [false] if not paginatable
       def paginatable?
-        paginatable =
-          # kaminari
-          @collection.respond_to?(:total_count) || \
-          @collection.respond_to?(:total_entries) # will_paginate
-        Logger.warn "#{@collection.inspect} is not paginatable." unless paginatable
-
-        paginatable
+        (@collection.respond_to?(:unscope) && @collection.respond_to?(:count)).tap do |paginatable|
+          Logger.warn "#{@collection} is not paginatable." unless paginatable
+        end
       end
 
       # @return [Integer] total count for the collection
       def total
-        # kaminari
-        @collection.try(:total_count) || \
-          @collection.try(:total_entries) # will_paginate
+        @collection.unscope(:offset, :limit).count
       end
 
-      # @return [Integer] page size from parameters or
-      #   {https://rubydoc.info/gems/wallaby-core/Wallaby/Configuration/Pagination#page_size-instance_method page_size}
-      #   Wallaby configuration
+      # @return [Integer] page size from parameters or Wallaby configuration
       def page_size
         (@params[:per] || Wallaby.configuration.pagination.page_size).to_i
       end

data/lib/adapters/wallaby/active_record/model_service_provider.rb

@@ -8,7 +8,6 @@ module Wallaby
       # @param action [String, Symbol]
       # @param authorizer
       # @return [ActionController::Parameters] whitelisted parameters
-      # @see Wallaby::ModelServiceProvider#permit
       def permit(params, action, authorizer)
         authorized_fields = authorizer.permit_params action, @model_class
         params.require(param_key).permit(authorized_fields || permitted_fields)
@@ -18,7 +17,6 @@ module Wallaby
       # @param params [ActionController::Parameters]
       # @param authorizer [Ability] for now
       # @return [ActiveRecord::Relation] relation
-      # @see Wallaby::ModelServiceProvider#collection
       def collection(params, authorizer)
         query = querier.search params
         query = query.order params[:sort] if params[:sort].present?
@@ -28,19 +26,15 @@ module Wallaby
       # @param query [ActiveRecord::Relation]
       # @param params [ActionController::Parameters]
       # @return [ActiveRecord::Relation] paginated query
-      # @see Wallaby::ModelServiceProvider#paginate
       def paginate(query, params)
-        # NOTE: do not take out the `.to_i` as will_paginate requires an integer `per_page`
         per = (params[:per] || Wallaby.configuration.pagination.page_size).to_i
-        query = query.page params[:page] if query.respond_to? :page
-        query = query.per per if query.respond_to? :per # kaminari
-        query = query.per_page per if query.respond_to? :per_page # will_paginate
+        page = [params[:page].to_i, 1].max # starting from page 1
+        query = query.offset((page - 1) * per).limit(per)
         query
       end
 
       # @note No mass assignment happens here!
       # @return [Object] new resource object
-      # @see Wallaby::ModelServiceProvider#new
       def new(_params, _authorizer)
         @model_class.new
       end
@@ -50,7 +44,6 @@ module Wallaby
       # @param id [Integer, String]
       # @return [Object] persisted resource object
       # @raise [Wallaby::ResourceNotFound] when record is not found
-      # @see Wallaby::ModelServiceProvider#find
       def find(id, _params, _authorizer)
         @model_class.find id
       rescue ::ActiveRecord::RecordNotFound
@@ -61,7 +54,6 @@ module Wallaby
       # @param resource [Object]
       # @param params [ActionController::Parameters]
       # @param authorizer [Wallaby::ModelAuthorizer]
-      # @see Wallaby::ModelServiceProvider#create
       def create(resource, params, authorizer)
         save __callee__, resource, params, authorizer
       end
@@ -70,14 +62,12 @@ module Wallaby
       # @param resource [Object]
       # @param params [ActionController::Parameters]
       # @param authorizer [Wallaby::ModelAuthorizer]
-      # @see Wallaby::ModelServiceProvider#update
       def update(resource, params, authorizer)
         save __callee__, resource, params, authorizer
       end
 
       # Remove a record from database
       # @param resource [Object]
-      # @see Wallaby::ModelServiceProvider#destroy
       def destroy(resource, _params, _authorizer)
         resource.destroy
       end
@@ -109,7 +99,8 @@ module Wallaby
 
       # See if a resource is valid
       # @param resource [Object]
-      # @return [Boolean]
+      # @return [true] if valid
+      # @return [false] otherwise
       def valid?(resource)
         validator.valid? resource
       end
@@ -137,22 +128,22 @@ module Wallaby
           permitter.simple_field_names << permitter.compound_hashed_fields
       end
 
-      # @see Wallaby::ModelServiceProvider::Permitter
+      # @return [Wallaby::ActiveRecord::ModelServiceProvider::Permitter]
       def permitter
         @permitter ||= Permitter.new @model_decorator
       end
 
-      # @see Wallaby::ModelServiceProvider::Querier
+      # @return [Wallaby::ActiveRecord::ModelServiceProvider::Querier]
       def querier
         @querier ||= Querier.new @model_decorator
       end
 
-      # @see Wallaby::ModelServiceProvider::Normalizer
+      # @return [Wallaby::ActiveRecord::ModelServiceProvider::Normalizer]
       def normalizer
         @normalizer ||= Normalizer.new @model_decorator
       end
 
-      # @see Wallaby::ModelServiceProvider::Validator
+      # @return [Wallaby::ActiveRecord::ModelServiceProvider::Validator]
       def validator
         @validator ||= Validator.new @model_decorator
       end

data/lib/adapters/wallaby/active_record/model_service_provider/permitter.rb

@@ -3,7 +3,7 @@
 module Wallaby
   class ActiveRecord
     class ModelServiceProvider
-      # Filter the params
+      # Whitelist the params for mass-assignment
       class Permitter
         # @param model_decorator [Wallaby::ModelDecorator]
         def initialize(model_decorator)

data/lib/adapters/wallaby/active_record/model_service_provider/querier.rb

@@ -26,7 +26,7 @@ module Wallaby
           scope.where query
         end
 
-        private
+        protected
 
         # @return [Arel::Table] arel table
         def table

data/lib/adapters/wallaby/active_record/model_service_provider/querier/escaper.rb

@@ -11,6 +11,14 @@ module Wallaby
           PCT = '%' # :nodoc:
 
           class << self
+            # @example Return the escaped keyword if the first/last char of the keyword is `%`/`_`
+            #   Wallaby::ActiveRecord::ModelServiceProvider::Querier::Escaper.execute('%something_else%')
+            #   # => '%something\_else%'
+            # @example Return the escaped keyword wrapped with `%` if the first/last char of the keyword is NOT `%`/`_`
+            #   Wallaby::ActiveRecord::ModelServiceProvider::Querier::Escaper.execute('keyword')
+            #   # => '%keyword%'
+            # @param keyword [String]
+            # @return [String] escaped string for LIKE query
             def execute(keyword)
               first = keyword.first
               last = keyword.last
@@ -23,6 +31,12 @@ module Wallaby
               "#{starting}#{escaped}#{ending}"
             end
 
+            protected
+
+            # @param first_condition [Boolean] first condition
+            # @param first_char [Boolean] first char
+            # @param second_condition [nil, String] second condition
+            # @param default_sign [String]
             def sign(
               first_condition, first_char, second_condition, default_sign = PCT
             )

data/lib/adapters/wallaby/active_record/model_service_provider/querier/wrapper.rb

@@ -4,7 +4,9 @@ module Wallaby
   class ActiveRecord
     class ModelServiceProvider
       class Querier
-        # Wrapper for the transform result
+        # Wrapper for the {Wallaby::ActiveRecord::ModelServiceProvider::Querier::Transformer} result.
+        # It's only used by {Wallaby::ActiveRecord::ModelServiceProvider::Querier} to tell it apart
+        # with non-{Wallaby::ActiveRecord::ModelServiceProvider::Querier::Transformer} result
         class Wrapper
           attr_reader :list
           delegate :push, to: :list
@@ -12,6 +14,7 @@ module Wallaby
           delegate :last, to: :list
           delegate :[], to: :last
 
+          # @param list [Array]
           def initialize(list = [])
             @list = list
           end

data/lib/adapters/wallaby/active_record/model_service_provider/validator.rb

@@ -3,7 +3,7 @@
 module Wallaby
   class ActiveRecord
     class ModelServiceProvider
-      # Validator
+      # Validate values for record create / update
       class Validator
         # @param model_decorator [Wallaby::ModelDecorator]
         def initialize(model_decorator)
@@ -11,7 +11,8 @@ module Wallaby
         end
 
         # @param resource [Object] resource object
-        # @return [Boolean] whether the resource object is valid
+        # @return [true] if the resource object is valid
+        # @return [false] otherwise
         def valid?(resource)
           resource.attributes.each do |field_name, values|
             metadata = @model_decorator.fields[field_name]
@@ -22,10 +23,11 @@ module Wallaby
           resource.errors.blank?
         end
 
-        private
+        protected
 
         # @param values [Array]
-        # @return [Boolean] whether the values are valid range values
+        # @return [true] if the values are valid range values
+        # @return [false] otherwise
         def valid_range_type?(values, metadata)
           !metadata \
             || !%w(daterange tsrange tstzrange).include?(metadata[:type]) \

data/lib/adapters/wallaby/active_record/pundit_provider.rb

@@ -2,7 +2,7 @@
 
 module Wallaby
   class ActiveRecord
-    # Pundit provider for ActiveRecord
+    # Pundit provider for {Wallaby::ActiveRecord}
     class PunditProvider < PunditAuthorizationProvider
       # Filter a scope
       # @param _action [Symbol, String]
@@ -11,7 +11,7 @@ module Wallaby
       def accessible_for(_action, scope)
         Pundit.policy_scope! user, scope
       rescue Pundit::NotDefinedError
-        Logger.warn "Cannot find scope policy for #{scope.inspect}."
+        Logger.warn "Cannot find scope policy for `#{scope}`."
         scope
       end
     end

data/lib/wallaby/active_record.rb

@@ -6,7 +6,6 @@ require 'wallaby/active_record/version'
 
 # All files required for ActiveRecord ORM
 require 'adapters/wallaby/active_record'
-require 'adapters/wallaby/active_record/logger'
 require 'adapters/wallaby/active_record/model_finder'
 
 # ModelPaginationProvider: begin

data/lib/wallaby/active_record/version.rb

@@ -2,6 +2,6 @@
 
 module Wallaby
   module ActiveRecordGem
-    VERSION = '0.2.4' # :nodoc:
+    VERSION = '0.2.5' # :nodoc:
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: wallaby-active_record
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - Tian Chen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-21 00:00:00.000000000 Z
+date: 2020-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -28,16 +28,16 @@ dependencies:
   name: wallaby-core
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.2.2
 - !ruby/object:Gem::Dependency
   name: cancancan
   requirement: !ruby/object:Gem::Requirement
@@ -106,7 +106,6 @@ files:
 - lib/adapters/wallaby/active_record.rb
 - lib/adapters/wallaby/active_record/cancancan_provider.rb
 - lib/adapters/wallaby/active_record/default_provider.rb
-- lib/adapters/wallaby/active_record/logger.rb
 - lib/adapters/wallaby/active_record/model_decorator.rb
 - lib/adapters/wallaby/active_record/model_decorator/fields_builder.rb
 - lib/adapters/wallaby/active_record/model_decorator/fields_builder/association_builder.rb

data/lib/adapters/wallaby/active_record/logger.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-module Wallaby
-  class ActiveRecord
-    # move this to wallaby-core later
-    class Logger
-      class << self
-        %w(debug warn info).each do |method_id|
-          define_method method_id do |message, replacements = {}|
-            source = caller(replacements.delete(:sourcing).try(:to_i) || 1, 1).first
-            Rails.logger.public_send(
-              method_id,
-              "#{method_id.upcase}: #{format message, replacements}\nfrom #{source}"
-            )
-            nil
-          end
-        end
-      end
-    end
-  end
-end