diff --git a/chrome/content/zotero/advancedSearch.js b/chrome/content/zotero/advancedSearch.js index 2f2e2c917..24b293b51 100644 --- a/chrome/content/zotero/advancedSearch.js +++ b/chrome/content/zotero/advancedSearch.js @@ -138,7 +138,9 @@ var ZoteroAdvancedSearch = new function() { this.onLibraryChange = function (libraryID) { _libraryID = libraryID; - document.getElementById('zotero-search-save').disabled = !Zotero.Libraries.isEditable(libraryID); + var library = Zotero.Libraries.get(libraryID); + var isEditable = library.editable && library.libraryType != 'publications'; + document.getElementById('zotero-search-save').disabled = !isEditable; } diff --git a/chrome/content/zotero/bindings/zoterosearch.xml b/chrome/content/zotero/bindings/zoterosearch.xml index f6d0e57cb..c21ba3510 100644 --- a/chrome/content/zotero/bindings/zoterosearch.xml +++ b/chrome/content/zotero/bindings/zoterosearch.xml @@ -88,27 +88,22 @@ var libraryID = this.searchRef.libraryID; var libraryIndex = 0; - // Add My Library - var menuitem = document.createElement('menuitem'); - menuitem.setAttribute('label', Zotero.getString('pane.collections.library')); - menuitem.setAttribute('libraryID', Zotero.Libraries.userLibraryID); - menupopup.appendChild(menuitem); - - // Add groups - var groups = Zotero.Groups.getAll(); - for (let i = 0; i < groups.length; i++) { - let group = groups[i]; + var libraries = Zotero.Libraries.getAll(); + var selectedIndex = 0; + var i = 0; + for (let library of libraries) { let menuitem = document.createElement('menuitem'); - menuitem.setAttribute('label', group.name); - menuitem.setAttribute('libraryID', group.libraryID); - if (group.libraryID == libraryID) { - libraryIndex = i + 1; - } + menuitem.setAttribute('label', library.name); + menuitem.setAttribute('libraryID', library.libraryID); menupopup.appendChild(menuitem); + if (library.libraryID == libraryID) { + selectedIndex = i; + } + i++ } menulist.appendChild(menupopup); - menulist.selectedIndex = libraryIndex; + menulist.selectedIndex = selectedIndex; if (this.searchRef.id) { this.id('libraryMenu').disabled = true;