@@ -, +, @@ --- C4/CourseReserves.pm | 8 ++++---- t/db_dependent/CourseReserves.t | 5 ++++- 2 files changed, 8 insertions(+), 5 deletions(-) --- a/C4/CourseReserves.pm +++ a/C4/CourseReserves.pm @@ -966,9 +966,9 @@ sub SearchCourses { my $enabled = $params{'enabled'} || '%'; my @params; - my $query = "SELECT c.* FROM courses c"; - - $query .= " + my $query = " + SELECT c.course_id, c.department, c.course_number, c.section, c.course_name, c.term, c.staff_note, c.public_note, c.students_count, c.enabled, c.timestamp + FROM courses c LEFT JOIN course_instructors ci ON ( c.course_id = ci.course_id ) LEFT JOIN borrowers b @@ -992,7 +992,7 @@ sub SearchCourses { ) AND c.enabled LIKE ? - GROUP BY c.course_id + GROUP BY c.course_id, c.department, c.course_number, c.section, c.course_name, c.term, c.staff_note, c.public_note, c.students_count, c.enabled, c.timestamp "; $term //= ''; --- a/t/db_dependent/CourseReserves.t +++ a/t/db_dependent/CourseReserves.t @@ -17,7 +17,7 @@ use Modern::Perl; -use Test::More tests => 26; +use Test::More tests => 27; use Koha::Database; use t::lib::TestBuilder; @@ -136,4 +136,7 @@ DelCourse($course_id); $course = GetCourse($course_id); ok( !defined( $course->{'course_id'} ), "DelCourse deleted course successfully" ); +$courses = SearchCourses(); # FIXME Lack of tests for SearchCourses +is( ref($courses), 'ARRAY', 'SearchCourses should not crash and return an arrayref' ); + $schema->storage->txn_rollback; --