From f632be948ab419d9ceb6157542736ba78085a081 Mon Sep 17 00:00:00 2001 From: Jason Etheridge Date: Wed, 20 Jul 2011 01:29:43 -0400 Subject: [PATCH] adjust legacy xul column definitions for dates/timestamps to ensure correct sorting Signed-off-by: Jason Etheridge Signed-off-by: Mike Rylander --- Open-ILS/xul/staff_client/server/circ/util.js | 36 ++++++++++++++++++++++++- Open-ILS/xul/staff_client/server/patron/util.js | 11 ++++++++ 2 files changed, 46 insertions(+), 1 deletion(-) diff --git a/Open-ILS/xul/staff_client/server/circ/util.js b/Open-ILS/xul/staff_client/server/circ/util.js index 43bf53150c..c85d77e4b8 100644 --- a/Open-ILS/xul/staff_client/server/circ/util.js +++ b/Open-ILS/xul/staff_client/server/circ/util.js @@ -887,6 +887,7 @@ circ.util.columns = function(modify,params) { 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.acp.status_changed_time(), '%{localized}' ); }, 'persist' : 'hidden width ordinal' + ,'sort_value' : function(my) { return util.date.db_date2Date( my.acp.status_changed_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -920,6 +921,7 @@ circ.util.columns = function(modify,params) { return ""; } } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.circ.xact_start() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -937,6 +939,7 @@ circ.util.columns = function(modify,params) { return ""; } } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.circ.checkin_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -948,6 +951,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.circ ? util.date.formatted_date( my.circ.xact_finish(), '%{localized}' ) : ""; }, + 'sort_value' : function(my) { return util.date.db_date2Date( my.circ.xact_finish() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -965,6 +969,7 @@ circ.util.columns = function(modify,params) { return ""; } } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.circ.due_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -976,7 +981,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.acp.create_date(), '%{localized}' ); } - ,'sort_value' : function(my) { return util.date.db_date2Date( my.acp.create_date() ).getTime();; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.acp.create_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -988,6 +993,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.acp.edit_date(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.acp.edit_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1139,6 +1145,7 @@ circ.util.columns = function(modify,params) { return ""; } } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.circ.stop_fines_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1234,6 +1241,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.circ ? util.date.formatted_date( my.circ.checkin_scan_time(), '%{localized}' ) : ""; }, + 'sort_value' : function(my) { return util.date.db_date2Date( my.circ.checkin_scan_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1275,6 +1283,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.bre ? util.date.formatted_date( my.bre.create_date(), '%{localized}' ) : ''; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.bre.create_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1286,6 +1295,7 @@ circ.util.columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.bre ? util.date.formatted_date( my.bre.edit_date(), '%{localized}' ) : ''; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.bre.edit_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1475,6 +1485,7 @@ circ.util.transit_columns = function(modify,params) { 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.atc.source_send_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.atc.source_send_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1500,6 +1511,7 @@ circ.util.transit_columns = function(modify,params) { 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.atc.dest_recv_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.atc.dest_recv_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1564,6 +1576,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.cancel_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.cancel_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1622,6 +1635,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.request_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.request_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1632,6 +1646,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.shelf_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.shelf_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1642,6 +1657,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.shelf_expire_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.shelf_expire_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1660,6 +1676,16 @@ circ.util.hold_columns = function(modify,params) { } return ""; } + ,'sort_value' : function(my) { + return util.date.db_date2Date( my.acp.create_date() ).getTime(); + if (my.ahr.transit() && my.ahr.transit().dest_recv_time()) { + return util.date.db_date2Date( my.ahr.transit().dest_recv_time() ).getTime(); + } + if (!my.ahr.transit() && my.ahr.capture_time()) { + return util.date.db_date2Date( my.ahr.capture_time() ).getTime(); + } + return 0; + } }, { 'persist' : 'hidden width ordinal', @@ -1670,6 +1696,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.ahr.capture_time() ? util.date.formatted_date( my.ahr.capture_time(), '%{localized}' ) : ""; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.capture_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1758,6 +1785,7 @@ circ.util.hold_columns = function(modify,params) { return util.date.formatted_date( my.ahr.thaw_date(), '%{localized}' ); } } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.thaw_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1839,6 +1867,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.ahr.expire_time() ? util.date.formatted_date( my.ahr.expire_time(), '%{localized}' ) : ''; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.expire_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1849,6 +1878,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.fulfillment_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.fulfillment_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -1895,6 +1925,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.prev_check_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.prev_check_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -2026,6 +2057,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.ahr.notify_time(), '%{localized}' ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.notify_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -2060,6 +2092,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.ahr.transit() ? util.date.formatted_date( my.ahr.transit().source_send_time(), '%{localized}' ) : ""; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.transit().source_send_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', @@ -2079,6 +2112,7 @@ circ.util.hold_columns = function(modify,params) { 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return my.ahr.transit() ? util.date.formatted_date( my.ahr.transit().dest_recv_time(), '%{localized}' ) : ""; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ahr.transit().dest_recv_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', diff --git a/Open-ILS/xul/staff_client/server/patron/util.js b/Open-ILS/xul/staff_client/server/patron/util.js index 0f235efb45..cf1a59cd9d 100644 --- a/Open-ILS/xul/staff_client/server/patron/util.js +++ b/Open-ILS/xul/staff_client/server/patron/util.js @@ -57,6 +57,7 @@ patron.util.mbts_columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'last_billing_ts', 'label' : commonStrings.getString('staff.mbts_last_billing_timestamp_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.mbts.last_billing_ts(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mbts.last_billing_ts() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'last_payment_note', 'label' : commonStrings.getString('staff.mbts_last_payment_note_label'), 'flex' : 2, @@ -70,16 +71,19 @@ patron.util.mbts_columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'last_payment_ts', 'label' : commonStrings.getString('staff.mbts_last_payment_timestamp_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.mbts.last_payment_ts(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mbts.last_payment_ts() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'mbts_xact_start', 'label' : commonStrings.getString('staff.mbts_xact_start_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return my.mbts.xact_start() ? util.date.formatted_date( my.mbts.xact_start(), "%{localized}" ) : ""; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mbts.xact_start() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'mbts_xact_finish', 'label' : commonStrings.getString('staff.mbts_xact_finish_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return my.mbts.xact_finish() ? util.date.formatted_date( my.mbts.xact_finish(), "%{localized}" ) : ""; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mbts.xact_finish() ).getTime(); } }, ]; for (var i = 0; i < c.length; i++) { @@ -136,6 +140,7 @@ patron.util.mb_columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'void_time', 'label' : commonStrings.getString('staff.mb_void_time_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.mb.void_time(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mb.void_time() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'amount', 'label' : commonStrings.getString('staff.mb_amount_label'), 'flex' : 1, @@ -150,6 +155,7 @@ patron.util.mb_columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'billing_ts', 'label' : commonStrings.getString('staff.mb_billing_ts_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.mb.billing_ts(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mb.billing_ts() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'note', 'label' : commonStrings.getString('staff.mb_note_label'), 'flex' : 2, @@ -216,6 +222,7 @@ patron.util.mp_columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'mp_payment_ts', 'label' : commonStrings.getString('staff.mp_payment_timestamp_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.mp.payment_ts(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.mp.payment_ts() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'mp_note', 'label' : commonStrings.getString('staff.mp_note_label'), 'flex' : 2, @@ -327,6 +334,7 @@ patron.util.ausp_columns = function(modify,params) { 'primary' : false, 'hidden' : false, 'editable' : false, 'render' : function(my) { return my.ausp ? util.date.formatted_date( my.ausp.set_date(), "%{localized}" ) : ''; } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.ausp.set_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'ausp_note', 'label' : commonStrings.getString('staff.ausp_note_label'), 'flex' : 1, @@ -435,11 +443,13 @@ patron.util.columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'au_create_date', 'label' : commonStrings.getString('staff.au_create_date_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.au.create_date(), "%{localized}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.au.create_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'expire_date', 'label' : commonStrings.getString('staff.au_expire_date_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.au.expire_date(), "%{localized_date}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.au.expire_date() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'home_ou', 'label' : commonStrings.getString('staff.au_home_library_label'), 'flex' : 1, @@ -478,6 +488,7 @@ patron.util.columns = function(modify,params) { 'persist' : 'hidden width ordinal', 'id' : 'dob', 'label' : commonStrings.getString('staff.au_birth_date_label'), 'flex' : 1, 'sort_type' : 'date', 'primary' : false, 'hidden' : true, 'editable' : false, 'render' : function(my) { return util.date.formatted_date( my.au.dob(), "%{localized_date}" ); } + ,'sort_value' : function(my) { return util.date.db_date2Date( my.au.dob() ).getTime(); } }, { 'persist' : 'hidden width ordinal', 'id' : 'ident_type', 'label' : commonStrings.getString('staff.au_ident_type_label'), 'flex' : 1, -- 2.11.0