From a7d44eb8c4ce0e33dcd5901c30a080e3f593c928 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>
Signed-off-by: Beth Willis <willis@noblenet.org>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
---
 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