Bug 30055: (QA follow-up) Minor QA fixes

[srvgit] / koha-tmpl / intranet-tmpl / prog / en / modules / members / search.tt

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/search.tt
index d74f044..7422381 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/search.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/search.tt
@@ -76,7 +76,7 @@
         </div>
 
         <div id="info" class="dialog message" style="display: none;"></div>
-        <div id="error" class="dialog alert" style="dispay: none;"></div>
+        <div id="error" class="dialog alert" style="display: none;"></div>
 
         <input type="hidden" id="firstletter_filter" value="" />
         <div id="searchresults" style="display:none;">
@@ -171,7 +171,7 @@
                     [% SET search_fields = Koha.Preference('DefaultPatronSearchFields') || 'surname,firstname,othernames,cardnumber,userid' %]
                     return [
                         [% FOR search_field IN search_fields.split(',') %]
-                        {"me.[% search_field %]":{"like":"%"+filter+"%"}},
+                        {"me.[% search_field | html %]":{"like":"%"+filter+"%"}},
                         [% END %]
                         [% IF Koha.Preference('ExtendedPatronAttributes') %]
                         {
@@ -182,6 +182,8 @@
                     ];
                 },
             };
+            [% SET default_sort_column = "name" %]
+            [% SET order_column_index = 0 %]
             patrons_table = $("#memberresultst").kohaTable({
                 "ajax": {
                     [% SWITCH filter %]
@@ -198,10 +200,12 @@
                     [% END %]
                 },
                 embed: ['extended_attributes'],
-                "order": [[ 1, "asc" ]],
                 "iDeferLoading": 0,
                 "columns": [
                     [% FOR column IN columns %]
+                        [% IF default_sort_column == column %]
+                            [% order_column_index = loop.count - 1%]
+                        [% END %]
                         [% SWITCH column %]
                             [% CASE 'cardnumber' %]
                             {
@@ -289,6 +293,7 @@
                         [% UNLESS loop.last %],[% END %]
                     [% END %]
                 ],
+                "order": [[ [% order_column_index | html %], "asc" ]],
                 'bAutoWidth': false,
                 'sPaginationType': 'full_numbers',
                 "iDisplayLength": [% Koha.Preference('PatronsPerPage') | html %],
@@ -339,11 +344,11 @@
                 [% CASE 'branch' %]
                     let library_id = $("#branchcode_filter").val();
                     patrons_table.find('thead tr:eq(1) th[data-filter="libraries"] select').val(library_id);
-                    table_dt.column([% loop.count - 1 %]).search(library_id);
+                    table_dt.column([% loop.count - 1 %]).search(library_id ? '^'+library_id+'$' : '');
                 [% CASE 'category' %]
                     let category_id = $("#categorycode_filter").val();
                     patrons_table.find('thead tr:eq(1) th[data-filter="categories"] select').val(category_id);
-                    table_dt.column([% loop.count - 1 %]).search(category_id);
+                    table_dt.column([% loop.count - 1 %]).search(category_id ? '^'+category_id+'$' : '');
                 [% END %]
             [% END %]
             table_dt.search("");