From b43ef34f3b6a3d5a33629eafece797f1ab108314 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Mon, 6 Jun 2016 11:28:39 -0400
Subject: [PATCH] Bug 16651: Notes field blank for 952$z in
 opac-course-details.pl
Content-Type: text/plain; charset=utf-8

This patch changes course reserves to check for item notes form the
course reserve and fallback to itemlevel notes if they are empty

To test:
1 - Enabvle course reserves
2 - Add some items
3 - Make sure the items have notes at the item level and not at course
reserves
4 - Notes don't display in staff or opac
5 - Apply patch
6 - Notes display in staff and opac
7 - Add notes at course reserves level
8 - These override the item level notes

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as advertised

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
---
 .../prog/en/modules/course_reserves/course-details.tt    |   14 ++++++++++++--
 .../bootstrap/en/modules/opac-course-details.tt          |    7 ++++++-
 2 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
index d90c32f..0bd675b 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-details.tt
@@ -151,8 +151,18 @@
                                     [% END %]
                                 [% END %]
                             </td>
-                            <td>[% cr.staff_note %]</td>
-                            <td>[% cr.public_note %]</td>
+                            <td>[% IF (cr.staff_note) %]
+                                    [% cr.staff_note %]
+                                [% ELSIF (cr.item.itemnotes_nonpublic) %]
+                                    [% cr.item.itemnotes_nonpublic %]
+                                [% END %]
+                            </td>
+                            <td>[% IF (cr.public_note) %]
+                                    [% cr.public_note %]
+                                [% ELSIF (cr.item.itemnotes) %]
+                                    [% cr.item.itemnotes %]
+                                [% END %]
+                            </td>
 
                             [% IF CAN_user_coursereserves_add_reserves %]
                                 <td><a href="add_items.pl?course_id=[% course.course_id %]&amp;barcode=[% cr.item.barcode %]&amp;action=lookup&amp;return=[% course.course_id %]">Edit</a></td>
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-course-details.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-course-details.tt
index 3241c3c..70d45b1 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-course-details.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-course-details.tt
@@ -72,7 +72,12 @@
                                     <td>[% cr.item.copynumber %]</td>
                                     <td>[% INCLUDE 'item-status.inc' item=cr.item issue=cr.issue %]</td>
                                     <td><span title="[% cr.issue.date_due %]">[% cr.issue.date_due | $KohaDates as_due_date => 1 %]</span></td>
-                                    <td>[% cr.public_note %]</td>
+                                    <td>[% IF ( cr.public_note ) %]
+                                            [% cr.public_note %]
+                                        [% ELSIF ( cr.item.itemnotes ) %]
+                                            [% cr.item.itemnotes %]
+                                        [% END %]
+                                    </td>
                                 </tr>
                             [% END %]
                         </tbody>
-- 
1.7.10.4