gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 14760: Disabled courses display in the course reserves list for items 

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 73e9dcb70f)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
(cherry picked from commit 9c31f1af9b)
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
This commit is contained in:
Kyle M Hall 2015-08-31 09:36:02 -04:00 committed by Liz Rea
parent 1b686528f7
commit bc72ef5cca
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
View File

@ -1291,14 +1291,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>