From d0b8b0d2fb12e9f1dc5e7092de4c974bba0e2b54 Mon Sep 17 00:00:00 2001
From: Jane Sandberg <sandbej@linnbenton.edu>
Date: Mon, 1 Feb 2021 18:04:02 -0800
Subject: [PATCH] LP1913221: Don't display blank course section number in OPAC

To test:

1) Use the Library Settings Editor to turn on the Course Materials
   Module.
2) In Local administration, go to the course reserves list.
3) Add several courses, some with section numbers, others without.
4) In the OPAC, use the Course Search function to search for the courses
   you just added.
5) Notice that the search results page has a place for section numbers,
   whether or not the course has one.
6) Click into some of the courses.  Notice that the course page also
   includes a place for section numbers, whether or not the course has
   one.
7) Apply this patch.
8) Repeat steps 4-6.  Notice that the blank "section number" areas no
   longer appear.

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
---
 Open-ILS/src/templates/opac/course/results.tt2    |  8 +++++---
 Open-ILS/src/templates/opac/parts/course/body.tt2 | 10 ++++++----
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/Open-ILS/src/templates/opac/course/results.tt2 b/Open-ILS/src/templates/opac/course/results.tt2
index 41966bf91d..14a350c02e 100644
--- a/Open-ILS/src/templates/opac/course/results.tt2
+++ b/Open-ILS/src/templates/opac/course/results.tt2
@@ -92,9 +92,11 @@
                 <div>
                   <span><strong>[% l('Course Number') %]</strong>: [% course.course_number %]</span>
                 </div>
-                <div>
-                  <span><strong>[% l('Section Number') %]</strong>: [% course.section_number %]</span>
-                </div>
+                [% IF course.section_number %]
+                  <div>
+                    <span><strong>[% l('Section Number') %]</strong>: [% course.section_number %]</span>
+                  </div>
+                [% END %]
               </div>
             </td>
             <td>
diff --git a/Open-ILS/src/templates/opac/parts/course/body.tt2 b/Open-ILS/src/templates/opac/parts/course/body.tt2
index 2d80caff69..e80cea059a 100644
--- a/Open-ILS/src/templates/opac/parts/course/body.tt2
+++ b/Open-ILS/src/templates/opac/parts/course/body.tt2
@@ -39,11 +39,13 @@
         <div>
           <span><strong>[% l('Course Number') %]: </strong></span>
           <span>[% ctx.course.course_number %]</span>
-        <div>
-        </div>
-          <span><strong>[% l('Section Number') %]: </strong></span>
-          <span>[% ctx.course.section_number %]</span>
         </div>
+        [% IF ctx.course.section_number %]
+          <div>
+            <span><strong>[% l('Section Number') %]: </strong></span>
+            <span>[% ctx.course.section_number %]</span>
+          </div>
+        [% END %]
         <div>
           [%
             owning_lib = ctx.get_aou(ctx.course.owning_lib);
-- 
2.11.0