From: Bill Erickson <berick@esilibrary.com>
Date: Wed, 1 Feb 2012 18:53:55 +0000 (-0500)
Subject: Paging repairs for Event definition admin UI
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=1127be90412dc57c7e318286a4f1e9312417478f;p=evergreen%2Fmasslnc.git

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 <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
---

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';