mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Fix context pane unnecessary update

Browse source 
Fixes #4029
This commit is contained in:
windingwind 2024-05-10 10:59:42 +08:00 • committed by Dan Stillman
parent 91c0c28b5d
commit 8b083aa426
1 changed files with 24 additions and 22 deletions
Download patch file Download diff file Expand all files Collapse all files

46
chrome/content/zotero/elements/contextPane.js
View file

@ -97,33 +97,35 @@
_handleItemUpdate(action, type, ids, extraData) {
// Update, remove or re-create item panes
for (let itemDetails of Array.from(this._itemPaneDeck.children)) {
let item = itemDetails.item;
let tabID = itemDetails.dataset.tabId;
if (!item) {
this._removeItemContext(tabID);
}
else if (item.parentID != itemDetails.parentID) {
this._removeItemContext(tabID);
this._addItemContext(tabID, item.itemID);
if (action === 'modify') {
for (let itemDetails of Array.from(this._itemPaneDeck.children)) {
let tabID = itemDetails.tabID;
let item = Zotero.Items.get(Zotero_Tabs._getTab(tabID)?.tab.data.itemID);
if ((item.parentID || itemDetails.parentID)
&& item.parentID !== itemDetails.parentID) {
this._removeItemContext(tabID);
this._addItemContext(tabID, item.itemID);
}
}
}
// Update notes lists for affected libraries
let libraryIDs = [];
for (let id of ids) {
let item = Zotero.Items.get(id);
if (item && (item.isNote() || item.isRegularItem())) {
libraryIDs.push(item.libraryID);
if (['add', 'delete', 'modify'].includes(action)) {
let libraryIDs = [];
for (let id of ids) {
let item = Zotero.Items.get(id);
if (item && (item.isNote() || item.isRegularItem())) {
libraryIDs.push(item.libraryID);
}
else if (action == 'delete') {
libraryIDs.push(extraData[id].libraryID);
}
}
else if (action == 'delete') {
libraryIDs.push(extraData[id].libraryID);
}
}
for (let context of Array.from(this._notesPaneDeck.children)) {
if (libraryIDs.includes(context.libraryID)) {
context.affectedIDs = new Set([...context.affectedIDs, ...ids]);
context.update();
for (let context of Array.from(this._notesPaneDeck.children)) {
if (libraryIDs.includes(context.libraryID)) {
context.affectedIDs = new Set([...context.affectedIDs, ...ids]);
context.update();
}
}
}
}