diff --git a/chrome/content/zotero/xpcom/itemTreeView.js b/chrome/content/zotero/xpcom/itemTreeView.js index 23fd83ddc..871610115 100644 --- a/chrome/content/zotero/xpcom/itemTreeView.js +++ b/chrome/content/zotero/xpcom/itemTreeView.js @@ -207,7 +207,11 @@ Zotero.ItemTreeView.prototype.notify = function(action, type, ids) else { var item = Zotero.Items.get(ids[i]); - + if (!item) { + // DEBUG: this shouldn't really happen but could if a + // modify comes in after a delete + continue; + } if(item.isRegularItem() || !item.getSource()) { //most likely, the note or attachment's parent was removed. diff --git a/chrome/content/zotero/xpcom/notifier.js b/chrome/content/zotero/xpcom/notifier.js index 0b39cf5b9..ddee8363e 100644 --- a/chrome/content/zotero/xpcom/notifier.js +++ b/chrome/content/zotero/xpcom/notifier.js @@ -267,6 +267,9 @@ Zotero.Notifier = new function(){ } + /* + * These should probably no longer be used now that we have event queuing + */ function disable(){ Zotero.debug('Disabling Notifier notifications'); _disabled = true;