Bugzilla – Attachment 76026 Details for
Bug 18322
Add facets for ccode to zebra
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18322: (follow-up) Use authorized value labels
Bug-18322-follow-up-Use-authorized-value-labels.patch (text/plain), 1.44 KB, created by
Tomás Cohen Arazi (tcohen)
on 2018-06-13 13:15:05 UTC
(
hide
)
Description:
Bug 18322: (follow-up) Use authorized value labels
Filename:
MIME Type:
Creator:
Tomás Cohen Arazi (tcohen)
Created:
2018-06-13 13:15:05 UTC
Size:
1.44 KB
patch
obsolete
>From 9bbe0edc6cf99a163a8559cee46b12ae959a1be3 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Wed, 13 Jun 2018 10:29:54 +0100 >Subject: [PATCH] Bug 18322: (follow-up) Use authorized value labels > >Rather than using the raw ccode's use the labels as defined in the >authorized values. > >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> >--- > C4/Search.pm | 7 +++++++ > 1 file changed, 7 insertions(+) > >diff --git a/C4/Search.pm b/C4/Search.pm >index 10973e3aba..51d1078a23 100644 >--- a/C4/Search.pm >+++ b/C4/Search.pm >@@ -587,6 +587,13 @@ sub getRecords { > $facet_label_value = $av->count ? $av->next->opac_description : ''; > } > >+ # also, if it's a collection code, use the name instead of the code >+ if ( $link_value =~ /ccode/ ) { >+ # TODO Retrieve all authorised values at once, instead of 1 query per entry >+ my $av = Koha::AuthorisedValues->search({ category => 'CCODE', authorised_value => $one_facet }); >+ $facet_label_value = $av->count ? $av->next->opac_description : ''; >+ } >+ > # but we're down with the whole label being in the link's title. > push @this_facets_array, > { >-- >2.17.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 18322
:
61581
|
61597
|
61612
|
61643
|
61657
|
67823
|
75997
|
76007
|
76008
|
76009
|
76021
|
76023
|
76024
|
76025
| 76026