LP#1741997: related headings in the browse list now hyperlink to heading, not bib... user/gmcharlt/lp1741997_browse_fixes
authorGalen Charlton <gmc@equinoxinitiative.org>
Thu, 28 Sep 2017 18:55:11 +0000 (18:55 +0000)
committerGalen Charlton <gmc@equinoxinitiative.org>
Mon, 8 Jan 2018 22:47:33 +0000 (17:47 -0500)
To test
-------
[1] Perform a browse search set up to include a related
    heading, e.g., a broader or narrower term.
[2] Click on the hyperlinked heading; note that it returns
    the bibs associated with that heading.
[3] Apply the patch and repeat step #2; this time, clicking
    the link should move you to the relevant heading in the
    browse list.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Open-ILS/src/templates/opac/browse.tt2

index ea27b7d..3337a1c 100644 (file)
                                                 <ul class="browse-result-authority-headings"> 
                                                 [% END %]
                                                 <li><span class="browse-result-authority-field-name">[% heading_use_label(use=h.type) %]</span>
-                                                <a href="[% mkurl(ctx.opac_root _ '/results', {query => 'identifier|authority_id[' _ h.target _ ']'}) %]">[% h.heading | html %]</a>
+                                                <a href="[% mkurl(ctx.opac_root _ '/browse', {bterm => h.heading}, ['bpivot']) %]">[% h.heading | html %]</a>
                                                 <span class="browse-result-authority-bib-links">([% h.target_count %])</span>
                                                 </li>
                                             [% END %]