From 1127be90412dc57c7e318286a4f1e9312417478f Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Wed, 1 Feb 2012 13:53:55 -0500 Subject: [PATCH] Paging repairs for Event definition admin UI If there are multiple event definitions with the same name that lie along the page boundaries, it's possible to page through event definitions seeing some def's multiple times and others never. This is a result of sorting solely on event def. name when fetching event def's for display. This patch adds secondary (and tertiary, and quaternary, and quinary) sort columns as tie breakers to guarantee a unique set of defs across pages. Signed-off-by: Bill Erickson Signed-off-by: Mike Rylander --- .../ui/default/conify/global/action_trigger/event_definition.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/web/js/ui/default/conify/global/action_trigger/event_definition.js b/Open-ILS/web/js/ui/default/conify/global/action_trigger/event_definition.js index 1eba4a653a..2867bb14ea 100644 --- a/Open-ILS/web/js/ui/default/conify/global/action_trigger/event_definition.js +++ b/Open-ILS/web/js/ui/default/conify/global/action_trigger/event_definition.js @@ -17,7 +17,7 @@ function loadEventDef() { eventDefGranularity.attr('value', null); edGrid.overrideEditWidgets.granularity = eventDefGranularity; edGrid.overrideEditWidgets.granularity.shove = {"create": ""}; - edGrid.loadAll({order_by:{atevdef : 'name'}}); + edGrid.loadAll({order_by:{atevdef : 'name, owner, hook, reactor, delay'}}); openils.widget.Textarea.width = '600px'; openils.widget.Textarea.height = '600px'; edGrid.overrideEditWidgetClass.template = 'openils.widget.Textarea'; -- 2.43.2