Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #7034 (closed defect: incomplete)

Opened 1 year ago

Last modified 6 months ago

[PATCH] Have ActiveRecord use :db date format rather than :default

Reported by: jeremymcanally Assigned to: core
Priority: normal Milestone: 1.2.4
Component: ActiveRecord Version: edge
Severity: normal Keywords: verified
Cc:

Description

This patch is in response to the unadulterated rage I experienced while working with something that needed this along with ticket #2965.

It forces ActiveRecord to use the :db date format when saving timestamps and dates to the database rather than the default like it used to; I'm guessing this is a fairly common situation since I'm not the only one who was hit by it.

Attachments

use_db_format_to_quote_dates.diff (1.0 kB) - added by jeremymcanally on 01/14/07 16:33:45.

Change History

01/14/07 16:33:45 changed by jeremymcanally

  • attachment use_db_format_to_quote_dates.diff added.

01/14/07 17:26:51 changed by jeremymcanally

  • version changed from 1.1.6 to 1.2.0rc2.

01/14/07 17:28:42 changed by jeremymcanally

  • version changed from 1.2.0rc2 to edge.

01/14/07 18:01:03 changed by dcmanges

patched worked for me testing w/ mysql

04/20/07 23:58:44 changed by josh

  • keywords set to verified.

+1 verified

07/21/07 10:14:52 changed by mpalmer

  • status changed from new to closed.
  • resolution set to incomplete.

The code around this part of the system has changed significantly in edge, and without any tests being provided it's hard to see whether the system now does what you want here.

I'm closing this ticket as incomplete, as it's lacking tests and (more importantly) the patch doesn't apply to the current edge.

If edge still doesn't do what you think it should, please re-open this ticket with a new patch that has relevant test cases and applies cleanly.

01/01/08 20:34:41 changed by michaelboutros

It seems that in 2.0 (or maybe earlier), this functionality has been added to the source (albeit in a different way), so this patch is no longer needed.