From d5e6370bbe4b0abbbe6d981b9da6757c8ff899e8 Mon Sep 17 00:00:00 2001
From: Andrew Dickinson <dickinsa@umich.edu>
Date: Sat, 25 Apr 2020 20:53:10 -0400
Subject: [PATCH] Fix "Skipping unsupported ALTER" warning on database
 migration

---
 .../versions/2dcb0c0048dc_autologger.py       | 38 ++++++++++++++-----
 1 file changed, 28 insertions(+), 10 deletions(-)

diff --git a/ihatemoney/migrations/versions/2dcb0c0048dc_autologger.py b/ihatemoney/migrations/versions/2dcb0c0048dc_autologger.py
index 08008355..b0b4f44c 100644
--- a/ihatemoney/migrations/versions/2dcb0c0048dc_autologger.py
+++ b/ihatemoney/migrations/versions/2dcb0c0048dc_autologger.py
@@ -165,21 +165,39 @@ def upgrade():
         sa.Column("remote_addr", sa.String(length=50), nullable=True),
         sa.PrimaryKeyConstraint("id"),
     )
-    op.add_column(
-        "project",
-        sa.Column(
-            "logging_preference",
-            sa.Enum("DISABLED", "ENABLED", "RECORD_IP", name="loggingmode"),
-            server_default="ENABLED",
-            nullable=False,
-        ),
-    )
+    bind = op.get_bind()
+    if bind.engine.name == "sqlite":
+        with op.batch_alter_table("project", recreate="always") as batch_op:
+            batch_op.add_column(
+                sa.Column(
+                    "logging_preference",
+                    sa.Enum("DISABLED", "ENABLED", "RECORD_IP", name="loggingmode"),
+                    server_default="ENABLED",
+                    nullable=False,
+                ),
+            )
+    else:
+        op.add_column(
+            "project",
+            sa.Column(
+                "logging_preference",
+                sa.Enum("DISABLED", "ENABLED", "RECORD_IP", name="loggingmode"),
+                server_default="ENABLED",
+                nullable=False,
+            ),
+        )
     # ### end Alembic commands ###
 
 
 def downgrade():
     # ### commands auto generated by Alembic - please adjust! ###
-    op.drop_column("project", "logging_preference")
+
+    bind = op.get_bind()
+    if bind.engine.name == "sqlite":
+        with op.batch_alter_table("project", recreate="always") as batch_op:
+            batch_op.drop_column("logging_preference")
+    else:
+        op.drop_column("project", "logging_preference")
     op.drop_table("transaction")
     op.drop_index(
         op.f("ix_project_version_transaction_id"), table_name="project_version"