dynamic_migrations 3.6.10 → 3.6.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93b2822411008957140dcf9902d9e769cfaf10ea99e9ac424f5c57effdaff8f4
-  data.tar.gz: 38e53d341c2f9fd15231f1eea9c670ed1f724d7ce4a6618b587645faafec7356
+  metadata.gz: 79b80c950a523a7f70c930e8c86774fa64009ae689f5164de3320dc8d55e4238
+  data.tar.gz: 649f3e0f23d676e97b8cf3c8389875ae2127701eb405c7c8947bf19de076b7f2
 SHA512:
-  metadata.gz: 1f383da41e391a24896b2ae6264710c891f9ab6a46382f70953cf424ccffcee1644e42123ccbe6cb29391d494bb9058c12c93c81a7cba9f20f20755e8eb43d10
-  data.tar.gz: b77823cc86a9f235be6ad33d6eb48b382c671ab8a60465435f060de14ad981acdc29767645af87e2128c817d9a0cdd24bc7aca1a9352a6be44289ac2d13688d5
+  metadata.gz: 94c1280b7c62a39f8337d1093731625b22049eab02b642f17a9c8d1e7f067ef90cabb1dcabda980acde0eb38639ddbd1d30c9b740d85535d0250295e032b2983
+  data.tar.gz: '0478e668cf87739da754981dd2be1acd6a21cdfb3ef41609043c12915a455a04ccf99f31e46b3d0a5630002a59ba6b085f65381a86ed1e4d17eeaa796e70b418'

data/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [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)
+
+
+### Bug Fixes
+
+* injecting dependent_function into templated triggers ([9e770a3](https://github.com/craigulliott/dynamic_migrations/commit/9e770a352b176c7b10471e9185ac01feca538c0c))
+* writing enums to the migration files as arrays of strings for more compatibility with special characters ([1910e0f](https://github.com/craigulliott/dynamic_migrations/commit/1910e0f709e5bd560696c2e2b3bd729c7a61e319))
+
 ## [3.6.10](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.9...v3.6.10) (2023-09-13)
 
 

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/enum.rb

@@ -21,7 +21,7 @@ module DynamicMigrations
             code_comment: code_comment,
             migration: <<~RUBY
               create_enum :#{enum.name}, [
-                :#{enum.values.join(",\n  :")}
+                "#{enum.values.join("\",\n  \"")}"
               ]
             RUBY
         end
@@ -40,7 +40,7 @@ module DynamicMigrations
             code_comment: code_comment,
             migration: <<~RUBY
               add_enum_values :#{updated_enum.name}, [
-                :#{added_values.join(",\n  :")}
+                "#{added_values.join("\",\n  \"")}"
               ]
             RUBY
         end

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

@@ -32,7 +32,15 @@ module DynamicMigrations
             end
 
             arguments = template_class.new(trigger, code_comment).fragment_arguments
-            add_fragment(**arguments)
+
+            # we only provide a dependent function if the function has more than one trigger
+            # or is in a different schema, otherwise it is added to the same migration as this
+            # trigger and we don't need to worry about dependencies
+            if trigger.function && (trigger.function.triggers.count > 1 || trigger.table.schema != trigger.function.schema)
+              add_fragment(dependent_function: trigger.function, **arguments)
+            else
+              add_fragment(**arguments)
+            end
 
           # no template, process this as a default trigger (takes all options)
           else

data/lib/dynamic_migrations/postgres/server/database/differences/to_migrations.rb

@@ -50,7 +50,7 @@ module DynamicMigrations
                 process_schema schema_name, differences[:configuration][:schemas][schema_name], differences[:database][:schemas][schema_name]
               end
 
-              # return the migrations organized by schema
+              # return the migrations (they are sorted via a dependency algorithm)
               @generator.migrations
             end
 

data/lib/dynamic_migrations/version.rb

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

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

data/sig/dynamic_migrations/postgres/generator/validation_template_base.rbs

@@ -12,7 +12,7 @@ module DynamicMigrations
         def initialize: (untyped validation, untyped code_comment) -> void
 
         # abstract method (should actually be added to child classes)
-        def fragment_arguments: -> {schema: Postgres::Server::Database::Schema, table: Postgres::Server::Database::Schema::Table, migration_method: :add_validation, object: Postgres::Server::Database::Schema::Table::Validation, code_comment: String?, migration: String}
+        def fragment_arguments: -> {schema: Postgres::Server::Database::Schema, table: Postgres::Server::Database::Schema::Table, migration_method: Symbol, object: untyped, code_comment: String?, migration: String, dependent_function: Postgres::Server::Database::Schema::Function?}
 
         private
         def assert_not_deferred!: -> nil

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dynamic_migrations
 version: !ruby/object:Gem::Version
-  version: 3.6.10
+  version: 3.6.12
 platform: ruby
 authors:
 - Craig Ulliott