The patch is in the "fix-typo" branch:

​https://gitorious.org/mediagoblin/berkerpeksag-mediagoblin/commits/80719e85eccb51ca4bb1559588f2aef257a4547f

A recent migration added a table for the CollectionItem model but didn't add the constraint of:

    __table_args__ = (
        UniqueConstraint('collection', 'media_entry'),
        {})

Since then we've moved to a new style of "creating models in migrations" which involves actually putting a copy of the original model inside the migrations file to make it easier to create the right database. However, anyone running from git master will currently pick up the "mistaken" commit.

There basically seem to be one of two possible solutions:

• One, try to create a migration that's "smart"-ish. We can't check for constraints... sqlalchemy doesn't have support for that... but we could maybe try adding the constraint and just catch the exception if it isn't allowed?
• That might not work though (and it does feel a bit "icky"). If it doesn't, we should simply correct the old migration and also create scripts that will add the constraint for people who may have picked up the bad migration from git master.

Please review and pull my branch fix_docs from my repo at ​git://gitorious.org/mediagoblin/spaetz-mediagoblin.git. It fixes a critical crasher.