From 499bb94087c67f0a9a123c9c756e847cc6818bfd Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Mon, 31 Aug 2015 09:36:02 -0400
Subject: [PATCH] Bug 14760: Disabled courses display in the course reserves
 list for items
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

If an item is on reserve for two courses but one of those courses is
disabled, both courses are still listed on opac-detail.pl!

Test Plan:
1) Enable course reserves
2) Create two courses
3) Place one item on reserve for both courses
4) Disable one of the two courses
5) View the record details for that record/item
6) You should see both courses listed in the course reserves column
7) Apply this patch
8) Reload the page
9) You should now only see the active course in the course reseves column

Followed test plan. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 73e9dcb70fcf24dc87af7c5af6f38102206863ae)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
(cherry picked from commit 9c31f1af9bb7b87dd794f40dd2bc7673b720c674)
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
---
 .../bootstrap/en/modules/opac-detail.tt        | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
index 536dc973f2..0a169ee408 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
@@ -1226,14 +1226,16 @@
                         <td>
                             [% IF ITEM_RESULT.course_reserves %]
                                 [% FOREACH r IN ITEM_RESULT.course_reserves %]
-                                    <p>
-                                        <a href="opac-course-details.pl?course_id=[% r.course.course_id %]">
-                                        [% r.course.course_name %]
-                                        <!--[% IF r.course.course_number %] [% r.course.course_number %] [% END %]-->
-                                        [% IF r.course.section %] [% r.course.section %] [% END %]
-                                        [% IF r.course.term %] [% AuthorisedValues.GetByCode( 'TERM', r.course.term ) %] [% END %]
-                                        </a>
-                                    </p>
+                                    [% IF r.course.enabled == 'yes' %]
+                                        <p>
+                                            <a href="opac-course-details.pl?course_id=[% r.course.course_id %]">
+                                            [% r.course.course_name %]
+                                            <!--[% IF r.course.course_number %] [% r.course.course_number %] [% END %]-->
+                                            [% IF r.course.section %] [% r.course.section %] [% END %]
+                                            [% IF r.course.term %] [% AuthorisedValues.GetByCode( 'TERM', r.course.term ) %] [% END %]
+                                            </a>
+                                        </p>
+                                    [% END %]
                                 [% END %]
                             [% END %]
                         </td>
-- 
2.39.5