@@ -, +, @@ - Apply with the other patches and make sure there are no errors when following the previous test plan. --- admin/itemtypes.pl | 4 ++-- koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) --- a/admin/itemtypes.pl +++ a/admin/itemtypes.pl @@ -105,8 +105,8 @@ if ( $op eq 'add_form' ) { my $checkinmsgtype = $input->param('checkinmsgtype'); my $hideinopac = $input->param('hideinopac') // 0; my $searchcategory = $input->param('searchcategory'); - my $can_suspend_hold = $input->param('can_suspend_hold'); - my $can_cancel_hold = $input->param('can_cancel_hold'); + my $can_suspend_hold = $input->param('can_suspend_hold') // 0; + my $can_cancel_hold = $input->param('can_cancel_hold') // 0; if ( $itemtype and $is_a_modif ) { # it's a modification $itemtype->description($description); --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc +++ a/koha-tmpl/opac-tmpl/bootstrap/en/includes/holds-table.inc @@ -183,7 +183,7 @@ [% END # / IF SuspendHoldsOpac %] [% IF ! onlyinfo %] - [% IF ( HOLD.is_cancelable_from_opac ) %] + [% IF ( HOLD.is_cancelable_from_opac && ItemTypes.CanCancelHold( HOLD.biblio.biblioitem.itemtype )) %]
--