Attachment 21212 Details for Bug 10691 – Bug 10691: 5xx not properly linked by authid in authority search result list [Staff]

[patch] Bug 10691: 5xx not properly linked by authid in authority search result list [Staff]

Bug-10691-5xx-not-properly-linked-by-authid-in-aut.patch (text/plain), 8.04 KB, created by Tomás Cohen Arazi (tcohen) on 2013-09-18 15:20:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2013-09-18 15:20:38 UTC

Size: 8.04 KB

patch

obsolete

>From 36feccb2a2c7953df8f8977d662375a46e9d5239 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@gmail.com>
>Date: Wed, 18 Sep 2013 12:18:20 -0300
>Subject: [PATCH] Bug 10691: 5xx not properly linked by authid in authority
> search result list [Staff]
>
>This patch changes the URL and data used to show the 'see from' and 'see also' links
>on the Staff's authority search results page.
>
>Bonus points: makes some strings translatable.
>
>To test:
>- On your dev setup (master) create some authority records (i created personal name authorities).
>- Pick one of them and link 400$a to another one, do the same with 500$a
>- Make sure zebra is running and changes got indexed.
>- In the staff interface search for the authority that is linked to the others.
>- Check the 'see also:' link points to an authority search
>- Apply the patch
>- Reload/re-do the search
>- Check the 'see also:' link points to the authority id.
>- Check that the authority search from the cataloguing interface still works as usual.
>
>Regards
>To+
>
>Sponsored-by: Universidad Nacional de Cordoba
>---
> .../en/includes/authorities-search-results.inc     |  107 ++++++++++++--------
> .../en/modules/authorities/searchresultlist.tt     |    5 +-
> 2 files changed, 70 insertions(+), 42 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>index 366c695..19ab9cc 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>@@ -11,69 +11,93 @@
>             <span class="RT">RT: [% heading | html %]</span>
>         [% END %]
>     [% ELSE %]
>-        [% IF ( label ) %]<span class="label">[% label | html %]</span>[% END %]
>+        <span class="label">
>+        [% IF ( type=='seefrom' ) %]
>+            used for/see from:
>+        [% ELSIF  ( type=='seealso' ) %]
>+            see also:
>+        [% END %]
>+        </span>
>         <span class="heading">
>-        [% IF ( linkpath && search ) %]<a href="[% link | url %][% search | url %]">[% heading | html %]</a>
>-        [% ELSE %][% heading | html %][% END %]
>+        [% IF ( linkpath && search ) %]
>+            <a href="[% link | url %][% authid | url %]">[% heading | html %]</a>
>+        [% ELSE %]
>+            [% heading | html %]
>+        [% END %]
>         </span>
>-        [% UNLESS ( type=='seefrom' || type=='seealso' ) %]<span class="type">[% SWITCH type %]
>-        [% CASE 'earlier' %](Earlier heading)
>-        [% CASE 'later' %](Later heading)
>-        [% CASE 'acronym' %](Acronym)
>-        [% CASE 'musical' %](Musical composition)
>-        [% CASE 'broader' %](Broader heading)
>-        [% CASE 'narrower' %](Narrower heading)
>-        [% CASE 'parent' %](Immediate parent body)
>-        [% CASE %][% IF type %]([% type | html %])[% END %]
>-        [% END %]</span>[% END %]
>+        [% UNLESS ( type=='seefrom' || type=='seealso' ) %]
>+        <span class="type">
>+            [% SWITCH type %]
>+            [% CASE 'earlier' %](Earlier heading)
>+            [% CASE 'later' %](Later heading)
>+            [% CASE 'acronym' %](Acronym)
>+            [% CASE 'musical' %](Musical composition)
>+            [% CASE 'broader' %](Broader heading)
>+            [% CASE 'narrower' %](Narrower heading)
>+            [% CASE 'parent' %](Immediate parent body)
>+            [% CASE %][% IF type %]([% type | html %])
>+            [% END %]
>+        [% END %]
>+        </span>
>+        [% END %]
>     [% END %]
> [% END %]
> [% BLOCK authresult %]
>     [% IF ( summary.summary ) %][% summary.summary | html %]:[% END %]
>     [% UNLESS ( summary.summaryonly ) %]
>         <div class="authorizedheading">
>-        [% FOREACH authorize IN summary.authorized %]
>-          <span class="authorizedheading">[% authorize.heading | html %]</span>
>-        [% END %]
>+          [% FOREACH authorize IN summary.authorized %]
>+            <span class="authorizedheading">[% authorize.heading | html %]</span>
>+          [% END %]
>         </div>
>         [% IF ( marcflavour == 'UNIMARC' ) %]
>-          [% IF summary.notes %]
>-           <div class="authres_notes">
>-           [% FOREACH note IN summary.notes %]
>-             [% note.note | html %]</span>
>-           [% END %]
>-           </div>
>-          [% END %]
>-          [% IF summary.seealso %]
>-            <div class="authres_seealso">
>-            [% FOREACH see IN summary.seealso %]
>-              [% PROCESS showreference heading=see.heading label="" type=see.type search='' %]
>-              [% IF ! loop.last %] ; [% END %]
>+            [% IF summary.notes %]
>+             <div class="authres_notes">
>+             [% FOREACH note IN summary.notes %]
>+               [% note.note | html %]</span>
>+             [% END %]
>+             </div>
>             [% END %]
>-            </div>
>-          [% END %]
>-          [% IF summary.otherscript %]
>-            <div class="authres_otherscript">
>-            [% FOREACH other IN summary.otherscript %]
>-              [% PROCESS language lang=other.lang | trim %]:
>-              [% other.term %]
>-              [% IF ! loop.last %] ; [% END %]
>+            [% IF summary.seealso %]
>+              <div class="authres_seealso">
>+              [% FOREACH see IN summary.seealso %]
>+                 [% PROCESS showreference heading=see.heading label="" type=see.type search='' %]
>+                 [% IF ! loop.last %] ; [% END %]
>+              [% END %]
>+              </div>
>+            [% END %]
>+            [% IF summary.otherscript %]
>+              <div class="authres_otherscript">
>+              [% FOREACH other IN summary.otherscript %]
>+                [% PROCESS language lang=other.lang | trim %]:
>+                [% other.term %]
>+                [% IF ! loop.last %] ; [% END %]
>+              [% END %]
>+              </div>
>             [% END %]
>-            </div>
>-          [% END %]
>-
>         [% ELSE %]
>             [% IF ( summary.seefrom ) %]
>                 [% FOREACH seefro IN summary.seefrom %]
>                     <div class="seefrom authref">
>-                    [% PROCESS showreference heading=seefro.heading label="used for/see from:" type=seefro.type search='' %]
>+                    [% PROCESS showreference
>+                        heading=seefro.heading
>+                        type=seefro.type
>+                        search=''
>+                        authid=seefro.authid
>+                    %]
>                     </div>
>                 [% END %]
>             [% END %]
>             [% IF ( summary.seealso ) %]
>                 [% FOREACH seeals IN summary.seealso %]
>                     <div class="seealso authref">
>-                    [% PROCESS showreference heading=seeals.heading label="see also:" type=seeals.type linkpath=link search=seeals.search %]
>+                    [% PROCESS showreference
>+                        heading=seeals.heading
>+                        type=seeals.type
>+                        linkpath=link
>+                        search=seeals.search
>+                        authid=seeals.authid
>+                    %]
>                     </div>
>                 [% END %]
>             [% END %]
>@@ -90,3 +114,4 @@
>    [% CASE %][% lang %]
>   [% END %]
> [% END %]
>+
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>index 445b541..7103d3d 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>@@ -75,7 +75,10 @@ function searchauthority() {
>     [% ELSE %]
>     <tr>
>     [% END %]
>-      <td>[% PROCESS authresult summary=resul.summary link="/cgi-bin/koha/authorities/authorities-home.pl?op=do_search&type=intranet&marclist=any&operator=contains&orderby=HeadingAsc&value=" %]</td>
>+      <td>[% PROCESS authresult
>+                summary=resul.summary
>+                link="/cgi-bin/koha/authorities/detail.pl?authid="
>+          %]</td>
>       <td><a href="detail.pl?authid=[% resul.authid %]">Details</a></td>
>   [% UNLESS ( resul.isEDITORS ) %]
>       <td>
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 10691: 21210 | 21212 | 21716 | 21717 | 21727 | 21728 | 21893 | 21894 | 21895 | 22211 | 22212 | 22213 | 22214 | 22215 | 22785 | 22786 | 22787 | 23393 | 23394 | 23528 | 23529 | 23530 | 24232 | 24233 | 24234 | 24461 | 24462 | 24463