baza_models 0.0.11 → 0.0.15

data/Rakefile

@@ -1,18 +1,16 @@
-# encoding: utf-8
-
 require "rubygems"
 require "bundler"
 begin
   Bundler.setup(:default, :development)
 rescue Bundler::BundlerError => e
-  $stderr.puts e.message
-  $stderr.puts "Run `bundle install` to install missing gems"
+  warn e.message
+  warn "Run `bundle install` to install missing gems"
   exit e.status_code
 end
 require "rake"
 
-require "jeweler"
-Jeweler::Tasks.new do |gem|
+require "juwelier"
+Juwelier::Tasks.new do |gem|
   # gem is a Gem::Specification... see http://guides.rubygems.org/specification-reference/ for more options
   gem.name = "baza_models"
   gem.homepage = "http://github.com/kaspernj/baza_models"
@@ -23,7 +21,7 @@ Jeweler::Tasks.new do |gem|
   gem.authors = ["kaspernj"]
   # dependencies defined in Gemfile
 end
-Jeweler::RubygemsDotOrgTasks.new
+Juwelier::RubygemsDotOrgTasks.new
 
 require "rspec/core"
 require "rspec/core/rake_task"

data/VERSION

@@ -1 +1 @@
-0.0.11
+0.0.15

data/baza_models.gemspec

@@ -1,17 +1,17 @@
-# Generated by jeweler
+# Generated by juwelier
 # DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# Instead, edit Juwelier::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: baza_models 0.0.11 ruby lib
+# stub: baza_models 0.0.15 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "baza_models".freeze
-  s.version = "0.0.11"
+  s.version = "0.0.15"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["kaspernj".freeze]
-  s.date = "2018-01-09"
+  s.date = "2021-10-04"
   s.description = "ActiveRecord like models for the Baza database framework".freeze
   s.email = "k@spernj.org".freeze
   s.extra_rdoc_files = [
@@ -20,9 +20,11 @@ Gem::Specification.new do |s|
   ]
   s.files = [
     ".document",
+    ".github/dependabot.yml",
     ".rspec",
     ".rubocop.yml",
     ".rubocop_todo.yml",
+    ".ruby-version",
     "Gemfile",
     "Gemfile.lock",
     "LICENSE.txt",
@@ -67,7 +69,7 @@ Gem::Specification.new do |s|
     "lib/baza_models/validators/length_validator.rb",
     "lib/baza_models/validators/presence_validator.rb",
     "lib/baza_models/validators/uniqueness_validator.rb",
-    "shippable.yml",
+    "peak_flow.yml",
     "spec/baza_models/autoloader_spec.rb",
     "spec/baza_models/baza_orm_adapter_spec.rb",
     "spec/baza_models/class_translation_spec.rb",
@@ -91,7 +93,6 @@ Gem::Specification.new do |s|
     "spec/baza_models/validators/format_validator_spec.rb",
     "spec/baza_models/validators/length_validator_spec.rb",
     "spec/baza_models/validators/uniqueness_validator_spec.rb",
-    "spec/baza_models_spec.rb",
     "spec/factories/organization.rb",
     "spec/factories/user.rb",
     "spec/spec_helper.rb",
@@ -104,63 +105,50 @@ Gem::Specification.new do |s|
   ]
   s.homepage = "http://github.com/kaspernj/baza_models".freeze
   s.licenses = ["MIT".freeze]
-  s.rubygems_version = "2.6.13".freeze
+  s.rubygems_version = "3.1.6".freeze
   s.summary = "ActiveRecord like models for the Baza database framework".freeze
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
+  end
 
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<array_enumerator>.freeze, [">= 0.0.10"])
-      s.add_runtime_dependency(%q<auto_autoloader>.freeze, [">= 0.0.1"])
-      s.add_runtime_dependency(%q<baza>.freeze, ["~> 0.0.21"])
-      s.add_runtime_dependency(%q<html_gen>.freeze, [">= 0.0.12"])
-      s.add_runtime_dependency(%q<simple_delegate>.freeze, [">= 0.0.2"])
-      s.add_runtime_dependency(%q<string-cases>.freeze, [">= 0.0.3"])
-      s.add_development_dependency(%q<best_practice_project>.freeze, [">= 0"])
-      s.add_development_dependency(%q<bundler>.freeze, ["~> 1.0"])
-      s.add_development_dependency(%q<factory_girl>.freeze, [">= 0"])
-      s.add_development_dependency(%q<jdbc-sqlite3>.freeze, [">= 0"])
-      s.add_development_dependency(%q<jeweler>.freeze, ["= 2.3.6"])
-      s.add_development_dependency(%q<orm_adapter>.freeze, [">= 0"])
-      s.add_development_dependency(%q<rdoc>.freeze, ["~> 3.12"])
-      s.add_development_dependency(%q<rspec>.freeze, ["= 3.6.0"])
-      s.add_development_dependency(%q<rubocop>.freeze, ["= 0.37.0"])
-      s.add_development_dependency(%q<sqlite3>.freeze, [">= 0"])
-    else
-      s.add_dependency(%q<array_enumerator>.freeze, [">= 0.0.10"])
-      s.add_dependency(%q<auto_autoloader>.freeze, [">= 0.0.1"])
-      s.add_dependency(%q<baza>.freeze, ["~> 0.0.21"])
-      s.add_dependency(%q<html_gen>.freeze, [">= 0.0.12"])
-      s.add_dependency(%q<simple_delegate>.freeze, [">= 0.0.2"])
-      s.add_dependency(%q<string-cases>.freeze, [">= 0.0.3"])
-      s.add_dependency(%q<best_practice_project>.freeze, [">= 0"])
-      s.add_dependency(%q<bundler>.freeze, ["~> 1.0"])
-      s.add_dependency(%q<factory_girl>.freeze, [">= 0"])
-      s.add_dependency(%q<jdbc-sqlite3>.freeze, [">= 0"])
-      s.add_dependency(%q<jeweler>.freeze, ["= 2.3.6"])
-      s.add_dependency(%q<orm_adapter>.freeze, [">= 0"])
-      s.add_dependency(%q<rdoc>.freeze, ["~> 3.12"])
-      s.add_dependency(%q<rspec>.freeze, ["= 3.6.0"])
-      s.add_dependency(%q<rubocop>.freeze, ["= 0.37.0"])
-      s.add_dependency(%q<sqlite3>.freeze, [">= 0"])
-    end
+  if s.respond_to? :add_runtime_dependency then
+    s.add_runtime_dependency(%q<array_enumerator>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<auto_autoloader>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<baza>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<html_gen>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<simple_delegate>.freeze, [">= 0"])
+    s.add_runtime_dependency(%q<string-cases>.freeze, [">= 0"])
+    s.add_development_dependency(%q<best_practice_project>.freeze, [">= 0"])
+    s.add_development_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_development_dependency(%q<factory_bot>.freeze, [">= 0"])
+    s.add_development_dependency(%q<jdbc-sqlite3>.freeze, [">= 0"])
+    s.add_development_dependency(%q<juwelier>.freeze, [">= 0"])
+    s.add_development_dependency(%q<orm_adapter>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rdoc>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rspec>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rubocop>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rubocop-performance>.freeze, [">= 0"])
+    s.add_development_dependency(%q<rubocop-rspec>.freeze, [">= 0"])
+    s.add_development_dependency(%q<sqlite3>.freeze, [">= 0"])
   else
-    s.add_dependency(%q<array_enumerator>.freeze, [">= 0.0.10"])
-    s.add_dependency(%q<auto_autoloader>.freeze, [">= 0.0.1"])
-    s.add_dependency(%q<baza>.freeze, ["~> 0.0.21"])
-    s.add_dependency(%q<html_gen>.freeze, [">= 0.0.12"])
-    s.add_dependency(%q<simple_delegate>.freeze, [">= 0.0.2"])
-    s.add_dependency(%q<string-cases>.freeze, [">= 0.0.3"])
+    s.add_dependency(%q<array_enumerator>.freeze, [">= 0"])
+    s.add_dependency(%q<auto_autoloader>.freeze, [">= 0"])
+    s.add_dependency(%q<baza>.freeze, [">= 0"])
+    s.add_dependency(%q<html_gen>.freeze, [">= 0"])
+    s.add_dependency(%q<simple_delegate>.freeze, [">= 0"])
+    s.add_dependency(%q<string-cases>.freeze, [">= 0"])
     s.add_dependency(%q<best_practice_project>.freeze, [">= 0"])
-    s.add_dependency(%q<bundler>.freeze, ["~> 1.0"])
-    s.add_dependency(%q<factory_girl>.freeze, [">= 0"])
+    s.add_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_dependency(%q<factory_bot>.freeze, [">= 0"])
     s.add_dependency(%q<jdbc-sqlite3>.freeze, [">= 0"])
-    s.add_dependency(%q<jeweler>.freeze, ["= 2.3.6"])
+    s.add_dependency(%q<juwelier>.freeze, [">= 0"])
     s.add_dependency(%q<orm_adapter>.freeze, [">= 0"])
-    s.add_dependency(%q<rdoc>.freeze, ["~> 3.12"])
-    s.add_dependency(%q<rspec>.freeze, ["= 3.6.0"])
-    s.add_dependency(%q<rubocop>.freeze, ["= 0.37.0"])
+    s.add_dependency(%q<rdoc>.freeze, [">= 0"])
+    s.add_dependency(%q<rspec>.freeze, [">= 0"])
+    s.add_dependency(%q<rubocop>.freeze, [">= 0"])
+    s.add_dependency(%q<rubocop-performance>.freeze, [">= 0"])
+    s.add_dependency(%q<rubocop-rspec>.freeze, [">= 0"])
     s.add_dependency(%q<sqlite3>.freeze, [">= 0"])
   end
 end

data/lib/baza_models/autoloader.rb

@@ -125,6 +125,7 @@ private
       orig_model = @models.detect { |array_model| array_model.id == orig_model_id }
 
       raise "Already autoloaded?" if orig_model.autoloads.key?(autoload_name)
+
       orig_model.autoloads[autoload_name] = model
 
       result.fetch(:models) << model

data/lib/baza_models/errors.rb

@@ -1,5 +1,6 @@
 class BazaModels::Errors
   class InvalidRecord < RuntimeError; end
+
   class RecordNotFound < RuntimeError; end
 
   def initialize
@@ -18,9 +19,7 @@ class BazaModels::Errors
       errors.each do |error|
         message = ""
 
-        unless attribute_name == :base
-          message << "#{StringCases.snake_to_camel(attribute_name)} "
-        end
+        message << "#{StringCases.snake_to_camel(attribute_name)} " unless attribute_name == :base
 
         message << error
         messages << message

data/lib/baza_models/helpers/ransacker_helper.rb

@@ -2,7 +2,7 @@ module BazaModels::Helpers::RansackerHelper
   def bm_paginate_content(collection)
     require "html_gen"
 
-    new_params = params.dup
+    new_params = params.dup.permit!
     current_page = collection.page
     total_pages = collection.total_pages
 
@@ -53,7 +53,7 @@ module BazaModels::Helpers::RansackerHelper
       new_params[:q][:s] = sort_asc
     end
 
-    href = url_for(new_params)
+    href = url_for(new_params.permit!)
 
     element = HtmlGen::Element.new(:a, str: label, attr: {href: href})
     element.html

data/lib/baza_models/model/belongs_to_relations.rb

@@ -25,9 +25,7 @@ module BazaModels::Model::BelongsToRelations
       @relationships[relation_name] = relation
 
       define_method(relation_name) do
-        if (model = @changes[relation_name])
-          model
-        elsif (model = autoloads[relation_name])
+        if (model = @changes[relation_name]) || (model = autoloads[relation_name])
           model
         else
           if relation[:class_name]

data/lib/baza_models/model/has_many_relations.rb

@@ -4,9 +4,9 @@ module BazaModels::Model::HasManyRelations
   end
 
   module ClassMethods
-    # rubocop:disable Style/PredicateName
+    # rubocop:disable Naming/PredicateName
     def has_many(relation_name, *all_args)
-      # rubocop:enable Style/PredicateName
+      # rubocop:enable Naming/PredicateName
 
       args = all_args.pop
 

data/lib/baza_models/model/has_one_relations.rb

@@ -4,9 +4,9 @@ module BazaModels::Model::HasOneRelations
   end
 
   module ClassMethods
-    # rubocop:disable Style/PredicateName
+    # rubocop:disable Naming/PredicateName
     def has_one(relation_name, *all_args)
-      # rubocop:enable Style/PredicateName
+      # rubocop:enable Naming/PredicateName
 
       args = all_args.pop
 

data/lib/baza_models/model/manipulation.rb

@@ -30,9 +30,9 @@ module BazaModels::Model::Manipulation
       fire_callbacks(:after_save)
       fire_callbacks(:after_create) if new_record
 
-      return true
+      true
     else
-      return false
+      false
     end
   end
 
@@ -47,14 +47,12 @@ module BazaModels::Model::Manipulation
 
     @data[:id] = db.insert(table_name, @changes, return_id: true)
 
-    if @autoloads
-      @autoloads.each do |relation_name, collection|
-        relation = self.class.relationships.fetch(relation_name)
+    @autoloads&.each do |relation_name, collection|
+      relation = self.class.relationships.fetch(relation_name)
 
-        collection.each do |model|
-          model.assign_attributes(relation.fetch(:foreign_key) => id)
-          model.create! if model.new_record?
-        end
+      collection.each do |model|
+        model.assign_attributes(relation.fetch(:foreign_key) => id)
+        model.create! if model.new_record?
       end
     end
 
@@ -63,7 +61,7 @@ module BazaModels::Model::Manipulation
 
   def create!
     if create
-      return true
+      true
     else
       raise BazaModels::Errors::InvalidRecord, errors.full_messages.join(". ")
     end
@@ -71,25 +69,29 @@ module BazaModels::Model::Manipulation
 
   def save!(args = {})
     if save(args)
-      return true
+      true
     else
       raise BazaModels::Errors::InvalidRecord, errors.full_messages.join(". ")
     end
   end
 
-  def update_attributes(attributes)
+  def update(attributes)
     assign_attributes(attributes)
     save
   end
 
-  def update_attributes!(attributes)
-    update!(attributes)
-  end
-
   def update!(attributes)
     raise BazaModels::Errors::InvalidRecord, @errors.full_messages.join(". ") unless update_attributes(attributes)
   end
 
+  def update_attributes(attributes)
+    update(attributes)
+  end
+
+  def update_attributes!(attributes)
+    update!(attributes)
+  end
+
   def assign_attributes(attributes)
     @changes.merge!(real_attributes(attributes))
   end
@@ -97,7 +99,7 @@ module BazaModels::Model::Manipulation
   def destroy
     if new_record?
       errors.add(:base, "cannot destroy new record")
-      return false
+      false
     else
       fire_callbacks(:before_destroy)
 
@@ -108,7 +110,7 @@ module BazaModels::Model::Manipulation
 
       db.delete(table_name, id: id)
       fire_callbacks(:after_destroy)
-      return true
+      true
     end
   end
 

data/lib/baza_models/model/queries.rb

@@ -7,6 +7,7 @@ module BazaModels::Model::Queries
     def find(id)
       row = db.select(table_name, {id: id}, limit: 1).fetch
       raise BazaModels::Errors::RecordNotFound, "Record not found by ID: #{id}" unless row
+
       new(row, init: true)
     end
 
@@ -23,6 +24,7 @@ module BazaModels::Model::Queries
     def find_or_initialize_by(data)
       model = find_by(data)
       return model if model
+
       new(data)
     end
 

data/lib/baza_models/model/scopes.rb

@@ -9,6 +9,7 @@ module BazaModels::Model::Scopes
       name = name.to_sym
 
       raise "Such a scope already exists" if @scopes.key?(name)
+
       @scopes[name] = {blk: blk}
 
       (class << self; self; end).__send__(:define_method, name) do

data/lib/baza_models/model/validations.rb

@@ -22,6 +22,7 @@ module BazaModels::Model::Validations
 
       validators[attribute_name].each do |validator|
         next unless validator.fire?(self)
+
         validator.validate(self, attribute_value)
       end
     end

data/lib/baza_models/model.rb

@@ -16,6 +16,7 @@ class BazaModels::Model
 
   attr_accessor :data, :db
   attr_reader :changes, :errors
+  attr_writer :db, :table_name
 
   # Define all callback methods.
   CALLBACK_TYPES = [
@@ -45,7 +46,7 @@ class BazaModels::Model
 
   QUERY_METHODS = [
     :average, :all, :any?, :destroy_all, :each, :empty?, :ids, :maximum, :minimum, :none?, :count, :find, :first, :find_first, :last,
-    :length, :size, :select, :includes, :joins, :group, :where, :order, :pluck, :preloads, :sum, :limit, :to_a, :accessible_by, :ransack
+    :length, :size, :select, :includes, :joins, :group, :where, :order, :pluck, :preloads, :sum, :limit, :accessible_by, :ransack
   ].freeze
   QUERY_METHODS.each do |query_method|
     (class << self; self; end).__send__(:define_method, query_method) do |*args, &blk|
@@ -75,9 +76,7 @@ class BazaModels::Model
     end
   end
 
-  # rubocop:disable Style/TrivialAccessors
   def new_record?
-    # rubocop:enable Style/TrivialAccessors
     @new_record
   end
 
@@ -87,21 +86,23 @@ class BazaModels::Model
 
   def db
     return @db if @db
+
     @db ||= self.class.db
   end
 
-  attr_writer :db
-
   def self.attribute_names
     init_model
     @table.columns.map(&:name).map(&:clone)
   end
 
   def self.db
-    @db = nil if @db && @db.closed?
+    @db = nil if
+@db&.closed?
     return @db if @db
+
     @db ||= BazaModels.primary_db
     raise "No Baza database has been configured" unless @db
+
     @db
   end
 
@@ -144,8 +145,6 @@ class BazaModels::Model
     attr_writer :db, :table_name
   end
 
-  attr_writer :table_name
-
   def autoloads
     @autoloads ||= {}
     @autoloads
@@ -187,7 +186,7 @@ class BazaModels::Model
   end
 
   def to_param
-    id.to_s if id
+    id&.to_s
   end
 
   def to_key
@@ -201,6 +200,7 @@ class BazaModels::Model
   def reload
     @data = db.single(table_name, {id: id}, limit: 1)
     raise BazaModels::Errors::RecordNotFound unless @data
+
     @changes = {}
     self
   end
@@ -233,15 +233,15 @@ class BazaModels::Model
     return false unless self.class == other.class
 
     if new_record? && other.new_record?
-      return merged_data == other.__send__(:merged_data)
+      merged_data == other.__send__(:merged_data)
     else
-      return id == other.id
+      id == other.id
     end
   end
 
-  # rubocop:disable Style/PredicateName
+  # rubocop:disable Naming/PredicateName
   def has_attribute?(name)
-    # rubocop:enable Style/PredicateName
+    # rubocop:enable Naming/PredicateName
     self.class.column_names.include?(name.to_s)
   end
 
@@ -260,6 +260,7 @@ class BazaModels::Model
 
   def read_attribute(attribute_name)
     return @changes.fetch(attribute_name) if @changes.key?(attribute_name)
+
     @data.fetch(attribute_name)
   end
 
@@ -271,6 +272,7 @@ class BazaModels::Model
     changed = false
     @changes.each do |key, value|
       next if @data.fetch(key) == value
+
       changed = true
       break
     end
@@ -283,10 +285,7 @@ protected
   class << self
     attr_reader :__blank_attributes
   end
-
-  # rubocop:disable Style/TrivialAccessors
   def self.model_initialized?
-    # rubocop:enable Style/TrivialAccessors
     @model_initialized
   end
 
@@ -310,11 +309,7 @@ protected
     end
 
     define_method("#{column_name}_changed?") do
-      if @changes.key?(column_name) && @changes.fetch(column_name) != @data.fetch(column_name)
-        true
-      else
-        false
-      end
+      @changes.key?(column_name) && @changes.fetch(column_name) != @data.fetch(column_name)
     end
 
     define_method("will_save_change_to_#{column_name}?") do
@@ -359,12 +354,11 @@ protected
     attributes.each do |attribute_name, attribute_value|
       belongs_to_relations = self.class.instance_variable_get(:@belongs_to_relations)
 
-      if belongs_to_relations
-        belongs_to_relations.each do |relation|
-          if attribute_name.to_s == relation[:relation_name].to_s
-            attribute_name = :"#{attribute_name}_id"
-            attribute_value = attribute_value.id if attribute_value
-          end
+
+      belongs_to_relations&.each do |relation|
+        if attribute_name.to_s == relation[:relation_name].to_s
+          attribute_name = :"#{attribute_name}_id"
+          attribute_value = attribute_value.id if attribute_value
         end
       end
 
@@ -385,16 +379,19 @@ protected
 
   def attribute_before_last_save(attribute_name)
     return @before_last_save.fetch(attribute_name) if @before_last_save.key?(attribute_name)
+
     @data.fetch(attribute_name)
   end
 
   def will_save_change_to_attribute?(attribute_name)
     return true if @changes.key?(attribute_name) && @changes[attribute_name] != data[attribute_name]
+
     false
   end
 
   def method_missing(method_name, *args, &blk)
     return @data.fetch(method_name) if @data.key?(method_name)
+
     super
   end
 end

data/lib/baza_models/query/inspector.rb

@@ -36,6 +36,7 @@ private
 
     relationship_pair = @model.relationships.detect { |key, _value| key == argument }
     raise "Could not find a relationship on #{@model.name} by that name: #{argument}" unless relationship_pair
+
     relationship = relationship_pair[1]
 
     table_name = relationship.fetch(:table_name)
@@ -60,6 +61,7 @@ private
 
       relationship_pair = @model.relationships.detect { |relationship_name, _relationship| relationship_name == key }
       raise "Could not find a relationship on #{@model.name} by that name: #{value}" unless relationship_pair
+
       relationship = relationship_pair[1]
 
       BazaModels::Query::Inspector.new(

data/lib/baza_models/query/pagination.rb

@@ -1,10 +1,8 @@
 module BazaModels::Query::Pagination
   def current_page
-    if @page
-      @page
-    else
-      raise "Page has not been set"
-    end
+    raise "Page has not been set" unless @page
+
+    @page
   end
 
   def out_of_bounds?
@@ -44,11 +42,11 @@ module BazaModels::Query::Pagination
   end
 
   def total_pages
-    pages_count = (count.to_f / per.to_f)
+    pages_count = (count.to_f / per)
 
     pages_count = 1 if pages_count.nan? || pages_count == Float::INFINITY
     pages_count = pages_count.ceil
-    pages_count = 1 if pages_count == 0
+    pages_count = 1 if pages_count.zero?
     pages_count
   end
 end

data/lib/baza_models/query/sql_generator.rb

@@ -9,7 +9,7 @@ class BazaModels::Query::SqlGenerator
     end
   end
 
-  def to_sql
+  def to_sql # rubocop:disable Metrics/CyclomaticComplexity, Metrics/AbcSize, Metrics/PerceivedComplexity
     sql = "SELECT "
 
     if @selects.empty?
@@ -68,9 +68,9 @@ class BazaModels::Query::SqlGenerator
         end
 
         if @reverse_order
-          if order =~ /\s+desc/i
+          if /\s+desc/i.match?(order)
             order = order.gsub(/\s+desc/i, " ASC")
-          elsif order =~ /\s+asc/i
+          elsif /\s+asc/i.match?(order)
             order = order.gsub(/\s+asc/i, " DESC")
           else
             order = "#{order} DESC"