dynamic_migrations 3.6.11 → 3.6.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f31ee44c0dced52756c6f563ea8b321b0cbce5d6cd53f7ef7fb796efb5f7e58a
-  data.tar.gz: f3151d84b1574d421d9c122310ce3394f98d4920aa571dc168de2858c91cdc7a
+  metadata.gz: 9ecc60f6bad6a321991985b6e7cfd024486ec79caa1b49b143caf3c844d28275
+  data.tar.gz: b16c5bb261de851a877d4b6e6067b2767badf82f895ce25e0816286d11ef83e8
 SHA512:
-  metadata.gz: c090b0758ed52dd8b21a5327e83f6d41f23afc5ad3c53bf265166a85245738c37c5188b15dfed27bb890b36db62896590c7dea3fc8d80e130120d43bd548e560
-  data.tar.gz: 74eb4c9ad0d25987f588ccdc0239e460bbc7169a82cec5fc7c837077e822570ac11d61ae7a136a27f49a5c7093c2e91100d22dd3247729bbbb63fd9c316e19f2
+  metadata.gz: 182c7cbc69b0ba406147e7e0d972887ec44d8171e00eb75b92a531702cd4abfaf18ab9c7e319ac7771f25696968582b56cf18fe275652f71f2426d1a1b6e363a
+  data.tar.gz: b0f0de5ca2c65380c25c80fe23c53942a576e4f4c9e7e3984d56dbf67d9afac6e2b16153f1d70012b7f0fb1dfd50b1b13e2cdebcf1c0333ea2cd86f90ded33df

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [3.6.13](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.12...v3.6.13) (2023-09-14)
+
+
+### Bug Fixes
+
+* providing foreign_schema_name to create foreign key constraint, and only providing non default arguments ([e45d499](https://github.com/craigulliott/dynamic_migrations/commit/e45d4994a9b947f0d626bdd4b0e2a36136412370))
+
+## [3.6.12](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.11...v3.6.12) (2023-09-13)
+
+
+### Bug Fixes
+
+* removing unused argument for create_function migrator ([bdb2ad5](https://github.com/craigulliott/dynamic_migrations/commit/bdb2ad58ff0c5aa3520eb38ab0d16b823014ce6e))
+
 ## [3.6.11](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.10...v3.6.11) (2023-09-13)
 
 

data/lib/dynamic_migrations/active_record/migrators/foreign_key_constraint.rb

@@ -11,7 +11,7 @@ module DynamicMigrations
         # because rails migrations don't support composite (multiple column) foreign keys
         # column_names and foreign_column_names can be a single column name or
         # an array of column names
-        def add_foreign_key table_name, column_names, foreign_schema_name, foreign_table_name, foreign_column_names, name:, initially_deferred: false, deferrable: false, on_delete: :no_action, on_update: :no_action, comment: nil
+        def add_foreign_key table_name, column_names, foreign_table_name, foreign_column_names, name:, foreign_schema: nil, initially_deferred: false, deferrable: false, on_delete: :no_action, on_update: :no_action, comment: nil
           if initially_deferred == true && deferrable == false
             raise DeferrableOptionsError, "A constraint can only be initially deferred if it is also deferrable"
           end
@@ -19,6 +19,8 @@ module DynamicMigrations
           # convert single column names into arrays, this simplifies the logic below
           column_names = column_names.is_a?(Array) ? column_names : [column_names]
           foreign_column_names = foreign_column_names.is_a?(Array) ? foreign_column_names : [foreign_column_names]
+          # default to the current schema
+          foreign_schema_name = foreign_schema.nil? ? schema_name : foreign_schema
 
           # allow it to be deferred, and defer it by default
           deferrable_sql = if initially_deferred

data/lib/dynamic_migrations/active_record/migrators/function.rb

@@ -9,7 +9,7 @@ module DynamicMigrations
         end
 
         # create a postgres function
-        def create_function table_name, function_name, comment: nil, &block
+        def create_function function_name, comment: nil, &block
           unless block
             raise MissingFunctionBlockError, "create_function requires a block"
           end
@@ -30,7 +30,7 @@ module DynamicMigrations
         end
 
         # update a postgres function
-        def update_function table_name, function_name, comment: nil, &block
+        def update_function function_name, comment: nil, &block
           unless block
             raise MissingFunctionBlockError, "create_function requires a block"
           end

data/lib/dynamic_migrations/postgres/generator/foreign_key_constraint.rb

@@ -12,13 +12,26 @@ module DynamicMigrations
           f_c_names = foreign_key_constraint.foreign_column_names
           foreign_column_names = (f_c_names.count == 1) ? ":#{f_c_names.first}" : "[:#{f_c_names.join(", :")}]"
 
-          options = {
-            name: ":#{foreign_key_constraint.name}",
-            initially_deferred: foreign_key_constraint.initially_deferred,
-            deferrable: foreign_key_constraint.deferrable,
-            on_delete: ":#{foreign_key_constraint.on_delete}",
-            on_update: ":#{foreign_key_constraint.on_update}"
-          }
+          options = {}
+          options[:name] = ":#{foreign_key_constraint.name}"
+
+          # only provide values if they are different from the defaults
+          unless foreign_key_constraint.initially_deferred == false
+            options[:initially_deferred] = foreign_key_constraint.initially_deferred
+          end
+          unless foreign_key_constraint.deferrable == false
+            options[:deferrable] = foreign_key_constraint.deferrable
+          end
+          unless foreign_key_constraint.on_delete == :no_action
+            options[:on_delete] = ":#{foreign_key_constraint.on_delete}"
+          end
+          unless foreign_key_constraint.on_update == :no_action
+            options[:on_update] = ":#{foreign_key_constraint.on_update}"
+          end
+          unless foreign_key_constraint.table.schema.name == foreign_key_constraint.foreign_schema_name
+            options[:foreign_schema] = ":#{foreign_key_constraint.foreign_schema_name}"
+          end
+
           unless foreign_key_constraint.description.nil?
             options[:comment] = <<~RUBY
               <<~COMMENT
@@ -27,6 +40,10 @@ module DynamicMigrations
             RUBY
           end
 
+          if foreign_key_constraint.table.schema != foreign_key_constraint.foreign_table.schema
+            options[:foreign_schema] = foreign_key_constraint.foreign_table.schema.name.to_s
+          end
+
           options_syntax = options.map { |k, v| "#{k}: #{v}" }.join(", ")
 
           add_fragment schema: foreign_key_constraint.table.schema,

data/lib/dynamic_migrations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DynamicMigrations
-  VERSION = "3.6.11"
+  VERSION = "3.6.13"
 end

data/sig/dynamic_migrations/active_record/migrators/foreign_key_constraint.rbs

@@ -5,7 +5,7 @@ module DynamicMigrations
   module ActiveRecord
     module Migrators
       module ForeignKeyConstraint
-        def add_foreign_key: (Symbol table_name, Array[Symbol] column_names, Symbol foreign_schema_name, Symbol foreign_table_name, Array[Symbol] foreign_column_names, name: Symbol, ?initially_deferred: bool, ?deferrable: bool, ?on_delete: Symbol, ?on_update: Symbol, ?comment: String?) -> void
+        def add_foreign_key: (Symbol table_name, Array[Symbol] column_names, Symbol foreign_table_name, Array[Symbol] foreign_column_names, name: Symbol, ?foreign_schema: Symbol?, ?initially_deferred: bool, ?deferrable: bool, ?on_delete: Symbol, ?on_update: Symbol, ?comment: String?) -> void
         def remove_foreign_key: (Symbol table_name, Symbol name) -> void
         def set_foreign_key_comment: (Symbol table_name, Symbol foreign_key_name, String comment) -> void
         def remove_foreign_key_comment: (Symbol table_name, Symbol foreign_key_name) -> void

data/sig/dynamic_migrations/active_record/migrators/function.rbs

@@ -5,8 +5,8 @@ module DynamicMigrations
   module ActiveRecord
     module Migrators
       module Function
-        def create_function: (Symbol table_name, Symbol function_name, ?comment: String?) -> void
-        def update_function: (Symbol table_name, Symbol function_name, ?comment: String?) -> void
+        def create_function: (Symbol function_name, ?comment: String?) -> void
+        def update_function: (Symbol function_name, ?comment: String?) -> void
         def drop_function: (Symbol function_name) -> void
         def set_function_comment: (Symbol function_name, String comment) -> void
         def remove_function_comment: (Symbol function_name) -> void

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dynamic_migrations
 version: !ruby/object:Gem::Version
-  version: 3.6.11
+  version: 3.6.13
 platform: ruby
 authors:
 - Craig Ulliott
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-09-13 00:00:00.000000000 Z
+date: 2023-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg