From 7c9b1f785c0583f5c883d1db2d4d87c3e8540e6c Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Thu, 8 Jun 2006 20:02:26 +0000 Subject: [PATCH] Only send itemView 'remove' notifictions for items if they're actually deleted from the DB, not if they're just removed from a collection --- chrome/chromeFiles/content/scholar/xpcom/data_access.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/chrome/chromeFiles/content/scholar/xpcom/data_access.js b/chrome/chromeFiles/content/scholar/xpcom/data_access.js index 762c9af812..ce88200261 100644 --- a/chrome/chromeFiles/content/scholar/xpcom/data_access.js +++ b/chrome/chromeFiles/content/scholar/xpcom/data_access.js @@ -1279,8 +1279,8 @@ Scholar.Collection.prototype.erase = function(deleteItems){ if (deleteItems){ // Delete items from DB Scholar.Items.get(descendents[i]['id']).erase(); + items.push(descendents[i]['id']); } - items.push(descendents[i]['id']); } } @@ -1298,7 +1298,9 @@ Scholar.Collection.prototype.erase = function(deleteItems){ Scholar.Collections.unload(collections); Scholar.Notifier.trigger('remove', 'collection', collections); - Scholar.Notifier.trigger('remove', 'item', items); + if (items.length){ + Scholar.Notifier.trigger('remove', 'item', items); + } }