From bc18f42f81e92968b14d87e6eb9efea093cd5ae5 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Tue, 27 Jun 2006 14:14:40 +0000
Subject: [PATCH] Fix issue with null values passed natively to DB query
 statements as bound statements

---
 chrome/chromeFiles/content/scholar/xpcom/db.js | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/chrome/chromeFiles/content/scholar/xpcom/db.js b/chrome/chromeFiles/content/scholar/xpcom/db.js
index b47b630b4..8f7271455 100644
--- a/chrome/chromeFiles/content/scholar/xpcom/db.js
+++ b/chrome/chromeFiles/content/scholar/xpcom/db.js
@@ -171,17 +171,17 @@ Scholar.DB = new function(){
 		if (statement && params){
 			for (var i=0; i<params.length; i++){
 				// Integer
-				if (typeof params[i]['int'] != 'undefined'){
+				if (params[i]!==null && typeof params[i]['int'] != 'undefined'){
 					var type = 'int';
 					var value = params[i]['int'];
 				}
 				// String
-				else if (typeof params[i]['string'] != 'undefined'){
+				else if (params[i]!==null && typeof params[i]['string'] != 'undefined'){
 					var type = 'string';
 					var value = params[i]['string'];
 				}
 				// Null
-				else if (typeof params[i]['null'] != 'undefined'){
+				else if (params[i]!==null && typeof params[i]['null'] != 'undefined'){
 					var type = 'null';
 				}
 				// Automatic (trust the JS type)
@@ -195,8 +195,7 @@ Scholar.DB = new function(){
 							break;
 						// Object
 						default:
-							// Null value will show as object
-							if (!params[i]){
+							if (params[i]===null){
 								var type = 'null';
 							}
 							else {