Attachment 5209 Details for Bug 6656 – [SIGNED-OFF] Bug 6656 : Fix for advanced search not respecting default sort order

[patch] [SIGNED-OFF] Bug 6656 : Fix for advanced search not respecting default sort order

0001-SIGNED-OFF-Bug-6656-Fix-for-advanced-search-not-resp.patch (text/plain), 1.95 KB, created by Nicole C. Engard on 2011-08-29 13:11:10 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicole C. Engard

Created: 2011-08-29 13:11:10 UTC

Size: 1.95 KB

patch

obsolete

>From 06d876ea908b1933534eabbeb344070cb914a6f5 Mon Sep 17 00:00:00 2001
>From: Chris Cormack <chrisc@catalyst.net.nz>
>Date: Mon, 29 Aug 2011 15:35:45 +1200
>Subject: [PATCH] [SIGNED-OFF] Bug 6656 : Fix for advanced search not respecting default sort order
>
>Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
>---
> catalogue/search.pl |    2 +-
> opac/opac-search.pl |    3 +--
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
>diff --git a/catalogue/search.pl b/catalogue/search.pl
>index e49507e..b9f9ed3 100755
>--- a/catalogue/search.pl
>+++ b/catalogue/search.pl
>@@ -294,7 +294,7 @@ if ( $template_type eq 'advsearch' ) {
>         my $default_sort_by =
>             C4::Context->preference('defaultSortField') . '_'
>           . C4::Context->preference('defaultSortOrder');
>-        $template->param( $default_sort_by => 1 );
>+        $template->param( sort_by => $default_sort_by  );
>     }
> 
>     # determine what to display next to the search boxes (ie, boolean option
>diff --git a/opac/opac-search.pl b/opac/opac-search.pl
>index 81b7ce6..8f6b67a 100755
>--- a/opac/opac-search.pl
>+++ b/opac/opac-search.pl
>@@ -198,7 +198,6 @@ $template->param(itemtypeloop => \@itemtypesloop);
> 
> # The following should only be loaded if we're bringing up the advanced search template
> if ( $template_type && $template_type eq 'advsearch' ) {
>-
>     # load the servers (used for searching -- to do federated searching, etc.)
>     my $primary_servers_loop;# = displayPrimaryServers();
>     $template->param(outer_servers_loop =>  $primary_servers_loop,);
>@@ -212,7 +211,7 @@ if ( $template_type && $template_type eq 'advsearch' ) {
>         my $default_sort_by =
>             C4::Context->preference('OPACdefaultSortField') . '_'
>           . C4::Context->preference('OPACdefaultSortOrder');
>-        $template->param( $default_sort_by => 1 );
>+        $template->param( sort_by => $default_sort_by );
>     }
> 
>     # determine what to display next to the search boxes (ie, boolean option
>-- 
>1.7.2.3
>

Actions: View | Diff | Splinter Review

Attachments on bug 6656: 5199 | 5208 | 5209 | 5301