Bug 10403: (follow-up) fix test to use vendor created earlier during test

[koha.git] / course_reserves / add_items.pl

diff --git a/course_reserves/add_items.pl b/course_reserves/add_items.pl
index bbf67d5..9429d91 100755 (executable)
--- a/course_reserves/add_items.pl
+++ b/course_reserves/add_items.pl
@@ -28,7 +28,7 @@ use C4::Koha;
 use C4::Biblio;
 use C4::Branch;
 
-use C4::CourseReserves;
+use C4::CourseReserves qw(GetCourse GetCourseItem GetCourseReserve ModCourseItem ModCourseReserve);
 
 my $cgi = new CGI;
 
@@ -36,15 +36,13 @@ my $action    = $cgi->param('action')    || '';
 my $course_id = $cgi->param('course_id') || '';
 my $barcode   = $cgi->param('barcode')   || '';
 
-die('No course_id provided') unless ($course_id);
-
 my $item = GetBiblioFromItemNumber( undef, $barcode );
 
 my $step = ( $action eq 'lookup' && $item ) ? '2' : '1';
 
+my $tmpl = ($course_id) ? "add_items-step$step.tt" : "invalid-course.tt";
 my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
-    {
-        template_name   => "course_reserves/add_items-step$step.tmpl",
+    {   template_name   => "course_reserves/$tmpl",
         query           => $cgi,
         type            => "intranet",
         authnotrequired => 0,
@@ -58,7 +56,8 @@ $template->param( course => GetCourse($course_id) );
 
 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'}
@@ -76,8 +75,7 @@ if ( $action eq 'lookup' ) {
         branches  => GetBranchesLoop(),
     );
 
-}
-elsif ( $action eq 'add' ) {
+} elsif ( $action eq 'add' ) {
     my $ci_id = ModCourseItem(
         itemnumber    => $cgi->param('itemnumber'),
         itype         => $cgi->param('itype'),