#446 Make timestamped migrations optional - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#446 ✓resolved

Make timestamped migrations optional

Reported by ronin-16348 (at lighthouseapp) | June 18th, 2008 @ 05:37 PM

Have the next_migration_string check for a config option to determine migration number formatting:

config.active_record.timestamped_migrations = true

I think the default should be false due to the "legacy" applications.

Comments and changes to this ticket

You flagged this item as spam.
ronin-16348 (at lighthouseapp) June 18th, 2008 @ 05:38 PM
- no changes were found...
You flagged this item as spam.
ronin-16348 (at lighthouseapp) June 18th, 2008 @ 05:40 PM
I didn't see the attachment, just noticed where references it on the screen. I removed the duplicate attachment.
You flagged this item as spam.
ronin-16348 (at lighthouseapp) June 18th, 2008 @ 05:50 PM
Forgot to add:
cattr_accessor :timestamped_migrations , :instance_writer => false
@@timestamped_migrations = false
This is required (as you guys know), in activerecord/lib/active_record/base.rb
Apologies, it's my first attempt at submitting a patch.
You flagged this item as spam.
ronin-16348 (at lighthouseapp) June 18th, 2008 @ 05:58 PM
new diff file attached with active_record/base included.
- optional_timestamped_migrations_2.diff 2.6 KB
Pratik July 9th, 2008 @ 01:24 PM
- State changed from “new” to “incomplete”
- Assigned user set to “Pratik”
- Tag set to “patch”
The default needs to be true.
Also, it'll be nice if you could send an email to rails core list to discuss this change.
Thanks.
You flagged this item as spam.
ronin-16348 (at lighthouseapp) July 11th, 2008 @ 02:32 PM
I just sent an email to the core list. Is this why it was marked as incomplete?
thanks,
andy
nikz July 17th, 2008 @ 02:42 AM
- no changes were found...
- optional_timestamped_migrations_with_flipped_default.diff 4.6 KB
You flagged this item as spam.
nikz July 17th, 2008 @ 02:42 AM
Uploaded a diff that flips the default and adds docs, so us luddites can have our numbers back ;)
Now you kids get off my dang lawn!
Repository July 17th, 2008 @ 02:57 AM
- State changed from “incomplete” to “resolved”
(from [bbab6391366f59189e84d2b8de2a63bea91a9851]) Set config.active_record.timestamped_migrations = false to have migrations with numeric prefix instead of UTC timestamp. [#446 state:resolved]
Signed-off-by: Pratik Naik
http://github.com/rails/rails/co...
You flagged this item as spam.
ronin-16348 (at lighthouseapp) July 17th, 2008 @ 03:01 AM
Thanks nikz!
Repository July 30th, 2008 @ 05:44 PM
(from [af3f2aad7ecb31a6e1d09f06775cb33b4ad64d91]) Set config.active_record.timestamped_migrations = false to have migrations with numeric prefix instead of UTC timestamp. [#446 state:resolved]
Signed-off-by: Pratik Naik
http://github.com/rails/rails/co...
Jeff Kreeftmeijer November 8th, 2010 @ 08:49 AM
- Tag cleared.
- Importance changed from “” to “Low”
Automatic cleanup of spam.