ISO dates for SQLite

Simplified the way that dates are handled for SQLite.

Author: apoco

lib/Drivers/DML/sqlite.js

@@ -260,21 +260,20 @@ Driver.prototype.valueToProperty = function (value, property) {
 			}
 			break;
 		case "date":
-			if (typeof value == 'string') {
-				if (value.indexOf('Z', value.length - 1) === -1) {
-					value = new Date(value + 'Z');
-				} else {
-					value = new Date(value);
-				}
-
-				if (this.config.timezone && this.config.timezone != 'local') {
-					var tz = convertTimezone(this.config.timezone);
+			if (typeof value === 'string') {
+				var hasTimezone = value.indexOf('Z') !== -1;
+				value = new Date(value);
 
+				if (!hasTimezone) {
 					// shift local to UTC
 					value.setTime(value.getTime() - (value.getTimezoneOffset() * 60000));
-					if (tz !== false) {
-						// shift UTC to timezone
-						value.setTime(value.getTime() - (tz * 60000));
+
+					if (this.config.timezone && this.config.timezone != 'local') {
+						var tz = convertTimezone(this.config.timezone);
+						if (tz !== false) {
+							// shift UTC to timezone
+							value.setTime(value.getTime() - (tz * 60000));
+						}
 					}
 				}
 			}
@@ -301,45 +300,8 @@ Driver.prototype.propertyToValue = function (value, property) {
 			}
 			break;
 		case "date":
-			if (this.config.query && this.config.query.strdates) {
-				if (value instanceof Date) {
-					var year = value.getUTCFullYear();
-					var month = value.getUTCMonth() + 1;
-					if (month < 10) {
-						month = '0' + month;
-					}
-					var date = value.getUTCDate();
-					if (date < 10) {
-						date = '0' + date;
-					}
-					var strdate = year + '-' + month + '-' + date;
-					if (property.time === false) {
-						value = strdate;
-						break;
-					}
-
-					var hours = value.getUTCHours();
-					if (hours < 10) {
-						hours = '0' + hours;
-					}
-					var minutes = value.getUTCMinutes();
-					if (minutes < 10) {
-						minutes = '0' + minutes;
-					}
-					var seconds = value.getUTCSeconds();
-					if (seconds < 10) {
-						seconds = '0' + seconds;
-					}
-					var millis = value.getUTCMilliseconds();
-					if (millis < 10) {
-						millis = '0' + millis;
-					}
-					if (millis < 100) {
-						millis = '0' + millis;
-					}
-					strdate += ' ' + hours + ':' + minutes + ':' + seconds + '.' + millis + '000';
-					value = strdate;
-				}
+			if (value instanceof Date) {
+				value = value.toISOString();
 			}
 			break;
 		default:

package.json

@@ -37,7 +37,7 @@
 	"analyse"     : false,
 	"dependencies": {
 		"enforce"      : "0.1.2",
-		"sql-query"    : "git+https://github.com/dresende/node-sql-query.git#v0.1.23",
+		"sql-query"    : "git+https://github.com/apoco/node-sql-query.git#sqlite-utc-dates",
 		"sql-ddl-sync" : "git+https://github.com/dresende/node-sql-ddl-sync.git#v0.3.10",
 		"hat"          : "0.0.3",
 		"lodash"       : "2.4.1"

test/integration/association-extend.js

@@ -7,6 +7,8 @@ describe("Model.extendsTo()", function() {
 	var Person = null;
 	var PersonAddress = null;
 
+	this.timeout(5000);
+
 	var setup = function () {
 		return function (done) {
 			Person = db.define("person", {

test/integration/association-hasmany-hooks.js

@@ -7,6 +7,8 @@ describe("hasMany hooks", function() {
 	var Person = null;
 	var Pet    = null;
 
+	this.timeout(5000);
+
 	var setup = function (props, opts) {
 		return function (done) {
 			db.settings.set('instance.cache', false);

test/integration/association-hasmany.js

@@ -6,11 +6,12 @@ var common   = require('../common');
 var protocol = common.protocol();
 
 describe("hasMany", function () {
-	this.timeout(4000);
 	var db     = null;
 	var Person = null;
 	var Pet    = null;
 
+	this.timeout(5000);
+
 	before(function(done) {
 		helper.connect(function (connection) {
 			db = connection;

test/integration/association-hasone-reverse.js

@@ -10,6 +10,8 @@ describe("hasOne", function () {
 	var Person = null;
 	var Pet = null;
 
+	this.timeout(5000);
+
 	var setup = function () {
 		return function (done) {
 			Person = db.define('person', {

test/integration/association-hasone.js

@@ -13,6 +13,8 @@ describe("hasOne", function() {
 	var treeId = null;
 	var stalkId = null;
 
+	this.timeout(5000);
+
 	var setup = function (opts) {
 		opts = opts || {};
 		return function (done) {

test/integration/db.js

@@ -284,7 +284,7 @@ describe("db.driver", function () {
 				db.driver.execQuery(query, args, function (err, data) {
 					should.not.exist(err);
 
-					should(JSON.stringify(data) == JSON.stringify([{ "what": "user login" }]));
+					JSON.stringify(data).should.equal(JSON.stringify([{ "what": "user login" }]));
 					done();
 				});
 			});

test/integration/model-find-chain.js

@@ -9,6 +9,8 @@ describe("Model.find() chaining", function() {
 	var Person = null;
 	var Dog = null;
 
+	this.timeout(5000);
+
 	var setup = function (extraOpts) {
 		if (!extraOpts) extraOpts = {};
 

test/integration/model-one.js

@@ -6,6 +6,8 @@ describe("Model.one()", function() {
 	var db = null;
 	var Person = null;
 
+	this.timeout(5000);
+
 	var setup = function () {
 		return function (done) {
 			Person = db.define("person", {