Browse Source

Bug 31893: Use checkauth and do not fetch about.tt template

This patch updates four scripts to use checkauth instead of get_template_and_user

reserve/modrequest.pl
reserve/modrequest_suspendall.pl
course_reserves/mod_course.pl
members/members-update-do.pl

No behvaiour change is expected

To test:
0 - Apply patch
1 - Place some holds for a patron
2 - From patron page, confirm you can suspend all
3 - confirm you can delete marked holds
4 - Edit a course in course-reserves
5 - Make a chaneg to personal details via OPAC
6 - Confirm change can be approved on staff side

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
master
Nick Clemens 3 weeks ago
committed by Martin Renvoize
parent
commit
fc650b3e87
Signed by: martin.renvoize GPG Key ID: 422B469130441A0F
  1. 11
      course_reserves/mod_course.pl
  2. 14
      members/members-update-do.pl
  3. 12
      reserve/modrequest.pl
  4. 12
      reserve/modrequest_suspendall.pl

11
course_reserves/mod_course.pl

@ -23,18 +23,13 @@ use CGI qw ( -utf8 );
use C4::Output;
use C4::Reserves;
use C4::Auth qw( get_template_and_user );
use C4::Auth qw( checkauth );
use C4::CourseReserves qw( DelCourse ModCourse ModCourseInstructors );
my $cgi = CGI->new;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{ template_name => "about.tt",
query => $cgi,
type => "intranet",
flagsrequired => { coursereserves => 'manage_courses' },
}
);
checkauth($cgi, 0, { coursereserves => 'manage_courses' }, 'intranet');
my $action = $cgi->param('action') || '';
my $course_id = $cgi->param('course_id');

14
members/members-update-do.pl

@ -19,7 +19,7 @@
use Modern::Perl;
use CGI qw ( -utf8 );
use C4::Auth qw( get_template_and_user );
use C4::Auth qw( checkauth );
use C4::Output;
use C4::Context;
use Koha::Patrons;
@ -27,17 +27,9 @@ use Koha::Patron::Modifications;
my $query = CGI->new;
# FIXME Should be a checkauth call
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => "about.tt",
query => $query,
type => "intranet",
flagsrequired => { borrowers => 'edit_borrowers' },
}
);
my ( $userid, $cookie, $sessionID, $flags ) = checkauth($query, 0, { borrowers => 'edit_borrowers' }, 'intranet');
my $logged_in_user = Koha::Patrons->find( $loggedinuser );
my $logged_in_user = Koha::Patrons->find({ userid => $userid });
my @params = $query->param;

12
reserve/modrequest.pl

@ -30,18 +30,12 @@ use Try::Tiny;
use C4::Output;
use C4::Reserves qw( ModReserve ModReserveCancelAll );
use C4::Auth qw( get_template_and_user );
use C4::Auth qw( checkauth );
use Koha::BackgroundJob::BatchUpdateBiblioHoldsQueue;
my $query = CGI->new;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => "about.tt",
query => $query,
type => "intranet",
flagsrequired => { reserveforothers => '*' },
}
);
checkauth($query, 0, { reserveforothers => '*' }, 'intranet');
my @reserve_id = $query->multi_param('reserve_id');
my @rank = $query->multi_param('rank-request');

12
reserve/modrequest_suspendall.pl

@ -26,17 +26,11 @@ use Modern::Perl;
use CGI qw ( -utf8 );
use C4::Output;
use C4::Reserves qw( SuspendAll );
use C4::Auth qw( get_template_and_user );
use C4::Auth qw( checkauth );
my $query = CGI->new;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => "about.tt",
query => $query,
type => "intranet",
flagsrequired => { reserveforothers => 1 },
}
);
checkauth($query, 0, { reserveforothers => '*' }, 'intranet');
my $borrowernumber = $query->param('borrowernumber');
my $suspend = $query->param('suspend');

Loading…
Cancel
Save