-
Notifications
You must be signed in to change notification settings - Fork 442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better Datetime handling #317
Conversation
The duplicate |
This makes sense for 1.2 (next release), since it's a better fix than the current hackery. ;) |
Most likely, I was just starting to get frustrated by the exception handling, as namespaces are a little new to me. I was really just trying to fix the issue that MySQL doesn't handle timezones at all. Realistically, it's still an issue, as if 2 different php scripts hit the same table, with different default TZ, they will get different times, but that's a little outside the scope of php-activerecord |
@VendanAndrews Agreed. We can't ensure data integrity across those kind of boundaries when MySQL doesn't help us any. Use PostgreSQL. O_O |
IMHO, yes, #311 is superseded by this. |
Closing in favor of #440 |
This is a series of fixes, that includes handling mysql datetimes better and improving datetime round-tripping. Also included is code to allow columns to cast based on whether value is coming from db or from user code, so that users can use the same format for any adapter(wasn't an issue, but may be with some DB's)