Explain about sqlite dropping the constraint and why we're adding it back manually.

1 files changed, 2 insertions, 0 deletions

diff --git a/mediagoblin/db/migrations.py b/mediagoblin/db/migrations.py
index fef353af..98e8b139 100644
--- a/mediagoblin/db/migrations.py
+++ b/mediagoblin/db/migrations.py
@@ -371,6 +371,8 @@ def pw_hash_nullable(db):
 
     user_table.c.pw_hash.alter(nullable=True)
 
+    # sqlite+sqlalchemy seems to drop this constraint during the
+    # migration, so we add it back here for now a bit manually.
     if db.bind.url.drivername == 'sqlite':
         constraint = UniqueConstraint('username', table=user_table)
         constraint.create()