Skip Menu |

This queue is for tickets about the DBIx-Class-TimeStamp CPAN distribution.

Report information
The Basics
Id: 111583
Status: new
Priority: 0/
Queue: DBIx-Class-TimeStamp

People
Owner: Nobody in particular
Requestors: mark [...] aufflick.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: set_on_create should only affect datetime columns
Since set_on_create has a generic meaning, it shouldn't blindly apply to all columns (I tried to make another plugin that also used that key to be consistent, with humorous results). Either (a) the key should be datetime specific, or preferably (b) we keep the key the same, but it only applies if the data_type matches/(?:timestamp|date)/ or if the user specifies via a specific key "set_on_create_datetime" to force the issue. This is roughly the approach that InflateColumn::DateTime uses. I'm happy to make and test a patch, but I wanted to see if you were happy with the approach before progressing.