From cb91cf587da0b2bd3767c66af756b984903dbe73 Mon Sep 17 00:00:00 2001
From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Thu, 23 May 2013 14:44:42 -0400
Subject: [PATCH] OPAC Browse: Show authority tracings only when
 inter-authority links exist

This is something of a reversion to an earlier behavior.  Now we don't
display See Also tracings from an authority records linked to a bib
heading unless that authority record tracing is itself linked to another
authority record where the same tracing is a main entry.  Without that
authority-to-authority link, we can't say whether any bibs are
authority-linked to the "see also" term.

Unlike before, the tracings are available to the template even if a<->a
links aren't there, so a site could make a customization if they
preferred to see all tracings, linked or not.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---
 Open-ILS/src/templates/opac/browse.tt2 | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/Open-ILS/src/templates/opac/browse.tt2 b/Open-ILS/src/templates/opac/browse.tt2
index ac3a25de59..7797444826 100644
--- a/Open-ILS/src/templates/opac/browse.tt2
+++ b/Open-ILS/src/templates/opac/browse.tt2
@@ -108,16 +108,18 @@
                                         field = acs.$field_id;
                                         headings = field_group.values.0;
                                         FOR h IN headings;
-                                            target_auth_id = h.target %]
+                                            target_auth_id = h.target;
 
+                                            # We could display headings without
+                                            # links here when target_auth_id is
+                                            # undef, if we wanted to.
+
+                                            IF target_auth_id %]
                                             <li><span class="browse-result-authority-field-name">[% field.name %]</span>
-                                            [% IF target_auth_id %]
                                             <a href="[% mkurl(ctx.opac_root _ '/results', {query => 'identifier|authority_id[' _ target_auth_id _ ']'}) %]">[% h.heading | html %]</a>
                                             <span class="browse-result-authority-bib-links">([% ctx.authority_counts.$target_auth_id %])</span>
-                                            [% ELSE;
-                                                h.heading | html;
-                                            END %]
                                             </li>
+                                            [% END %]
                                         [% END %]
                                     [% END %]
                                 [% END %]
-- 
2.11.0