dynamic_migrations 3.6.2 → 3.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a237bb66c7567c302f4f01ada5c06be36187234cd430da63a753eac9c3c0320
-  data.tar.gz: ab87c5f523b6a682a4e6b648b050e609fe1bfca14bcf1fdebd813b04a5f5091b
+  metadata.gz: b60f89c7a66fe00d71581049f4886a5baa6ca89255dbb47315637bfa6fc5d9be
+  data.tar.gz: cde91d7188a94a3443c8331c6efa9e966a8e86fee236fb156adc577cfebbc156
 SHA512:
-  metadata.gz: 94080c180556da70c9bba292ec6de10de980ed95ff71705c14b320a56b507eae50cccbc1654354184406bce6386b03bde27afdae2615c8060818f8ab5f876bc1
-  data.tar.gz: e1800b2c149556be65affbf35321abd7f50282efe54dcb33af379cc24ea493c7c72663eff1b7e4379eef4ac8eb852b5a36f8532f844ebc7eadb6784b495e62ff
+  metadata.gz: 2a047c9a0d916de3bd28cd8bf41cbb91d7603eb562d68a5e5a9c099a1418d25a9756787ed735fe0badd8de712418366567b3af65cb86cf1c6b3c3898390c8415
+  data.tar.gz: 6ff07ad81f425dc6c2804e4dc38c36ee4122ff89b449fa9c98065213ca2730f0cf3f4e3c014f465482835c54d6d3f52453a0e0b463a694517b6c0c039e0bb3e9

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [3.6.3](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.2...v3.6.3) (2023-09-13)
+
+
+### Bug Fixes
+
+* passing correct type when generating column migrations ([e345d56](https://github.com/craigulliott/dynamic_migrations/commit/e345d56cdb11bb965c71c251d82ce2087b416f47))
+
 ## [3.6.2](https://github.com/craigulliott/dynamic_migrations/compare/v3.6.1...v3.6.2) (2023-09-13)
 
 

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

@@ -32,9 +32,17 @@ module DynamicMigrations
 
           options_syntax = options.map { |k, v| "#{k}: #{v}" }.join(", ")
 
-          data_type = column.data_type
+          data_type = column.data_type.to_s
+          # if it's an array, then we need to remove the [] from the end
           if column.array?
-            data_type = "\"#{data_type}\""
+            data_type = data_type.sub(/\[\]\z/, "")
+          end
+          # if its a custom type (has special characters) then we need to quote it
+          # otherwise, present it as a symbol
+          data_type = if data_type.match?(/\A\w+\z/)
+            ":#{data_type}"
+          else
+            "\"#{data_type}\""
           end
 
           add_fragment schema: column.table.schema,
@@ -43,7 +51,7 @@ module DynamicMigrations
             object: column,
             code_comment: code_comment,
             migration: <<~RUBY
-              add_column :#{column.table.name}, :#{column.name}, :#{data_type}, #{options_syntax}
+              add_column :#{column.table.name}, :#{column.name}, #{data_type}, #{options_syntax}
             RUBY
         end
 

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

@@ -88,6 +88,10 @@ module DynamicMigrations
             options = {}
             options[:null] = column.null
 
+            if column.array?
+              options[:array] = true
+            end
+
             unless column.default.nil?
               options[:default] = "\"#{column.default}\""
             end
@@ -103,7 +107,20 @@ module DynamicMigrations
 
             options_syntax = options.map { |k, v| "#{k}: #{v}" }.join(", ")
 
-            lines << "t.#{column.data_type} :#{column.name}, #{options_syntax}"
+            data_type = column.data_type.to_s
+            # if it's an array, then we need to remove the [] from the end
+            if column.array?
+              data_type = data_type.sub(/\[\]\z/, "")
+            end
+            # if its a custom type (has special characters) then we need to quote it
+            # otherwise, present it as a symbol
+            data_type = if data_type.match?(/\A\w+\z/)
+              ":#{data_type}"
+            else
+              "\"#{data_type}\""
+            end
+
+            lines << "t.column #{data_type}, :#{column.name}, #{options_syntax}"
           end
 
           if timestamps.any?

data/lib/dynamic_migrations/version.rb

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

metadata

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