Skip to content
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

"Update" Notes causes database insert error #3748

Closed
crossan007 opened this issue Dec 16, 2017 · 0 comments
Closed

"Update" Notes causes database insert error #3748

crossan007 opened this issue Dec 16, 2017 · 0 comments

Comments

@crossan007
Copy link
Contributor

crossan007 commented Dec 16, 2017

Creating a new person via ORM does not correctly handle timeline note date fields (updated and created)

This occurs be cause the person ORM defines a null DateTime for the note's nte_DateEntered when the person action is "update".

Null Dates are prohibited in modern versions of MySQL: dreamwidth/dreamwidth#1822

This surfaced in @phili67's pr #3659

fail orm

@crossan007 crossan007 changed the title Update" "Update" Notes causes database insert error Dec 16, 2017
@crossan007 crossan007 added this to the On-Deck milestone Dec 16, 2017
@crossan007 crossan007 self-assigned this Dec 16, 2017
crossan007 added a commit that referenced this issue Dec 16, 2017
@ghost ghost added In Review and removed bug labels Dec 16, 2017
@ghost ghost removed the In Review label Dec 16, 2017
DawoudIO pushed a commit that referenced this issue Dec 16, 2017
@crossan007 crossan007 modified the milestones: On-Deck, 2.9.3 Dec 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant