From ae21b3113eb1cc61ef04cb177f83028f3f9c3564 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Mon, 27 Nov 2017 17:32:04 -0500
Subject: [PATCH] Avoid unnecessary item map refreshes during sorting

---
 chrome/content/zotero/xpcom/itemTreeView.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/chrome/content/zotero/xpcom/itemTreeView.js b/chrome/content/zotero/xpcom/itemTreeView.js
index 2317c053d..fa66d79bb 100644
--- a/chrome/content/zotero/xpcom/itemTreeView.js
+++ b/chrome/content/zotero/xpcom/itemTreeView.js
@@ -1461,9 +1461,10 @@ Zotero.ItemTreeView.prototype.sort = function (itemIDs) {
 		
 		for (let i = parentRows.length - 1; i >= 0; i--) {
 			let row = parentRows[i];
-			this._closeContainer(row);
-			this.toggleOpenState(row);
+			this._closeContainer(row, true);
+			this.toggleOpenState(row, true);
 		}
+		this._refreshItemRowMap();
 		
 		let numSorted = itemIDs.length - skipped.length;
 		if (numSorted) {