fix bug never removing check constraint

instructure · Sep 7, 2023 · f4d96fc · f4d96fc
1 parent 558c666
commit f4d96fc
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    activerecord-pg-extensions (0.5.0)
+    activerecord-pg-extensions (0.5.1)
       activerecord (~> 7.0.0)
       railties (~> 7.0.0)
 

diff --git a/gemfiles/rails_7.0.gemfile.lock b/gemfiles/rails_7.0.gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    activerecord-pg-extensions (0.5.0)
+    activerecord-pg-extensions (0.5.1)
       activerecord (~> 7.0.0)
       railties (~> 7.0.0)
 

diff --git a/lib/active_record/pg_extensions/pessimistic_migrations.rb b/lib/active_record/pg_extensions/pessimistic_migrations.rb
@@ -96,8 +96,8 @@ def add_check_constraint(table_name, expression, if_not_exists: false, **options
       end
 
       if ActiveRecord.version < Gem::Version.new("7.1")
-        def remove_check_constraint(table_name, expression = nil, **options)
-          return if options[:if_exists] && !check_constraint_for(table_name, expression, **options)
+        def remove_check_constraint(table_name, expression = nil, if_exists: false, **options)
+          return if if_exists && !check_constraint_for(table_name, expression, **options)
 
           super
         end

diff --git a/lib/active_record/pg_extensions/version.rb b/lib/active_record/pg_extensions/version.rb
@@ -2,6 +2,6 @@
 
 module ActiveRecord
   module PGExtensions
-    VERSION = "0.5.0"
+    VERSION = "0.5.1"
   end
 end