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.
This commit is contained in:
Abe Jellinek 2025-08-20 01:23:47 -04:00 • committed by GitHub
parent bdf49b9b00
commit d35bdd333b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -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,