From b84181766df6edc436f4bb6e9e8a16d3107a3da2 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Tue, 12 Sep 2006 06:53:48 +0000 Subject: [PATCH] Fix bug in db.js::statementQuery() that prevented binding a single string parameter without putting it in an array Also fixed array detection in flattenArguments() to handle a null value --- chrome/chromeFiles/content/scholar/xpcom/db.js | 2 +- chrome/chromeFiles/content/scholar/xpcom/scholar.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/chrome/chromeFiles/content/scholar/xpcom/db.js b/chrome/chromeFiles/content/scholar/xpcom/db.js index 6741b8733f..6a92139916 100644 --- a/chrome/chromeFiles/content/scholar/xpcom/db.js +++ b/chrome/chromeFiles/content/scholar/xpcom/db.js @@ -172,7 +172,7 @@ Scholar.DB = new function(){ if (statement && params){ // If single parameter, wrap in an array - if (!params.length){ + if (typeof params != 'object' || params===null){ params = [params]; } diff --git a/chrome/chromeFiles/content/scholar/xpcom/scholar.js b/chrome/chromeFiles/content/scholar/xpcom/scholar.js index 1aad4741b9..25f9348b4b 100644 --- a/chrome/chromeFiles/content/scholar/xpcom/scholar.js +++ b/chrome/chromeFiles/content/scholar/xpcom/scholar.js @@ -306,7 +306,7 @@ var Scholar = new function(){ */ function flattenArguments(args){ // Put passed scalar values into an array - if (typeof args!='object'){ + if (typeof args!='object' || args===null){ args = [args]; }