[open-ils-commits] [GIT] Evergreen ILS branch rel_2_2 updated. f8dfc40d33fafdbf10810fc320736e7100e7ae3e

Evergreen Git git at git.evergreen-ils.org
Tue Apr 10 13:41:22 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, rel_2_2 has been updated
       via  f8dfc40d33fafdbf10810fc320736e7100e7ae3e (commit)
      from  5c8d88ad61b434dc03bd912d6cd58ede3fc958e6 (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 f8dfc40d33fafdbf10810fc320736e7100e7ae3e
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