Attachment 37040 Details for Bug 13879 – Bug 13879: Fix sort on patron search (guarantor and linked to orders)

[patch] Bug 13879: Fix sort on patron search (guarantor and linked to orders)

Bug-13879-Fix-sort-on-patron-search-guarantor-and-.patch (text/plain), 2.25 KB, created by Jonathan Druart on 2015-03-20 13:17:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2015-03-20 13:17:27 UTC

Size: 2.25 KB

patch

obsolete

>From 4f92d50f59e8d480ec6931b0c3aa8df5fc1fbbfa Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Fri, 20 Mar 2015 14:07:03 +0100
>Subject: [PATCH] Bug 13879: Fix sort on patron search (guarantor and linked to
> orders)
>
>This patch fixes a regression introduced by bug 13021.
>
>Test plan:
>1/ Go on a child detail page > edit
>2/ Click on "Set a patron" as a guarantor
>3/ Launch a search a confirm that you are able to sort on card, name and
>date of birth.
>Note that the Address column is not sortable (it's a limitation, could
>be fixed later).
>4/ Go on the acquisition module, create an order.
>5/ Click on the "Add user" button.
>6/ Launch a search a confirm that you are able to sort on card, name and
>category
>---
> .../intranet-tmpl/prog/en/modules/common/patron_search.tt  | 14 ++++++++++++--
> 1 file changed, 12 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt
>index 3e93899..fd133e4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt
>@@ -32,8 +32,16 @@ $(document).ready(function(){
>             aoData.push({
>                 'name': 'searchmember',
>                 'value': $("#searchmember_filter").val()
>-            },
>-            {
>+            },{
>+                'name': 'name_sorton',
>+                'value': 'borrowers.surname borrowers.firstname'
>+            },{
>+                'name': 'category_sorton',
>+                'value': 'categories.description',
>+            },{
>+                'name': 'branch_sorton',
>+                'value': 'branches.branchname'
>+            },{
>                 'name': 'template_path',
>                 'value': '[% json_template %]',
>             }
>@@ -58,6 +66,8 @@ $(document).ready(function(){
>             [% FOR column IN columns %]
>                 [% IF column == 'action' %]
>                     { 'mDataProp': 'dt_action', 'bSortable': false }
>+                [% ELSIF column == 'address' %]
>+                    { 'mDataProp': 'dt_address', 'bSortable': false }
>                 [% ELSE %]
>                     { 'mDataProp': 'dt_[% column %]' }
>                 [% END %]
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 13879: 37040 | 37059 | 37072