Avoids JSON parsing values when they are already objects (and not string buffers) (dresende#168)

dresende · dresende · commit 47e5084d584e · 2013-05-28T16:21:35.000+01:00
diff --git a/lib/Drivers/DML/mysql.js b/lib/Drivers/DML/mysql.js
@@ -225,6 +225,9 @@ Driver.prototype.valueToProperty = function (value, property) {
 		case "boolean":
 			return !!value;
 		case "object":
+			if (typeof value == "object" && !Buffer.isBuffer(value)) {
+				return value;
+			}
 			try {
 				return JSON.parse(value);
 			} catch (e) {
diff --git a/lib/Drivers/DML/postgres.js b/lib/Drivers/DML/postgres.js
@@ -250,6 +250,9 @@ Driver.prototype.clear = function (table, cb) {
 Driver.prototype.valueToProperty = function (value, property) {
 	switch (property.type) {
 		case "object":
+			if (typeof value == "object" && !Buffer.isBuffer(value)) {
+				return value;
+			}
 			try {
 				return JSON.parse(value);
 			} catch (e) {
diff --git a/lib/Drivers/DML/sqlite.js b/lib/Drivers/DML/sqlite.js
@@ -198,6 +198,9 @@ Driver.prototype.valueToProperty = function (value, property) {
 		case "boolean":
 			return !!value;
 		case "object":
+			if (typeof value == "object" && !Buffer.isBuffer(value)) {
+				return value;
+			}
 			try {
 				return JSON.parse(value);
 			} catch (e) {
