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>

diff --git a/Open-ILS/src/templates/opac/course/results.tt2 b/Open-ILS/src/templates/opac/course/results.tt2
index 41966bf..14a350c 100644 (file)
--- 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 2d80caf..e80cea0 100644 (file)
--- 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);