From d35bdd333b7d4077db83140e92446c7a0dc22747 Mon Sep 17 00:00:00 2001 From: Abe Jellinek Date: Wed, 20 Aug 2025 01:23:47 -0400 Subject: [PATCH] Fix errors when multi-selection contains collections in trash (#5497) By using getSelectedItems() instead of getSelectedObjects(). The code below all assumes that `items` contains items, and we handle general (item or collection) options above, so this seems more correct. And fix duplicate var. --- chrome/content/zotero/zoteroPane.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/chrome/content/zotero/zoteroPane.js b/chrome/content/zotero/zoteroPane.js index efa1da5e70..0164a6ff9b 100644 --- a/chrome/content/zotero/zoteroPane.js +++ b/chrome/content/zotero/zoteroPane.js @@ -3671,12 +3671,12 @@ var ZoteroPane = new function () { show.add(m.loadReport); } - var items = this.getSelectedObjects(); + var items = this.getSelectedItems(); if (items.length > 0) { // Multiple items selected if (items.length > 1) { - var multiple = '.multiple'; + multiple = '.multiple'; var canMerge = true, canIndex = true,