U4-7090 - Upgrade error with dates

Created by Shannon Deminick 11 Sep 2015, 09:07:57 Updated by Shannon Deminick 11 Sep 2015, 12:02:38

With our Insert.IntoTable.Row syntax it looks like we are not using SQL parameters for the values which is causing problems when converting DateTime instances, we need to ensure SQL parameters are used for this.

Comments

Shannon Deminick 11 Sep 2015, 09:49:31

We cannot use SQL parameters in migrations (currently) because it would be a bunch of work to not have breaking changes since we'd have to make a new parameterized migration thing since currently migrations purely return strings. In the meantime, we've got a FormatDateTime method on the SqlSyntax which should work across all servers (based on some research done today)


Priority: Major

Type: Task

State: Fixed

Assignee: Shannon Deminick

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions:

Due in version: 7.3.0

Sprint:

Story Points:

Cycle: