Thread skipNotify through addItem

chrome/content/zotero/xpcom/data/collection.js

@@ -358,8 +358,8 @@ Zotero.Collection.prototype._finalizeSave = Zotero.Promise.coroutine(function* (
  * @param {Number} itemID
  * @return {Promise}
  */
-Zotero.Collection.prototype.addItem = function (itemID) {
+Zotero.Collection.prototype.addItem = function (itemID, options) {
-	return this.addItems([itemID]);
+	return this.addItems([itemID], options);
 }
 
 
@@ -371,7 +371,9 @@ Zotero.Collection.prototype.addItem = function (itemID) {
  * @param {Number[]} itemIDs
  * @return {Promise}
  */
-Zotero.Collection.prototype.addItems = Zotero.Promise.coroutine(function* (itemIDs) {
+Zotero.Collection.prototype.addItems = Zotero.Promise.coroutine(function* (itemIDs, options = {}) {
+	options.skipDateModifiedUpdate = true;
+
 	if (!itemIDs || !itemIDs.length) {
 		return;
 	}
@@ -389,9 +391,7 @@ Zotero.Collection.prototype.addItems = Zotero.Promise.coroutine(function* (itemI
 		
 		let item = this.ChildObjects.get(itemID);
 		item.addToCollection(this.id);
-		yield item.save({
+		yield item.save(options);
-			skipDateModifiedUpdate: true
-		});
 	}
 	
 	yield this.loadDataType('childItems');

chrome/content/zotero/xpcom/data/item.js

@@ -1396,9 +1396,11 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 			
 			let newParentItemNotifierData = {};
 			//newParentItemNotifierData[newParentItem.id] = {};
+			if (!env.options.skipNotifier) {
 				Zotero.Notifier.queue(
 					'modify', 'item', parentItemID, newParentItemNotifierData, env.options.notifierQueue
 				);
+			}
 			
 			switch (Zotero.ItemTypes.getName(itemTypeID)) {
 				case 'note':
@@ -1422,6 +1424,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 				
 				let newParentItemNotifierData = {};
 				//newParentItemNotifierData[newParentItem.id] = {};
+				if (!env.options.skipNotifier) {
 					Zotero.Notifier.queue(
 						'modify',
 						'item',
@@ -1430,6 +1433,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 						env.options.notifierQueue
 					);
 				}
+			}
 			
 			let oldParentKey = this._previousData.parentKey;
 			let oldParentItemID;
@@ -1438,6 +1442,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 				if (oldParentItemID) {
 					let oldParentItemNotifierData = {};
 					//oldParentItemNotifierData[oldParentItemID] = {};
+					if (!env.options.skipNotifier) {
 						Zotero.Notifier.queue(
 							'modify',
 							'item',
@@ -1446,6 +1451,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 							env.options.notifierQueue
 						);
 					}
+				}
 				else {
 					Zotero.debug("Old source item " + oldParentKey
 						+ " didn't exist in Zotero.Item.save()", 2);
@@ -1465,6 +1471,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 						parentItem.addToCollection(changedCollections[i]);
 						this.removeFromCollection(changedCollections[i]);
 						
+						if (!env.options.skipNotifier) {
 							Zotero.Notifier.queue(
 								'remove',
 								'collection-item',
@@ -1473,6 +1480,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 								env.options.notifierQueue
 							);
 						}
+					}
 					let parentOptions = {
 						skipDateModifiedUpdate: true
 					};
@@ -1536,10 +1544,12 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 		yield Zotero.DB.queryAsync(sql, itemID);
 		
 		// Refresh trash
+		if (!env.options.skipNotifier) {
 			Zotero.Notifier.queue('refresh', 'trash', this.libraryID, {}, env.options.notifierQueue);
 			if (this._deleted) {
 				Zotero.Notifier.queue('trash', 'item', this.id, {}, env.options.notifierQueue);
 			}
+		}
 		
 		if (parentItemID) {
 			reloadParentChildItems[parentItemID] = true;
@@ -1658,10 +1668,12 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 				tag: tag.tag,
 				type: tagType
 			};
+			if (!env.options.skipNotifier) {
 				Zotero.Notifier.queue(
 					'add', 'item-tag', this.id + '-' + tagID, notifierData, env.options.notifierQueue
 				);
 			}
+		}
 		
 		if (toRemove.length) {
 			for (let i=0; i<toRemove.length; i++) {
@@ -1674,10 +1686,13 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 					libraryID: this.libraryID,
 					tag: tag.tag
 				};
+
+				if (!env.options.skipNotifier) {
 					Zotero.Notifier.queue(
 						'remove', 'item-tag', this.id + '-' + tagID, notifierData, env.options.notifierQueue
 					);
 				}
+			}
 			Zotero.Prefs.set('purge.tags', true);
 		}
 	}
@@ -1709,6 +1724,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 					+ "(collectionID, itemID, orderIndex) VALUES (?, ?, ?)";
 				yield Zotero.DB.queryAsync(sql, [collectionID, this.id, orderIndex]);
 				
+				if (!env.options.skipNotifier) {
 					Zotero.Notifier.queue(
 						'add',
 						'collection-item',
@@ -1717,6 +1733,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 						env.options.notifierQueue
 					);
 				}
+			}
 			
 			// Add this item to any loaded collections' cached item lists after commit
 			Zotero.DB.addCurrentCallback("commit", function () {
@@ -1735,6 +1752,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 			for (let i=0; i<toRemove.length; i++) {
 				let collectionID = toRemove[i];
 				
+				if (!env.options.skipNotifier) {
 					Zotero.Notifier.queue(
 						'remove',
 						'collection-item',
@@ -1743,6 +1761,7 @@ Zotero.Item.prototype._saveData = Zotero.Promise.coroutine(function* (env) {
 						env.options.notifierQueue
 					);
 				}
+			}
 			
 			// Remove this item from any loaded collections' cached item lists after commit
 			Zotero.DB.addCurrentCallback("commit", function () {