From 9657f893644414060631c76938fa1907642283ce Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Thu, 23 Oct 2014 12:03:31 -0400 Subject: [PATCH] JBAS-234 sort patron event rows descending Due to how the event rows are inserted, the server side "desc" sort becomes in effect an "asc" sort. To achieve a final sort of descending, this patch makes the server-side sort ascending. Signed-off-by: Bill Erickson --- Open-ILS/xul/staff_client/server/patron/trigger_events.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Open-ILS/xul/staff_client/server/patron/trigger_events.js b/Open-ILS/xul/staff_client/server/patron/trigger_events.js index 29c3fa21bf..c2cdd94954 100644 --- a/Open-ILS/xul/staff_client/server/patron/trigger_events.js +++ b/Open-ILS/xul/staff_client/server/patron/trigger_events.js @@ -210,7 +210,9 @@ function populate_list() { var method = $('circ').checked ? 'FM_ATEV_APROPOS_CIRC' : 'FM_ATEV_APROPOS_AHR'; if (xul_param('copy_id')) { method += '_VIA_COPY'; } - var filter = {"event":{"state":"complete"}, "order_by":[{"class":"atev", "field":"run_time", "direction":"desc"}]}; + // Sort completed events "ascending". The grid inserts the rows + // bottom to top, so this results in a final sort of descending. + var filter = {"event":{"state":"complete"}, "order_by":[{"class":"atev", "field":"run_time", "direction":"asc"}]}; if ($('pending').checked) { filter.event.state = 'pending'; filter.order_by[0].direction = 'asc'; } if ($('error').checked) { filter.event.state = 'error'; } -- 2.11.0