updateable_views_inheritance 1.5.1 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4134e2e8c505f5ca5c6d53762497a18b168b5cb43027a121f337908d1595d5b
-  data.tar.gz: 7dc39d94d346f6efef8862c2482afe72d97f745b822b59b313041dcb21fb9ebc
+  metadata.gz: 9822cbe06d8820cf3d4dcf96e4bf9c527f130b16d83edc1335858e250e392b55
+  data.tar.gz: 32755fb6541e0acb5ee155cb5ceb94021a9c838a28b73c033cc0b8e5c8a60110
 SHA512:
-  metadata.gz: e50eeaa1e8e686fe94bd47f02f8109335b7f38efa94e65c108dc5e0c5dcc7509e476ee14ce3de31160673d6bcce0f540dcbf0142f774b1db7f017ed106251b15
-  data.tar.gz: e2e3e32f9b7e6f77c113c1d80701f6a1345a4352df0779d5cfc3f450bfe11fe04c6b65955ba07de440c4733df71ba84073294f274e6c14e4b80fda1024391208
+  metadata.gz: 6fbed4ffbc44f4841eeb0613fe64196f9e38b7e6f01c94aaa3a786d70401b3685a12c7a078d7a2bf26e58e6c615511619abc8b8d4195cf3f0bfddeaafc80ce24
+  data.tar.gz: b954e8fc8706faa9e081710292992e1cd60fd3003aa4e7b500f9c52ee71e4c5c5aab944277ab7d48b94f1279f9797dabef61c73cb8b7781dae2a4a2b5178892d

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 1.8.0 (19 April 2026)
+
+Upgrade to Rails 8.1
+
+## 1.7.0 (11 March 2025)
+
+Upgrade to Rails 7 (tested with 7.0, 7.1, 7.2 and ruby 2.7.3, 3.1.6)
+
 ## 1.5.1 (27 February 2025)
 
 Upgrade to Rails 5.1

data/Gemfile

@@ -3,9 +3,5 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in updateable_views_inheritance.gemspec
 gemspec
 
-if RUBY_VERSION > "2"
-  gem 'byebug'
-else
-  gem 'debugger'
-end
+gem 'byebug'
 

data/lib/updateable_views_inheritance/active_record_extensions.rb

@@ -9,7 +9,7 @@ module UpdateableViewsInheritance# :nodoc:
         if object.class.name == self.name || self.disable_inheritance_instantiation
           object
         else
-          object.class.find(attributes.with_indifferent_access[:id])
+          object.class.find(attributes.to_h.with_indifferent_access[:id])
         end
       end
     end

data/lib/updateable_views_inheritance/postgresql_adapter.rb

@@ -90,19 +90,29 @@ module ActiveRecord #:nodoc:
           if parent
             reset_pk_sequence!(parent, pk, sequence)
           else
-            unless pk and sequence
+            unless pk && sequence
               default_pk, default_sequence = pk_and_sequence_for(table)
+
               pk ||= default_pk
               sequence ||= default_sequence
             end
-            if pk
-              if sequence
-                select_value <<-SQL, 'Reset sequence'
-                  SELECT setval('#{sequence}', (SELECT COALESCE(MAX(#{pk})+(SELECT increment_by FROM #{sequence}), (SELECT min_value FROM #{sequence})) FROM #{table}), false)
-                SQL
-              else
-                @logger.warn "#{table} has primary key #{pk} with no default sequence" if @logger
+
+            if @logger && pk && !sequence
+              @logger.warn "#{table} has primary key #{pk} with no default sequence."
+            end
+
+            if pk && sequence
+              quoted_sequence = quote_table_name(sequence)
+              max_pk = query_value("SELECT MAX(#{quote_column_name pk}) FROM #{quote_table_name(table)}", "SCHEMA")
+              if max_pk.nil?
+                if postgresql_version >= 100000
+                  minvalue = query_value("SELECT seqmin FROM pg_sequence WHERE seqrelid = #{quote(quoted_sequence)}::regclass", "SCHEMA")
+                else
+                  minvalue = query_value("SELECT min_value FROM #{quoted_sequence}", "SCHEMA")
+                end
               end
+
+              query_value("SELECT setval(#{quote(quoted_sequence)}, #{max_pk ? max_pk : minvalue}, #{max_pk ? true : false})", "SCHEMA")
             end
           end
         end

data/lib/updateable_views_inheritance/version.rb

@@ -1,3 +1,3 @@
 module UpdateableViewsInheritance
-  VERSION = "1.5.1"
+  VERSION = "1.8.0"
 end

data/test/content_test.rb

@@ -2,11 +2,10 @@ require_relative 'test_helper'
 
 class UpdateableViewsInheritanceContentTest < ActiveSupport::TestCase
   def setup
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 5)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(5)
     ActiveRecord::FixtureSet.reset_cache
   end
 
-
   def test_find
     ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :steam_locomotives)
     locomotive =  Locomotive.find(1)
@@ -33,10 +32,7 @@ class UpdateableViewsInheritanceContentTest < ActiveSupport::TestCase
     ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :electric_locomotives)
     ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :steam_locomotives)
 
-    binds = [[ElectricLocomotive.columns.find { |c| c.name == 'electricity_consumption'}, 40],
-             [ElectricLocomotive.columns.find { |c| c.name == 'max_speed'},              120],
-             [ElectricLocomotive.columns.find { |c| c.name == 'name'},                'BoBo'],
-             [ElectricLocomotive.columns.find { |c| c.name == 'type'},  'ElectricLocomotive']]
+    binds = [40, 120, 'BoBo', 'ElectricLocomotive']
     res = ActiveRecord::Base.connection.exec_query(<<-SQL, 'Test prepared statement', binds)
       INSERT INTO electric_locomotives (electricity_consumption, max_speed, name, type) VALUES ($1, $2, $3, $4) RETURNING id
     SQL
@@ -63,7 +59,7 @@ class UpdateableViewsInheritanceContentTest < ActiveSupport::TestCase
   def test_update
     ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', :steam_locomotives)
     steam_locomotive = Locomotive.find(1)
-    steam_locomotive.update_attributes( :name => 'Rocket')
+    steam_locomotive.update(name: 'Rocket')
     steam_locomotive.reload
     assert_equal 'Rocket', steam_locomotive.name
   end

data/test/deep_hierarchy_test.rb

@@ -2,12 +2,12 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 
 class DeepHierarchyTest < ActiveSupport::TestCase
   def setup
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 8)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(8)
 
     ActiveRecord::FixtureSet.reset_cache
     # order of fixtures is important for the test - last loaded should not be with max(id)
-    %w(boats electric_trains rack_trains steam_trains cars maglev_trains bicycles).each do |f|
-      ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', f)
+    %w(boats electric_trains rack_trains steam_trains maglev_trains bicycles).each do |f|
+      ActiveRecord::FixtureSet.create_fixtures("#{__dir__}/fixtures/", f)
     end
     @connection = ActiveRecord::Base.connection
   end
@@ -41,7 +41,7 @@ class DeepHierarchyTest < ActiveSupport::TestCase
 
   def test_single_table_inheritance_deeper_hierarchy_contents
     mag = MaglevTrain.first
-    assert_equal [mag.id, mag.name, mag.number_of_rails, mag.max_speed, mag.magnetic_field, (sprintf("%.2f",mag.electricity_consumption))], (@connection.query("SELECT id, name, number_of_rails, max_speed, magnetic_field, electricity_consumption FROM all_vehicles WHERE id=#{mag.id}").first)
+    assert_equal [mag.id, mag.name, mag.number_of_rails, mag.max_speed, mag.magnetic_field, mag.electricity_consumption], (@connection.query("SELECT id, name, number_of_rails, max_speed, magnetic_field, electricity_consumption FROM all_vehicles WHERE id=#{mag.id}").first)
   end
 
   class OrderColumnsInAggregateView < ActiveRecord::Migration[4.2]

data/test/dummy/config/application.rb

@@ -7,6 +7,7 @@ require "updateable_views_inheritance"
 
 module Dummy
   class Application < Rails::Application
+    config.load_defaults 8.1
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
@@ -42,12 +43,6 @@ module Dummy
     # This is necessary if your schema can't be completely dumped by the schema dumper,
     # like if you have constraints or database-specific column types
     # config.active_record.schema_format = :sql
-
-    # Enable the asset pipeline
-    config.assets.enabled = true
-
-    # Version of your assets, change this if you want to expire all your assets
-    config.assets.version = '1.0'
   end
 end
 

data/test/instantiation_test.rb

@@ -2,8 +2,7 @@ require_relative 'test_helper'
 
 class InstantiationTest < ActiveSupport::TestCase
   def setup
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 7)
-
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(7)
     ActiveRecord::FixtureSet.reset_cache
     # order of fixtures is important for the test - last loaded should not be with max(id)
     %w[steam_locomotives electric_locomotives maglev_locomotives bicycles].each do |f|

data/test/migration_test.rb

@@ -7,20 +7,20 @@ class UpdateableViewsInheritanceMigrationTest < ActiveSupport::TestCase
   end
 
   def test_create_child_default
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 2)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(2)
     assert_equal %w(coal_consumption id max_speed name type water_consumption),
                  @connection.columns(:steam_locomotives).map{ |c| c.name }.sort
   end
 
   def test_create_child_explicit_table
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 3)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(3)
     assert_equal %w(electricity_consumption id max_speed name type),
                  @connection.columns(:electric_locomotives).map{ |c| c.name }.sort
   end
 
   def test_drop_child
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 3)
-    ActiveRecord::Migrator.down(File.dirname(__FILE__) + '/fixtures/migrations/', 2)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(3)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(2)
     assert_equal %w(steam_locomotives), @connection.views.sort
     assert_equal %w(ar_internal_metadata
                     locomotives

data/test/schema_test.rb

@@ -2,7 +2,7 @@ require_relative 'test_helper'
 
 class SchemaTest < ActiveSupport::TestCase
   def setup
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 5)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(5)
     @connection = ActiveRecord::Base.connection
   end
 
@@ -273,7 +273,7 @@ class SchemaTest < ActiveSupport::TestCase
   end
 
   def test_rebuild_all_parent_and_children_views
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 7)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(7)
     @connection.execute "DROP VIEW all_locomotives" #FIXME: single table inheritance view should be rebuilt as well
     ChangeTablesInTwoInheritanceChains.new.up
 

data/test/single_table_inheritance_test.rb

@@ -2,7 +2,7 @@ require_relative 'test_helper'
 
 class SingleTableInheritanceAggregateViewTest < ActiveSupport::TestCase
   def setup
-    ActiveRecord::Migrator.up(File.dirname(__FILE__) + '/fixtures/migrations/', 6)
+    ActiveRecord::MigrationContext.new("#{__dir__}/fixtures/migrations").migrate(6)
     # order of fixtures is important for the test - last loaded should not be with max(id)
     %w(electric_locomotives maglev_locomotives steam_locomotives).each do |f|
       ActiveRecord::FixtureSet.create_fixtures(File.dirname(__FILE__) + '/fixtures/', f)

data/test/test_helper.rb

@@ -33,9 +33,7 @@ require 'byebug'
 
 class ActiveSupport::TestCase #:nodoc:
   include ActiveRecord::TestFixtures
-  self.fixture_path = "#{File.dirname(__FILE__)}/fixtures/"
   ActiveRecord::Migration.verbose = false
-  # self.use_transactional_fixtures = true
 
   # def teardown
   #   ActiveRecord::Migrator.down(File.dirname(__FILE__) + '/fixtures/migrations/', 1)

data/updateable_views_inheritance.gemspec

@@ -18,14 +18,13 @@ Gem::Specification.new do |s|
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
   s.require_paths = ["lib"]
 
-  s.add_dependency "activerecord", "~> 5.1.7"
+  s.add_dependency "activerecord", '>= 8', '< 9'
   s.add_dependency "pg"
 
-  s.add_development_dependency 'bigdecimal', '1.3.5'
   s.add_development_dependency "bundler"
   s.add_development_dependency "minitest"
   s.add_development_dependency "minitest-reporters"
-  s.add_development_dependency "rails", '~> 5.1.7'
+  s.add_development_dependency "rails"
   s.add_development_dependency "rake"
   s.add_development_dependency "simplecov"
   s.add_development_dependency "simplecov_json_formatter"

metadata

@@ -1,30 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: updateable_views_inheritance
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.8.0
 platform: ruby
 authors:
 - Sava Chankov
 - Denitsa Belogusheva
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-27 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '8'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '8'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '9'
 - !ruby/object:Gem::Dependency
   name: pg
   requirement: !ruby/object:Gem::Requirement
@@ -39,20 +44,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: bigdecimal
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 1.3.5
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 1.3.5
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -99,16 +90,16 @@ dependencies:
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 5.1.7
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -270,7 +261,6 @@ homepage: http://github.com/tutuf/updateable_views_inheritance
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -285,8 +275,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Class table inheritance for ActiveRecord
 test_files: