From c561ca129efd5bdc5641cdd05282a17bddf8b114 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Fri, 2 Jun 2006 21:11:14 +0000 Subject: [PATCH] Reverting r111 change to bound parameters for Item.erase() (minus the exception fix) -- bound parameters/multiple statements/no can do Item deleting should work properly now --- .../chromeFiles/content/scholar/xpcom/data_access.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/chrome/chromeFiles/content/scholar/xpcom/data_access.js b/chrome/chromeFiles/content/scholar/xpcom/data_access.js index 160866c9bc..99d625ae90 100644 --- a/chrome/chromeFiles/content/scholar/xpcom/data_access.js +++ b/chrome/chromeFiles/content/scholar/xpcom/data_access.js @@ -662,12 +662,12 @@ Scholar.Item.prototype.erase = function(){ Scholar.Collections.get(parentCollectionIDs[i]).removeItem(this.getID()); } - sql = "DELETE FROM itemCreators WHERE itemID=?1;\n"; - sql += "DELETE FROM itemKeywords WHERE itemID=?1;\n"; - sql += "DELETE FROM itemData WHERE itemID=?1;\n"; - sql += "DELETE FROM items WHERE itemID=?1;"; + sql = 'DELETE FROM itemCreators WHERE itemID=' + this.getID() + ";\n"; + sql += 'DELETE FROM itemKeywords WHERE itemID=' + this.getID() + ";\n"; + sql += 'DELETE FROM itemData WHERE itemID=' + this.getID() + ";\n"; + sql += 'DELETE FROM items WHERE itemID=' + this.getID() + ";\n"; - Scholar.DB.query(sql, [{'int':this.getID()}]); + Scholar.DB.query(sql); Scholar.Creators.purge(); try {