From 73210058130eb5784f7aa2aed5c6c4f173a4b235 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 --- .../web/js/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.11.0