Bugzilla – Attachment 42121 Details for
Bug 14760
Disabled courses display in the course reserves list for items
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14760 - Disabled courses display in the course reserves list for items
Bug-14760---Disabled-courses-display-in-the-course.patch (text/plain), 2.88 KB, created by
Kyle M Hall (khall)
on 2015-08-31 13:38:38 UTC
(
hide
)
Description:
Bug 14760 - Disabled courses display in the course reserves list for items
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2015-08-31 13:38:38 UTC
Size:
2.88 KB
patch
obsolete
>From c352be41ea90bf311b1807e8bb13689b626a42cc 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 > >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 >--- > .../opac-tmpl/bootstrap/en/modules/opac-detail.tt | 18 ++++++++++-------- > 1 files 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 6799e23..e796c49 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt >@@ -1304,14 +1304,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> >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 14760
:
42121
|
42328
|
42347