online_migrations 0.9.0 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f7396712072756a2c7de52a7057f0c46da8ce26bf59270f0f2dea42bf43d0b3
-  data.tar.gz: a3a26397aa2a279e996e8af2e7c698ebabc73ee38bad89b84e98cbddb609feb4
+  metadata.gz: 579a0db844c82c2c9153a1a3f44bfcffe929011112b132afe73b1dfdf68f4583
+  data.tar.gz: b88f26034f0c1d044607ef191ca3feae71e3c6093cb05bd9a277790447da01af
 SHA512:
-  metadata.gz: d897c787f1b4e23436ecc362dd49a97eba529e7f0613fe71d38b4fde1cd740a40e0fa6ba2ef59d32f0ab6c1050ebc98728a1830d22679f98671305e1d82d6eb3
-  data.tar.gz: 10c2d6cc695042052c994111f71df5e19d17cf2b81a72b4ca2bce835ecced1a91dad0bfe5d2091079661c1d97dbbf066e641b80ceca196be7dd5c0e859f8ea09
+  metadata.gz: 229a2a31c44a358425a684ea8d1771d98ecee6412c9b4480e30e2cd68e53c0c9e77039959fa8f379ad0a83fb9f592d133d29fe43e2c79ae4dbf674cd9c210598
+  data.tar.gz: 9255d5ad7b7d350021cc553b5d91bec852cfbecc0b9c7d6296606beffae266de72096cdf5a4d12dd88d921fb0af60a9e870d08a676a90fad552c5bd2d36096f6

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 ## master (unreleased)
 
+## 0.9.2 (2023-11-02)
+
+- Fix checking which expression indexes to copy when changing column type
+
+## 0.9.1 (2023-10-30)
+
+- Fix copying expression indexes when changing column type
+
 ## 0.9.0 (2023-10-27)
 
 - Add ability to use custom raw sql for `backfill_column_for_type_change`'s `type_cast_function`

data/lib/online_migrations/change_column_type_helpers.rb

@@ -405,9 +405,15 @@ module OnlineMigrations
         to_column    = to_column.to_s
 
         __indexes_for(table_name, from_column).each do |index|
-          new_columns = index.columns.map do |column|
-            column == from_column ? to_column : column
-          end
+          new_columns =
+            # Expression index.
+            if index.columns.is_a?(String)
+              index.columns.gsub(/\b#{from_column}\b/, to_column)
+            else
+              index.columns.map do |column|
+                column == from_column ? to_column : column
+              end
+            end
 
           # This is necessary as we can't properly rename indexes such as "taggings_idx".
           if !index.name.include?(from_column)
@@ -444,7 +450,13 @@ module OnlineMigrations
       def __indexes_for(table_name, column_name)
         column_name = column_name.to_s
 
-        indexes(table_name).select { |index| index.columns.include?(column_name) }
+        indexes(table_name).select do |index|
+          if index.columns.is_a?(String)
+            index.columns =~ /\b#{column_name}\b/
+          else
+            index.columns.include?(column_name)
+          end
+        end
       end
 
       # While its rare for a column to have multiple foreign keys, PostgreSQL supports this.

data/lib/online_migrations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OnlineMigrations
-  VERSION = "0.9.0"
+  VERSION = "0.9.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: online_migrations
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.2
 platform: ruby
 authors:
 - fatkodima
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-26 00:00:00.000000000 Z
+date: 2023-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord