From c8d3dbc752deaf8e43024488feae18f067dc70c1 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 20 Mar 2015 14:07:03 +0100 Subject: [PATCH] [PASSED QA] 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 Without patch, I could not sort on names. With patch, sorting works as dexcribed in Test plan Signed-off-by: Marc Veron Signed-off-by: Kyle M Hall --- .../prog/en/modules/common/patron_search.tt | 14 ++++++++++++-- 1 files 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 %] -- 1.7.2.5