[open-ils-commits] [GIT] Evergreen ILS branch master updated. 17b8f4abb1d922522ffadc440f4b6e3660718874

Evergreen Git git at git.evergreen-ils.org
Tue Apr 10 13:41:21 EDT 2012


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, master has been updated
       via  17b8f4abb1d922522ffadc440f4b6e3660718874 (commit)
      from  ffafabde6a24eb05527e0b5ffadf381ad9399ee9 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 17b8f4abb1d922522ffadc440f4b6e3660718874
Author: Jason Etheridge <jason at esilibrary.com>
Date:   Tue Apr 10 13:15:09 2012 -0400

    fix sort bug introduced with multi-sort
    
    The bug happens on columns without a defined sort_value
    
    Signed-off-by: Jason Etheridge <jason at esilibrary.com>
    Signed-off-by: Thomas Berezansky <tsbere at mvlc.org>

diff --git a/Open-ILS/xul/staff_client/chrome/content/util/list.js b/Open-ILS/xul/staff_client/chrome/content/util/list.js
index b35c289..61d4853 100644
--- a/Open-ILS/xul/staff_client/chrome/content/util/list.js
+++ b/Open-ILS/xul/staff_client/chrome/content/util/list.js
@@ -1749,8 +1749,8 @@ util.list.prototype = {
                                 } else {
                                     a = a.value;
                                     b = b.value;
-                                    if (obj.columns[p].getAttribute('sort_type')) {
-                                        switch(obj.columns[p].getAttribute('sort_type')) {
+                                    if (obj.columns[p].sort_type) {
+                                        switch(obj.columns[p].sort_type) {
                                             case 'date' :
                                                 JSAN.use('util.date'); // to pull in dojo.date.locale
                                                 a = dojo.date.locale.parse(a,{});

-----------------------------------------------------------------------

Summary of changes:
 .../xul/staff_client/chrome/content/util/list.js   |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS


More information about the open-ils-commits mailing list