dynamic_migrations 3.8.0 → 3.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 437a84189a939e185f89db2de8395bc1fb8dc9f159b3948a97dcc5b04c4fedb7
-  data.tar.gz: 6e1df320ac06c0b0ca9d6aeb3abf684e47b9daac4ee1def93180de801f73125b
+  metadata.gz: 1235cefb0bc31fd54a095fbef1111be30cb13f89bb6c4358364f35594ffe95ff
+  data.tar.gz: e9050d22292034176525471025a8981fba83c8945b8a81f754c3c2a222cc4538
 SHA512:
-  metadata.gz: c2ce93a8d273abfb0c2ede80b7564216a8409bcd8a43b8fe6c85043eed83d750f8a2b2e53996afdc241e13db9f6cdf2c3f52034a5e93965663684bc121237b54
-  data.tar.gz: bb3378cc475e8e3c16dd2d3a9a5754484da539d3ba1feb4839d2312c1f24cb147ea11534ba3a9bfbcda827bee4bc5ae18cc2c7a2b23bdd67c84a23503fc4fcab
+  metadata.gz: 2be636e7faf4eb9d05d8f6fb507603aa362f341982a44f99e7e35dd365481fe7042b04e1dcd743981ea321082af61be38e8a1b73c31109e09252f3736c51afb3
+  data.tar.gz: 9db574a68a9009bee00bb5e8d16604b577e0d33a999ece3df62144516a15ba8f0187e624346ab87913f3c04805de4140eb0dd5b68c5d0f1fd30dff7010efef97

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [3.8.1](https://github.com/craigulliott/dynamic_migrations/compare/v3.8.0...v3.8.1) (2023-10-06)
+
+
+### Bug Fixes
+
+* reusing exiting connection when with_connection is called (it was trying to open a new connection and caused an error) ([840a5cd](https://github.com/craigulliott/dynamic_migrations/commit/840a5cda04d6bc49d73160d2bad7271780027c84))
+
 ## [3.8.0](https://github.com/craigulliott/dynamic_migrations/compare/v3.7.0...v3.8.0) (2023-10-04)
 
 

data/lib/dynamic_migrations/postgres/server/database/connection.rb

@@ -33,19 +33,31 @@ module DynamicMigrations
             end
           end
 
+          def connected?
+            !@connection.nil?
+          end
+
           # Opens a connection to the database server, and yields the provided block
           # before automatically closing the connection again. This is useful for
           # executing one time queries against the database server.
           def with_connection &block
-            # create a temporary connection to the server
-            connect
+            # create a temporary connection to the server (unless we are already connected)
+            already_connected = connected?
+            unless already_connected
+              connect
+            end
+
             # perform work with the connection
             # todo: `yield connection` would have been preferred, but rbs/steep doesnt understand that syntax
             if block.is_a? Proc
               result = block.call connection
             end
-            # close the connection
-            disconnect
+
+            # close the connection (unless we were already connected)
+            if already_connected
+              disconnect
+            end
+
             # return whever was returned from within the block
             result
           end

data/lib/dynamic_migrations/version.rb

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

data/sig/dynamic_migrations/postgres/server/database/connection.rbs

@@ -7,6 +7,7 @@ module DynamicMigrations
 
           def connect: -> PG::Connection
           def connection: -> PG::Connection
+          def connected?: -> bool
           def disconnect: -> void
           def with_connection: -> untyped
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dynamic_migrations
 version: !ruby/object:Gem::Version
-  version: 3.8.0
+  version: 3.8.1
 platform: ruby
 authors:
 - Craig Ulliott
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-10-04 00:00:00.000000000 Z
+date: 2023-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg