Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
warn identify_myself(%params) if $DEBUG;
my $course_id = $params{'course_id'};
warn identify_myself(%params) if $DEBUG;
my $course_id = $params{'course_id'};
- my $enabled = $params{'enabled'} || 0;
+ my $enabled = $params{'enabled'} || 0;
my $lookfor = ( $enabled eq 'yes' ) ? 'no' : 'yes';
my $lookfor = ( $enabled eq 'yes' ) ? 'no' : 'yes';
if ( $enabled eq 'yes' ) {
foreach my $course_reserve (@$course_reserves) {
if ( $enabled eq 'yes' ) {
foreach my $course_reserve (@$course_reserves) {
- if (
- CountCourseReservesForItem(
+ if (CountCourseReservesForItem(
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes'
)
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes'
)
EnableOrDisableCourseItem(
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes',
EnableOrDisableCourseItem(
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes',
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes'
)
ci_id => $course_reserve->{'ci_id'},
enabled => 'yes'
)
EnableOrDisableCourseItem(
ci_id => $course_reserve->{'ci_id'},
enabled => 'no',
EnableOrDisableCourseItem(
ci_id => $course_reserve->{'ci_id'},
enabled => 'no',
return unless ( $cardnumbers || $borrowernumbers );
return if ( $cardnumbers && $borrowernumbers );
return unless ( $cardnumbers || $borrowernumbers );
return if ( $cardnumbers && $borrowernumbers );
- my (@cardnumbers, @borrowernumbers);
+ my ( @cardnumbers, @borrowernumbers );
@cardnumbers = @$cardnumbers if ( ref($cardnumbers) eq 'ARRAY' );
@borrowernumbers = @$borrowernumbers
if ( ref($borrowernumbers) eq 'ARRAY' );
@cardnumbers = @$cardnumbers if ( ref($cardnumbers) eq 'ARRAY' );
@borrowernumbers = @$borrowernumbers
if ( ref($borrowernumbers) eq 'ARRAY' );
my $dbh = C4::Context->dbh;
my $dbh = C4::Context->dbh;
- $dbh->do( "DELETE FROM course_instructors WHERE course_id = ?",
- undef, $course_id )
+ $dbh->do( "DELETE FROM course_instructors WHERE course_id = ?", undef, $course_id )
if ( $mode eq 'replace' );
my $query;
if ( $mode eq 'replace' );
my $query;
FROM borrowers
WHERE $field IN ( $placeholders )
";
FROM borrowers
WHERE $field IN ( $placeholders )
";
$query = "
DELETE FROM course_instructors
WHERE course_id = ?
$query = "
DELETE FROM course_instructors
WHERE course_id = ?
course_item => $course_item,
%params
);
course_item => $course_item,
%params
);
$ci_id = _AddCourseItem(%params);
}
$ci_id = _AddCourseItem(%params);
}
foreach (@FIELDS) {
if ( $params{$_}
&& $course_item->{$_}
foreach (@FIELDS) {
if ( $params{$_}
&& $course_item->{$_}
- && $params{$_} ne $course_item->{$_} )
- {
+ && $params{$_} ne $course_item->{$_} ) {
$mod_params{$_} = $params{$_};
}
}
$mod_params{$_} = $params{$_};
}
}
- my $query =
- "UPDATE course_items SET "
- . join( ',', map { "$_=?" } @fields_to_update )
- . " WHERE ci_id = ?";
+ my $query = "UPDATE course_items SET " . join( ',', map { "$_=?" } @fields_to_update ) . " WHERE ci_id = ?";
C4::Context->dbh->do( $query, undef, @values_to_update, $params{'ci_id'} )
if (@values_to_update);
C4::Context->dbh->do( $query, undef, @values_to_update, $params{'ci_id'} )
if (@values_to_update);
}
ModItem( $mod_item_params, undef, $course_item->{'itemnumber'} );
}
ModItem( $mod_item_params, undef, $course_item->{'itemnumber'} );
- my $query =
- "UPDATE course_items SET "
- . join( ',', map { "$_=NULL" } @fields_to_null )
- . " WHERE ci_id = ?";
+ my $query = "UPDATE course_items SET " . join( ',', map { "$_=NULL" } @fields_to_null ) . " WHERE ci_id = ?";
C4::Context->dbh->do( $query, undef, $ci_id ) if (@fields_to_null);
}
C4::Context->dbh->do( $query, undef, $ci_id ) if (@fields_to_null);
}
";
$sth = $dbh->prepare($query);
$sth->execute($cr_id);
";
$sth = $dbh->prepare($query);
$sth->execute($cr_id);
my $query = "
SELECT * FROM course_reserves
WHERE course_id = ? AND ci_id = ?
my $query = "
SELECT * FROM course_reserves
WHERE course_id = ? AND ci_id = ?
return unless ( $course_id && $ci_id );
return unless ( $course_id && $ci_id );
- my $course_reserve =
- GetCourseReserve( course_id => $course_id, ci_id => $ci_id );
+ my $course_reserve = GetCourseReserve( course_id => $course_id, ci_id => $ci_id );
my $cr_id;
my $dbh = C4::Context->dbh;
my $cr_id;
my $dbh = C4::Context->dbh;
WHERE cr_id = ?
";
$dbh->do( $query, undef, $staff_note, $public_note, $cr_id );
WHERE cr_id = ?
";
$dbh->do( $query, undef, $staff_note, $public_note, $cr_id );
my $query = "
INSERT INTO course_reserves SET
course_id = ?,
my $query = "
INSERT INTO course_reserves SET
course_id = ?,
staff_note = ?,
public_note = ?
";
staff_note = ?,
public_note = ?
";
- $dbh->do( $query, undef, $course_id, $ci_id, $staff_note,
- $public_note );
- $cr_id =
- $dbh->last_insert_id( undef, undef, 'course_reserves', 'cr_id' );
+ $dbh->do( $query, undef, $course_id, $ci_id, $staff_note, $public_note );
+ $cr_id = $dbh->last_insert_id( undef, undef, 'course_reserves', 'cr_id' );
}
my $course = GetCourse($course_id);
}
my $course = GetCourse($course_id);
my (%params) = @_;
warn identify_myself(%params) if $DEBUG;
my (%params) = @_;
warn identify_myself(%params) if $DEBUG;
- my $course_id = $params{'course_id'};
- my $ci_id = $params{'ci_id'};
- my $include_items = $params{'include_items'};
- my $include_count = $params{'include_count'};
+ my $course_id = $params{'course_id'};
+ my $ci_id = $params{'ci_id'};
+ my $include_items = $params{'include_items'};
+ my $include_count = $params{'include_count'};
my $include_courses = $params{'include_courses'};
return unless ( $course_id || $ci_id );
my $include_courses = $params{'include_courses'};
return unless ( $course_id || $ci_id );
if ($include_items) {
foreach my $cr (@$course_reserves) {
$cr->{'course_item'} = GetCourseItem( ci_id => $cr->{'ci_id'} );
if ($include_items) {
foreach my $cr (@$course_reserves) {
$cr->{'course_item'} = GetCourseItem( ci_id => $cr->{'ci_id'} );
- $cr->{'item'} = GetBiblioFromItemNumber( $cr->{'itemnumber'} );
- $cr->{'issue'} = GetOpenIssue( $cr->{'itemnumber'} );
+ $cr->{'item'} = GetBiblioFromItemNumber( $cr->{'itemnumber'} );
+ $cr->{'issue'} = GetOpenIssue( $cr->{'itemnumber'} );
}
}
if ($include_count) {
foreach my $cr (@$course_reserves) {
}
}
if ($include_count) {
foreach my $cr (@$course_reserves) {
- $cr->{'reserves_count'} =
- CountCourseReservesForItem( ci_id => $cr->{'ci_id'} );
+ $cr->{'reserves_count'} = CountCourseReservesForItem( ci_id => $cr->{'ci_id'} );
}
}
if ($include_courses) {
foreach my $cr (@$course_reserves) {
}
}
if ($include_courses) {
foreach my $cr (@$course_reserves) {
- $cr->{'courses'} =
- GetCourses( itemnumber => $cr->{'itemnumber'} );
+ $cr->{'courses'} = GetCourses( itemnumber => $cr->{'itemnumber'} );
## If there are no other course reserves for this item
## delete the course_item as well
## If there are no other course reserves for this item
## delete the course_item as well
- unless ( CountCourseReservesForItem( ci_id => $course_reserve->{'ci_id'} ) )
- {
+ unless ( CountCourseReservesForItem( ci_id => $course_reserve->{'ci_id'} ) ) {
DelCourseItem( ci_id => $course_reserve->{'ci_id'} );
}
DelCourseItem( ci_id => $course_reserve->{'ci_id'} );
}
return unless ( $ci_id || $itemnumber );
return unless ( $ci_id || $itemnumber );
- my $course_item =
- GetCourseItem( ci_id => $ci_id, itemnumber => $itemnumber );
+ my $course_item = GetCourseItem( ci_id => $ci_id, itemnumber => $itemnumber );
my @params = ( $course_item->{'ci_id'} );
push( @params, $enabled ) if ($enabled);
my @params = ( $course_item->{'ci_id'} );
push( @params, $enabled ) if ($enabled);
my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare($query);
my $dbh = C4::Context->dbh;
my $sth = $dbh->prepare($query);
- $sth->execute(@params, $enabled);
+ $sth->execute( @params, $enabled );
my $courses = $sth->fetchall_arrayref( {} );
my $courses = $sth->fetchall_arrayref( {} );
sub GetByCode {
my ( $self, $category, $code, $opac ) = @_;
sub GetByCode {
my ( $self, $category, $code, $opac ) = @_;
- return encode('UTF-8', GetAuthorisedValueByCode( $category, $code, $opac ) );
+ return encode( 'UTF-8', GetAuthorisedValueByCode( $category, $code, $opac ) );
my $sth = C4::Context->dbh->prepare($query);
$sth->execute($branchcode);
my $b = $sth->fetchrow_hashref();
my $sth = C4::Context->dbh->prepare($query);
$sth->execute($branchcode);
my $b = $sth->fetchrow_hashref();
- return encode('UTF-8', $b->{'branchname'});
+ return encode( 'UTF-8', $b->{'branchname'} );
my $sth = C4::Context->dbh->prepare($query);
$sth->execute($itemtype);
my $d = $sth->fetchrow_hashref();
my $sth = C4::Context->dbh->prepare($query);
$sth->execute($itemtype);
my $d = $sth->fetchrow_hashref();
- return encode('UTF-8', $d->{'description'});
+ return encode( 'UTF-8', $d->{'description'} );
my $step = ( $action eq 'lookup' && $item ) ? '2' : '1';
my $step = ( $action eq 'lookup' && $item ) ? '2' : '1';
-my $tmpl = ( $course_id ) ? "add_items-step$step.tt" : "invalid-course.tt";
+my $tmpl = ($course_id) ? "add_items-step$step.tt" : "invalid-course.tt";
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
- {
- template_name => "course_reserves/$tmpl",
+ { template_name => "course_reserves/$tmpl",
query => $cgi,
type => "intranet",
authnotrequired => 0,
query => $cgi,
type => "intranet",
authnotrequired => 0,
if ( $action eq 'lookup' ) {
my $course_item = GetCourseItem( itemnumber => $item->{'itemnumber'} );
if ( $action eq 'lookup' ) {
my $course_item = GetCourseItem( itemnumber => $item->{'itemnumber'} );
- my $course_reserve = ($course_item)
+ my $course_reserve =
+ ($course_item)
? GetCourseReserve(
course_id => $course_id,
ci_id => $course_item->{'ci_id'}
? GetCourseReserve(
course_id => $course_id,
ci_id => $course_item->{'ci_id'}
branches => GetBranchesLoop(),
);
branches => GetBranchesLoop(),
);
-}
-elsif ( $action eq 'add' ) {
+} elsif ( $action eq 'add' ) {
my $ci_id = ModCourseItem(
itemnumber => $cgi->param('itemnumber'),
itype => $cgi->param('itype'),
my $ci_id = ModCourseItem(
itemnumber => $cgi->param('itemnumber'),
itype => $cgi->param('itype'),
my $flagsrequired;
$flagsrequired->{coursereserves} = 'delete_reserves' if ( $action eq 'del_reserve' );
my $flagsrequired;
$flagsrequired->{coursereserves} = 'delete_reserves' if ( $action eq 'del_reserve' );
-my $tmpl = ( $course_id ) ? "course-details.tt" : "invalid-course.tt";
+my $tmpl = ($course_id) ? "course-details.tt" : "invalid-course.tt";
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
- {
- template_name => "course_reserves/$tmpl",
+ { template_name => "course_reserves/$tmpl",
query => $cgi,
type => "intranet",
authnotrequired => 0,
query => $cgi,
type => "intranet",
authnotrequired => 0,
my $cgi = new CGI;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my $cgi = new CGI;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
- {
- template_name => "course_reserves/course.tt",
+ { template_name => "course_reserves/course.tt",
query => $cgi,
type => "intranet",
authnotrequired => 0,
query => $cgi,
type => "intranet",
authnotrequired => 0,
$template->param(
departments => GetAuthorisedValues('DEPARTMENT'),
$template->param(
departments => GetAuthorisedValues('DEPARTMENT'),
- terms => GetAuthorisedValues('TERM'),
+ terms => GetAuthorisedValues('TERM'),
);
output_html_with_http_headers $cgi, $cookie, $template->output;
);
output_html_with_http_headers $cgi, $cookie, $template->output;
my $cgi = new CGI;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my $cgi = new CGI;
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
- {
- template_name => "about.tmpl",
+ { template_name => "about.tmpl",
query => $cgi,
type => "intranet",
authnotrequired => 0,
query => $cgi,
type => "intranet",
authnotrequired => 0,
if ( $action eq 'del' ) {
DelCourse( $cgi->param('course_id') );
print $cgi->redirect("/cgi-bin/koha/course_reserves/course-reserves.pl");
if ( $action eq 'del' ) {
DelCourse( $cgi->param('course_id') );
print $cgi->redirect("/cgi-bin/koha/course_reserves/course-reserves.pl");
my %params;
$params{'course_id'} = $cgi->param('course_id')
my %params;
$params{'course_id'} = $cgi->param('course_id')
$params{'staff_note'} = $cgi->param('staff_note');
$params{'public_note'} = $cgi->param('public_note');
$params{'students_count'} = $cgi->param('students_count');
$params{'staff_note'} = $cgi->param('staff_note');
$params{'public_note'} = $cgi->param('public_note');
$params{'students_count'} = $cgi->param('students_count');
- $params{'enabled'} = ( $cgi->param('enabled') eq 'on' ) ? 'yes' : 'no';
+ $params{'enabled'} = ( $cgi->param('enabled') eq 'on' ) ? 'yes' : 'no';
my $course_id = ModCourse(%params);
my $course_id = ModCourse(%params);
cardnumbers => \@instructors,
course_id => $course_id
);
cardnumbers => \@instructors,
course_id => $course_id
);
- print $cgi->redirect(
- "/cgi-bin/koha/course_reserves/course-details.pl?course_id=$course_id");
+ print $cgi->redirect("/cgi-bin/koha/course_reserves/course-details.pl?course_id=$course_id");
my $cgi = new CGI;
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
my $cgi = new CGI;
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
- {
- template_name => "opac-course-details.tmpl",
+ { template_name => "opac-course-details.tmpl",
query => $cgi,
type => "opac",
authnotrequired => 1,
query => $cgi,
type => "opac",
authnotrequired => 1,
my $action = $cgi->param('action') || '';
my $course_id = $cgi->param('course_id');
my $action = $cgi->param('action') || '';
my $course_id = $cgi->param('course_id');
-die( "No course_id given" ) unless ( $course_id );
+die("No course_id given") unless ($course_id);
if ( $action eq 'del_reserve' ) {
DelCourseReserve( cr_id => $cgi->param('cr_id') );
}
if ( $action eq 'del_reserve' ) {
DelCourseReserve( cr_id => $cgi->param('cr_id') );
}
-my $course = GetCourse( $course_id );
+my $course = GetCourse($course_id);
my $course_reserves = GetCourseReserves( course_id => $course_id, include_items => 1, include_count => 1 );
$template->param(
my $course_reserves = GetCourseReserves( course_id => $course_id, include_items => 1, include_count => 1 );
$template->param(
course_reserves => $course_reserves,
);
course_reserves => $course_reserves,
);
my $cgi = new CGI;
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
my $cgi = new CGI;
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
- {
- template_name => "opac-course-reserves.tmpl",
+ { template_name => "opac-course-reserves.tmpl",
query => $cgi,
type => "opac",
authnotrequired => 1,
query => $cgi,
type => "opac",
authnotrequired => 1,
my $courses = SearchCourses( term => $search_on, enabled => 'yes' );
if ( @$courses == 1 ) {
my $courses = SearchCourses( term => $search_on, enabled => 'yes' );
if ( @$courses == 1 ) {
- print $cgi->redirect("/cgi-bin/koha/opac-course-details.pl?course_id=" . $courses->[0]->{'course_id'});
+ print $cgi->redirect( "/cgi-bin/koha/opac-course-details.pl?course_id=" . $courses->[0]->{'course_id'} );
} else {
$template->param( courses => $courses );
output_html_with_http_headers $cgi, $cookie, $template->output;
} else {
$template->param( courses => $courses );
output_html_with_http_headers $cgi, $cookie, $template->output;