Bug 13879: Fix sort on patron search (guarantor and linked to orders)
authorJonathan Druart <jonathan.druart@biblibre.com>
Fri, 20 Mar 2015 13:07:03 +0000 (14:07 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 8 Apr 2015 13:52:37 +0000 (10:52 -0300)
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 <veron@veron.ch>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/modules/common/patron_search.tt

index 3e93899..fd133e4 100644 (file)
@@ -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 %]