dynamic_migrations 3.6.10 → 3.6.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93b2822411008957140dcf9902d9e769cfaf10ea99e9ac424f5c57effdaff8f4
-  data.tar.gz: 38e53d341c2f9fd15231f1eea9c670ed1f724d7ce4a6618b587645faafec7356
+  metadata.gz: f31ee44c0dced52756c6f563ea8b321b0cbce5d6cd53f7ef7fb796efb5f7e58a
+  data.tar.gz: f3151d84b1574d421d9c122310ce3394f98d4920aa571dc168de2858c91cdc7a
 SHA512:
-  metadata.gz: 1f383da41e391a24896b2ae6264710c891f9ab6a46382f70953cf424ccffcee1644e42123ccbe6cb29391d494bb9058c12c93c81a7cba9f20f20755e8eb43d10
-  data.tar.gz: b77823cc86a9f235be6ad33d6eb48b382c671ab8a60465435f060de14ad981acdc29767645af87e2128c817d9a0cdd24bc7aca1a9352a6be44289ac2d13688d5
+  metadata.gz: c090b0758ed52dd8b21a5327e83f6d41f23afc5ad3c53bf265166a85245738c37c5188b15dfed27bb890b36db62896590c7dea3fc8d80e130120d43bd548e560
+  data.tar.gz: 74eb4c9ad0d25987f588ccdc0239e460bbc7169a82cec5fc7c837077e822570ac11d61ae7a136a27f49a5c7093c2e91100d22dd3247729bbbb63fd9c316e19f2

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [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/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.11"
 end

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.11
 platform: ruby
 authors:
 - Craig Ulliott