[PATCH] Detect duplicate migration names

Good call. You could even just re-use IllegalMigrationNameError here, since it's essentially illegal to reuse a migration name. :)

I don't think you have resolved it ;-)

I have created a new patch in Git:

http://github.com/aslakhellesoy/...

The situation where this can happen is rare, but possible:

ruby script\generate migration chunky
# commit it

# days or weeks later
ruby script\generate migration bacon
# Hmm, this should be called chunky (overlooking there is already a chunky
# migration from before). Rename the migration file to (same_timestamp)_chunky.rb 
# and the migration class to Chunky. There are now two Chunky migrations
# with different timestamps.
rake db:migrate
# This should have thrown an error, but it doesn't.

Steve taught me off line how to make a fix on a branch - I'll do that next time!

With auto-incrementing ids, collision was guaranteed. With the timestamp ids, you have to be creating the same migration at the very same second for a collision to happen. That's unlikely enough that we don't need more software to deal with, imo.

This is not about ids or timestamps at all. It's about two migrations with identical class names. The class in the latest file will overwrite the class in the former, and the former migration will never run.

This is possible if a migration is renamed, and Rails will never warn you about it. (I realise migrations should never be renamed, but it happens sometimes with new uncommitted migrations).

It's likely to happen in projects with many migrations if a developer renames an uncommitted migration after it's generated.

I'm not going to persist more, just point to the fact that the patch is only 5 lines (plus tests).

Here's Aslak's patch -- I had it handy...

In my second comment there is a link to a changeset in Git, which you can just merge into the master branch. Here is the link again:

http://github.com/aslakhellesoy/...

One of the benefits of Git is that you don't need patch files in an issue tracker anymore. Just pull patches from other people's clones.

A

Pulling one-off patches is a huge pain in the ass. It's fine for long-running, expansive branches, but for simple fixes it's far too much work. So we continue to use patch files in the issue trackers for anything but stuff like docrails and rack on rails.

Yeah, I can see how it might be a PITA for small patches. I'll use attachments for future small patches.

You may want to add a note about this in the sidebar here at Lighthouse. Something like "Small patches as attachments in Lighthouse - big contributions in Git clones".

Thanks for taking the time to review this patch - however small it is.

Automatic cleanup of spam.