Attachment 11861 Details for Bug 8215 – Bug 8215 - Add Course Reserves - Followup - One Hit Searching Bugfix

[patch] Bug 8215 - Add Course Reserves - Followup - One Hit Searching Bugfix

Bug-8215---Add-Course-Reserves---Followup---One-Hi.patch (text/plain), 1.27 KB, created by Kyle M Hall (khall) on 2012-08-28 12:12:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2012-08-28 12:12:11 UTC

Size: 1.27 KB

patch

obsolete

>From 03f8b9691b0db25abf605475d4556624d824becd Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Tue, 28 Aug 2012 08:09:46 -0400
>Subject: [PATCH] Bug 8215 - Add Course Reserves - Followup - One Hit Searching Bugfix
>Content-Type: text/plain; charset="utf-8"
>
>When searching courses, a search that results in a single hit will
>automatically forward to the details for that course. If only a single
>course has been added to the system, Koha will always forward to that
>course and there will be no way to get to course-reserves.pl to add
>a second course!
>
>Fixed by testing for an actual search, rather than looking only at
>the number of hits returned.
>---
> course_reserves/course-reserves.pl |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/course_reserves/course-reserves.pl b/course_reserves/course-reserves.pl
>index 91d6e3e..8907460 100755
>--- a/course_reserves/course-reserves.pl
>+++ b/course_reserves/course-reserves.pl
>@@ -46,7 +46,7 @@ if ($search_on) {
> }
> 
> my $courses = GetCourses(%params);
>-if ( @$courses == 1 ) {
>+if ( $search_on && @$courses == 1 ) {
>     print $cgi->redirect("/cgi-bin/koha/course_reserves/course-details.pl?course_id=" . $courses->[0]->{'course_id'});
> } else {
>     $template->param( courses => $courses );
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 8215: 9990 | 10181 | 10224 | 11131 | 11612 | 11657 | 11677 | 11678 | 11679 | 11680 | 11681 | 11682 | 11683 | 11860 | 11861 | 11862 | 11865 | 11875 | 11881 | 11882 | 11980 | 12039 | 12120 | 12128 | 12188 | 12189 | 12192 | 12246 | 12361 | 12400 | 12413 | 12414 | 12593 | 12627 | 13496 | 13497 | 13498 | 13499 | 15195 | 15196 | 15197 | 15198 | 15199 | 16228 | 16229 | 16251 | 16252 | 16532 | 16533 | 16534 | 16536 | 16537 | 16545 | 16567 | 16568 | 16587 | 16588 | 16599 | 16621 | 16623 | 16634 | 17333 | 17334 | 17335 | 17336 | 17337 | 17377 | 17378 | 17379 | 17894 | 17895 | 17896 | 17897 | 17898 | 17899 | 18274 | 18275 | 18276 | 18277 | 18278 | 18279 | 18282 | 18283 | 18284 | 18285 | 18286 | 18287 | 18288 | 18289 | 18290 | 18291 | 18292 | 18293 | 21885 | 23264