From: Jason Etheridge <jason@esilibrary.com>
Date: Mon, 1 Oct 2012 20:49:04 +0000 (-0400)
Subject: LP#1010027: tweak patron columns in patron search interface
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b844a3c1fbda7fc66193e5c3f400ba0b3e35d7de;p=evergreen%2Fpines.git

LP#1010027: tweak patron columns in patron search interface

specifically, remove the Mailing Address and Billing Address columns that come
off the actor table, since they are redundant with the Mailing Addr: Address ID
and Billing Addr: Address ID columns that come off the address table.

Also, alphabetize the columns coming directly off of the user table. The columns
coming off the library card and address tables remain unsorted; I believe their
order is as listed in the fm_IDL.xml file for their corresponding fieldmapper
definitions.

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jennifer Pringle <jpringle@sitka.bclibraries.ca>
Signed-off-by: Ben Shum <bshum@biblio.org>
---

diff --git a/Open-ILS/xul/staff_client/server/patron/search_result.js b/Open-ILS/xul/staff_client/server/patron/search_result.js
index 49c452228e..4e95573c0a 100644
--- a/Open-ILS/xul/staff_client/server/patron/search_result.js
+++ b/Open-ILS/xul/staff_client/server/patron/search_result.js
@@ -37,7 +37,7 @@ patron.search_result.prototype = {
 
         JSAN.use('patron.util');
         var columns = obj.list.fm_columns('au',{
-            '*' : { 'remove_virtual' : true, 'expanded_label' : false, 'hidden' : true },
+            '*' : { 'remove_virtual' : true, 'expanded_label' : false, 'hidden' : true, 'sort_headers' : true },
             'au_barcode' : { 'hidden' : false },
             'au_barred' : { 'hidden' : false },
             'au_family_name' : { 'hidden' : false },
@@ -47,6 +47,8 @@ patron.search_result.prototype = {
             'au_profile' : { 'fleshed_display_field' : 'name' },
             'au_ident_type' : { 'fleshed_display_field' : 'name' },
             'au_ident_type2' : { 'fleshed_display_field' : 'name' },
+            'au_mailing_address' : { 'remove_me' : true },
+            'au_billing_address' : { 'remove_me' : true },
             'au_net_access_level' : { 'fleshed_display_field' : 'name' }
         }).concat(
             obj.list.fm_columns('ac',{