scriptoriumadmin/zotero
1
0
Fork 0
mirror of https://github.com/zotero/zotero.git synced 2026-08-28 05:25:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Pass save options through in Zotero.Collection::removeItems()

Browse source 
addItems() passes its options to Zotero.Item::save(), but removeItems()
dropped everything but skipEditCheck, so callers couldn't batch the
resulting notifications.
This commit is contained in:
Dan Stillman 2026-08-23 12:08:58 -04:00
parent ff93139cce
commit b6efbe880e
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
chrome/content/zotero/xpcom/data/collection.js
View file

@ -456,6 +456,8 @@ Zotero.Collection.prototype.removeItem = function (itemID, options = {}) {
* Does not require a separate save()
*/
Zotero.Collection.prototype.removeItems = async function (itemIDs, options = {}) {
options.skipDateModifiedUpdate = true;
if (!itemIDs || !itemIDs.length) {
return;
}
@ -473,10 +475,7 @@ Zotero.Collection.prototype.removeItems = async function (itemIDs, options = {})
let item = await this.ChildObjects.getAsync(itemID);
item.removeFromCollection(this.id);
await item.save({
skipDateModifiedUpdate: true,
skipEditCheck: options.skipEditCheck
})
await item.save(options);
}
};