After update 6002 the schema module reports a mismatch. The attached patch fixed this for me.
I'm running postgresql 8.4

Comments

wedge’s picture

StatusFileSize
new601 bytes

Oops, missed that db_change_field() removes the index. New path attached.

Island Usurper’s picture

Status: Needs review » Needs work

Since it's really only the index that is not consistent, I think this update function can just add it instead of doing a full table alter. There doesn't seem to be a good way to avoid duplicate index errors, though.

However, your patch still doesn't actually add the index. You have a closing parenthesis in the wrong place.

Island Usurper’s picture

Issue tags: +Release blocker
Island Usurper’s picture

Status: Needs work » Needs review
StatusFileSize
new1.3 KB

It's got some ugly error suppression, but that's because there isn't an easy way to tell if the index already exists or not. I suppose the @ operator would be cleaner, but I don't want to suppress *all* possible errors.

Island Usurper’s picture

Status: Needs review » Fixed
StatusFileSize
new1.57 KB

Of course, Postgres has a different error message to be suppressed. Two of them, oddly enough. Now the patch works on MySQL and Postgres, so it's been committed.

Status: Fixed » Closed (fixed)
Issue tags: -Release blocker

Automatically closed -- issue fixed for 2 weeks with no activity.