Bug 41422: Fix I18N issue with FilterSearchResultsByLoggedInBranch
To test: 1. Apply patch 2. Search fof the FilterSearchResultsByLoggedInBranch system preference and set it to 'Do'. 3. Do a catalog search and notice the 'Show local items only' button above the "Location" column. 4. Click it to show information ( in that column ) that is only related to the branch you are currently logged in at. 5. In the same result set have some items that are checked out, withdrawn, lost, damaged, and notforlaon. 6. Play more with the search results and ensure everything is accurate. 7. Set the FilterSearchResultsByLoggedInBranch system preference to 'Don't' and confirm that there is no button to filter by branch anymore. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
1f51dab88e
commit
27f4bb4c9d
|
|
@ -556,7 +556,10 @@
|
|||
<div id="searchresults">
|
||||
[% INCLUDE 'page-numbers.inc' %]
|
||||
[% IF Koha.Preference('FilterSearchResultsByLoggedInBranch') %]
|
||||
<button href="#" id="toggleitems" class="btn btn-primary btn-xs show-local-items" style="float: right;">Show local items only</button>
|
||||
<button id="toggleitems" class="btn btn-primary btn-xs" data-mode="all" style="float: right;">
|
||||
<span class="when-all">[% t("Show local items only") | html %]</span>
|
||||
<span class="when-local" style="display:none">[% t("Show items in all libraries") | html %]</span>
|
||||
</button>
|
||||
[% END %]
|
||||
<form action="/cgi-bin/koha/catalogue/search.pl" method="get" name="bookbag_form" id="bookbag_form">
|
||||
[% IF ( searchdesc ) %]
|
||||
|
|
@ -986,7 +989,12 @@
|
|||
}
|
||||
});
|
||||
$('#toggleitems').click( function() {
|
||||
$('#toggleitems').text() === 'Show local items only' ? $('#toggleitems').text("Show items in all libraries") : $('#toggleitems').text("Show local items only");
|
||||
let this_toggle = $(this);
|
||||
let show_all = this_toggle.data('mode') === 'all';
|
||||
|
||||
this_toggle.find('.when-local, .when-all').toggle();
|
||||
this_toggle.data('mode', show_all ? 'local' : 'all');
|
||||
|
||||
$('.branch_specific').toggle();
|
||||
$('.all').toggle();
|
||||
$('ul[class*="_loop_items"] li').each( function() {
|
||||
|
|
@ -994,11 +1002,11 @@
|
|||
$(this).toggle();
|
||||
}
|
||||
});
|
||||
$(this).toggleClass('show-local-items show-all-items');
|
||||
if ($(this).hasClass('show-local-items')) {
|
||||
localStorage.setItem("show_local_items", 'all');
|
||||
} else {
|
||||
|
||||
if (show_all) {
|
||||
localStorage.setItem("show_local_items", 'local');
|
||||
} else {
|
||||
localStorage.setItem("show_local_items", 'all');
|
||||
}
|
||||
});
|
||||
[% END %]
|
||||
|
|
|
|||
Loading…
Reference in New Issue