www

commit 6174641cfa1b36d0952d5a95b2e43ad5300b5ddf
parent bb489a45c36148b551763d3b1630ae7e288597f4
Author: Dan Stillman <dstillman@zotero.org>
Date:   Sat,  1 Apr 2017 03:13:26 -0400

Reload primary data after tag rename instead of updating .synced

Follow-up to bb489a45c3 to mirror what Tags.removeFromLibrary() does

Diffstat:
M
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
@@ -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);
 				})
 			);