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

Fix a /saveItems session management race condition bug

Browse source
This commit is contained in:
Adomas Venčkauskas 2018-12-14 14:11:42 +02:00
parent a24cada451
commit 90a70f7c31
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
chrome/content/zotero/xpcom/connector/server_connector.js
View file

@ -742,9 +742,10 @@ Zotero.Server.Connector.SaveItems.prototype = {
session,
targetID,
requestData,
function (topLevelItems) {
function (jsonItems, items) {
session.addItems(items);
// Only return the properties the connector needs
topLevelItems = topLevelItems.map((item) => {
jsonItems = jsonItems.map((item) => {
let o = {
id: item.id,
title: item.title,
@ -764,7 +765,7 @@ Zotero.Server.Connector.SaveItems.prototype = {
};
return o;
});
resolve([201, "application/json", JSON.stringify({items: topLevelItems})]);
resolve([201, "application/json", JSON.stringify({items: jsonItems})]);
}
)
// Add items to session once all attachments have been saved

2
chrome/content/zotero/xpcom/translation/translate_item.js
View file

@ -179,7 +179,7 @@ Zotero.Translate.ItemSaver.prototype = {
}.bind(this));
if (itemsDoneCallback) {
itemsDoneCallback(items.map(item => jsonByItem.get(item)));
itemsDoneCallback(items.map(item => jsonByItem.get(item)), items);
}
// Save standalone attachments