@@ -, +, @@ - removing authnotrequired flag from scripts - fixing opac buttons - chmod +x for recalls test files --- koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-detail-sidebar.inc | 2 +- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recall.tt | 2 +- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 4 ++-- members/recallshistory.pl | 1 - opac/opac-recall.pl | 1 - opac/opac-recalls.pl | 1 - recalls/recall_pickup_slip.pl | 1 - recalls/recalls_old_queue.pl | 1 - recalls/recalls_overdue.pl | 1 - recalls/recalls_queue.pl | 1 - recalls/recalls_to_pull.pl | 1 - recalls/recalls_waiting.pl | 1 - recalls/request.pl | 1 - t/db_dependent/Koha/Patron.t | 2 +- t/db_dependent/Koha/Recall.t | 0 t/db_dependent/Koha/Recalls.t | 0 16 files changed, 5 insertions(+), 15 deletions(-) mode change 100644 => 100755 t/db_dependent/Koha/Recall.t mode change 100644 => 100755 t/db_dependent/Koha/Recalls.t --- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-detail-sidebar.inc +++ a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-detail-sidebar.inc @@ -12,7 +12,7 @@ [% END %] [% IF Koha.Preference('UseRecalls') %] -
  • +
  • Place recall
  • [% END %] --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recall.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-recall.tt @@ -151,7 +151,7 @@
    - + Cancel
    --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt @@ -777,7 +777,7 @@ - + [% END %] @@ -1126,7 +1126,7 @@ }); } - $("#cancel_recall").click(function(e){ + $(".cancel_recall").click(function(e){ return confirmDelete(_("Are you sure you want to remove this recall?")); }); }); --- a/members/recallshistory.pl +++ a/members/recallshistory.pl @@ -27,7 +27,6 @@ my ($template, $loggedinuser, $cookie)= get_template_and_user( template_name => "members/recallshistory.tt", query => $input, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => 1 }, debug => 1, } --- a/opac/opac-recall.pl +++ a/opac/opac-recall.pl @@ -29,7 +29,6 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user( template_name => "opac-recall.tt", query => $query, type => "opac", - authnotrequired => 0, } ); --- a/opac/opac-recalls.pl +++ a/opac/opac-recalls.pl @@ -29,7 +29,6 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user( template_name => "opac-recalls.tt", query => $input, type => "opac", - authnotrequired => 0, } ); --- a/recalls/recall_pickup_slip.pl +++ a/recalls/recall_pickup_slip.pl @@ -30,7 +30,6 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( template_name => "circ/printslip.tt", query => $input, type => "intranet", - authnotrequired => 0, flagsrequired => { circulate => "circulate_remaining_permissions" }, } ); --- a/recalls/recalls_old_queue.pl +++ a/recalls/recalls_old_queue.pl @@ -28,7 +28,6 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( template_name => "recalls/recalls_old_queue.tt", query => $query, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => "manage_recalls" }, debug => 1, } --- a/recalls/recalls_overdue.pl +++ a/recalls/recalls_overdue.pl @@ -29,7 +29,6 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( template_name => "recalls/recalls_overdue.tt", query => $query, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => "manage_recalls" }, debug => 1, } --- a/recalls/recalls_queue.pl +++ a/recalls/recalls_queue.pl @@ -28,7 +28,6 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( template_name => "recalls/recalls_queue.tt", query => $query, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => 'manage_recalls' }, debug => 1, } --- a/recalls/recalls_to_pull.pl +++ a/recalls/recalls_to_pull.pl @@ -28,7 +28,6 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( template_name => "recalls/recalls_to_pull.tt", query => $query, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => 'manage_recalls' }, debug => 1, } --- a/recalls/recalls_waiting.pl +++ a/recalls/recalls_waiting.pl @@ -31,7 +31,6 @@ my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( template_name => "recalls/recalls_waiting.tt", query => $query, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => "manage_recalls" }, debug => 1, } --- a/recalls/request.pl +++ a/recalls/request.pl @@ -30,7 +30,6 @@ my ($template, $loggedinuser, $cookie)= get_template_and_user( template_name => "recalls/request.tt", query => $input, type => "intranet", - authnotrequired => 0, flagsrequired => { recalls => "manage_recalls" }, debug => 1, } --- a/t/db_dependent/Koha/Patron.t +++ a/t/db_dependent/Koha/Patron.t @@ -19,7 +19,7 @@ use Modern::Perl; -use Test::More tests => 6; +use Test::More tests => 7; use Test::Exception; use Koha::Database;