Bug 8367: (follow-up) Remove pickup delay message from OPAC
It adds unnecessary complexity and information. Signed-off-by: Emmi Takkinen <emmi.takkinen@koha-suomi.fi> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Kelly <kelly@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Kelly <kelly@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
ffde32e0d7
commit
f47c5f09f7
2 changed files with 0 additions and 43 deletions
|
@ -283,10 +283,6 @@
|
|||
[% END # / UNLESS singleBranchMode %]
|
||||
</ul>
|
||||
|
||||
[% IF pickup_delays %]
|
||||
<p class="pickup_delay" style="display:none;">When your item is ready for pickup, you will have <span id="pickup_delay_days_[% bibitemloo.biblionumber | html %]"></span> days to pick it up from this library.</p>
|
||||
[% END %]
|
||||
|
||||
<a class="toggle-hold-options" id="toggle-hold-options-[% bibitemloo.biblionumber | html %]" style="display:none;" href="#">Show more options</a>
|
||||
|
||||
<div id="hold-options-[% bibitemloo.biblionumber | html %]" class="hold-options">
|
||||
|
@ -510,20 +506,6 @@
|
|||
var MSG_EMPTY_START_DATE = _("Hold start date should be filled.");
|
||||
var MSG_EMPTY_END_DATE = _("Hold expiration date should be filled.");
|
||||
|
||||
function calculate_delay_days(){
|
||||
$(".branch > select").each(function(){
|
||||
var selected_branch = $(this).val();
|
||||
var branch_biblionumber = $(this).attr('id');
|
||||
var biblionumber = branch_biblionumber.slice( branch_biblionumber.indexOf('_')+1 );
|
||||
[% FOREACH branchcode IN pickup_delays.keys %]
|
||||
var branchcode = "[% branchcode | html %]";
|
||||
if ( branchcode == selected_branch ) {
|
||||
$("#pickup_delay_days_" + biblionumber).text([% pickup_delays.$branchcode | html %]);
|
||||
}
|
||||
[% END %]
|
||||
});
|
||||
}
|
||||
|
||||
$(document).ready(function() {
|
||||
|
||||
$(".toggle_unholdable").click(function(e){
|
||||
|
@ -531,12 +513,6 @@
|
|||
$(this).parent('div').find(".unholdable").toggle();
|
||||
});
|
||||
|
||||
calculate_delay_days();
|
||||
$(".pickup_delay").show();
|
||||
$(".branch > select").change(function(){
|
||||
calculate_delay_days();
|
||||
});
|
||||
|
||||
$("#hold-request-form").preventDoubleFormSubmit();
|
||||
var copiesRowId = null;
|
||||
var wasSpecific = false;
|
||||
|
|
|
@ -626,23 +626,4 @@ if (
|
|||
);
|
||||
}
|
||||
|
||||
my @branches = Koha::Libraries->search->as_list;
|
||||
my %hold_pickup_delay_by_branch = ();
|
||||
|
||||
foreach my $branch ( @branches ) {
|
||||
my $rule = Koha::CirculationRules->get_effective_rule({
|
||||
categorycode => $patron->categorycode,
|
||||
branchcode => $branch->branchcode,
|
||||
itemtype => undef,
|
||||
rule_name => 'holds_pickup_period',
|
||||
});
|
||||
if ( $rule and $rule->rule_value ) {
|
||||
$hold_pickup_delay_by_branch{$branch->branchcode} = $rule->rule_value;
|
||||
} else {
|
||||
$hold_pickup_delay_by_branch{$branch->branchcode} = C4::Context->preference('ReservesMaxPickUpDelay');
|
||||
}
|
||||
}
|
||||
|
||||
$template->param( pickup_delays => \%hold_pickup_delay_by_branch );
|
||||
|
||||
output_html_with_http_headers $query, $cookie, $template->output, undef, { force_no_caching => 1 };
|
||||
|
|
Loading…
Reference in a new issue