From 6174641cfa1b36d0952d5a95b2e43ad5300b5ddf Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Sat, 1 Apr 2017 03:13:26 -0400 Subject: [PATCH] Reload primary data after tag rename instead of updating .synced Follow-up to bb489a45c3 to mirror what Tags.removeFromLibrary() does --- chrome/content/zotero/xpcom/data/tags.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/chrome/content/zotero/xpcom/data/tags.js b/chrome/content/zotero/xpcom/data/tags.js index 55250db05..599e70eec 100644 --- a/chrome/content/zotero/xpcom/data/tags.js +++ b/chrome/content/zotero/xpcom/data/tags.js @@ -258,13 +258,11 @@ Zotero.Tags = new function() { + 'WHERE tagID=? AND itemID IN (' + placeholders + ')'; yield Zotero.DB.queryAsync(sql, [newTagID, oldTagID].concat(chunk)); - sql = 'UPDATE items SET clientDateModified=?, synced=0 ' + sql = 'UPDATE items SET synced=0, clientDateModified=? ' + 'WHERE itemID IN (' + placeholders + ')' yield Zotero.DB.queryAsync(sql, [Zotero.DB.transactionDateTime].concat(chunk)); - chunk.forEach(id => Zotero.Items.get(id).updateSynced(false, true)); - - yield Zotero.Items.reload(oldItemIDs, ['tags'], true); + yield Zotero.Items.reload(oldItemIDs, ['primaryData', 'tags'], true); }) );