www

commit fd5c48ac513fb5481997327e335e21f3adbf2697
parent ed6d628f4e1af57578f4f42b47f886a996d8ca36
Author: Dan Stillman <dstillman@zotero.org>
Date:   Fri,  1 Sep 2017 13:21:43 -0400

Simplify Zotero.ItemTreeView.prototype.getSortedItems()

Diffstat:
M
chrome/content/zotero/xpcom/itemTreeView.js
 | 
15
+++------------

1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/chrome/content/zotero/xpcom/itemTreeView.js b/chrome/content/zotero/xpcom/itemTreeView.js
@@ -2308,20 +2308,11 @@ Zotero.ItemTreeView.prototype.getVisibleFields = function() {
 /**
  * Returns an array of items of visible items in current sort order
  *
- * @param	bool	asIDs		Return itemIDs
- * @return	array				An array of Zotero.Item objects or itemIDs
+ * @param {Boolean} asIDs - Return itemIDs
+ * @return {Zotero.Item[]|Integer[]} - An array of Zotero.Item objects or itemIDs
  */
 Zotero.ItemTreeView.prototype.getSortedItems = function(asIDs) {
-	var items = [];
-	for (let item of this._rows) {
-		if (asIDs) {
-			items.push(item.ref.id);
-		}
-		else {
-			items.push(item.ref);
-		}
-	}
-	return items;
+	return this._rows.map(row => asIDs ? row.ref.id : row.ref);
 }