From: Galen Charlton <gmc@equinoxinitiative.org>
Date: Thu, 27 Jul 2017 20:51:25 +0000 (+0000)
Subject: LP#1638299: improve display of scope notes in headings browse
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=17b6d9e2c58cd6f5da3b96ae6986e75438c8126f;p=working%2FEvergreen.git

LP#1638299: improve display of scope notes in headings browse

Scope notes are now displayed only under uses of a heading as a main
heading.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
---

diff --git a/Open-ILS/src/templates/opac/browse.tt2 b/Open-ILS/src/templates/opac/browse.tt2
index ed946094a9..4e7424185e 100644
--- a/Open-ILS/src/templates/opac/browse.tt2
+++ b/Open-ILS/src/templates/opac/browse.tt2
@@ -99,7 +99,6 @@
                             </p>
                     </div>
                     [% END %]
-
                     <ul class="browse-result-list">
                     [% FOR result IN ctx.browse_results %]
                         <li class="browse-result">
@@ -119,14 +118,13 @@
                                 [% END %]
                             </span>
 
+                            [% FOR a IN result.authorities;
+                                    PROCESS authority_notes authority=a IF !sees.grep(a.id);
+                               END %]
                             [% auth_headings_ul_added = 0; %]
                                 [% seenit = {}; # for headings we've rendered
                                    FOR a IN result.sees;
-                                    PROCESS authority_notes authority=a;
-
-                                    # Other than displaying public general
-                                    # notes, we can go no further sans
-                                    # control_set.
+                                    # We can go no further sans control_set.
                                     NEXT UNLESS a.control_set;
 
                                     # get_authority_fields is fast and cache-y.
@@ -158,12 +156,8 @@
                                         [% END %]
                                     [% END %]
                                 [% END %]
-                                [% FOR a IN result.authorities; # I don't think we'll ever get here...
-                                    PROCESS authority_notes authority=a IF !sees.grep(a.id);
-
-                                    # Other than displaying public general
-                                    # notes, we can go no further sans
-                                    # control_set.
+                                [% FOR a IN result.authorities;
+                                    # We can go no further sans control_set.
                                     NEXT UNLESS a.control_set;
 
                                     # get_authority_fields is fast and cache-y.