Bugzilla – Attachment 20039 Details for
Bug 10656
shelving locations not alpha on search
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10656 - shelving locations not alpha on search
Bug-10656---shelving-locations-not-alpha-on-search.patch (text/plain), 1.62 KB, created by
Owen Leonard
on 2013-08-01 13:22:38 UTC
(
hide
)
Description:
Bug 10656 - shelving locations not alpha on search
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2013-08-01 13:22:38 UTC
Size:
1.62 KB
patch
obsolete
>From 85d0c0ded028a8e461d2852de982b6729912378f Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Thu, 1 Aug 2013 09:14:08 -0400 >Subject: [PATCH] Bug 10656 - shelving locations not alpha on search >Content-Type: text/plain; charset="utf-8" > >Collection codes and shelving locations are displayed in the OPAC and >staff client via GetAuthorisedValues which currently sorts results by >"lib, lib_opac." Consequently if lib (the description for the staff >client) doesn't match lib_opac (the description for the OPAC) sorting >will appear to be nonsensical in the OPAC. GetAuthorisedValues can be >passed an $opac parameter, so this should be used to switch how reuslts >are sorted. This patch implements such a switch. > >To test, modify your collection code or shelving location authorized >values so that lib and lib_opac do not match. Set your >AdvancedSearchTypes system preference to display the modified authorized >values and view the advanced search page in the OPAC and staff client. >Sorting should be correct in each case according to the correct value >(lib in the staff client, lib_opac in the OPAC). >--- > C4/Koha.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/C4/Koha.pm b/C4/Koha.pm >index ea8c2a6..1317006 100644 >--- a/C4/Koha.pm >+++ b/C4/Koha.pm >@@ -1076,7 +1076,7 @@ sub GetAuthorisedValues { > if(@where_strings > 0) { > $query .= " WHERE " . join(" AND ", @where_strings); > } >- $query .= " GROUP BY lib ORDER BY category, lib, lib_opac"; >+ $query .= " GROUP BY lib ORDER BY category, " . ($opac ? "lib_opac, lib" : "lib, lib_opac"); > > my $sth = $dbh->prepare($query); > >-- >1.7.9.5
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 10656
:
19978
|
19979
|
20039
|
20040
|
20502
|
20507
|
20508
|
20509
|
20512
|
20513
|
20514
|
20825
|
20826
|
20827