Gracefully handle reporter class with no labels

[transitory.git] / Open-ILS / web / reports / xul / source-setup.js

diff --git a/Open-ILS/web/reports/xul/source-setup.js b/Open-ILS/web/reports/xul/source-setup.js
index 245657b..d3b919c 100644 (file)
--- a/Open-ILS/web/reports/xul/source-setup.js
+++ b/Open-ILS/web/reports/xul/source-setup.js
@@ -25,13 +25,21 @@ function sortNames (a,b) {
 }
 
 function sortLabels (a,b) {
-       var aname =  a.getAttributeNS(rptNS, 'label').toLowerCase();
-       if (!aname) aname = a.getAttribute('name');
-       if (!aname) aname = a.getAttribute('id');
+       var aname =  a.getAttributeNS(rptNS, 'label');
 
-       var bname =  b.getAttributeNS(rptNS, 'label').toLowerCase();
-       if (!bname) bname = b.getAttribute('name');
-       if (!bname) bname = b.getAttribute('id');
+       if (aname) {
+               aname = aname.toLowerCase();
+       } else {
+               aname = a.getAttribute('name') || a.getAttribute('id');
+       }
+
+       var bname =  b.getAttributeNS(rptNS, 'label');
+
+       if (bname) {
+               bname = bname.toLowerCase();
+       } else {
+               bname = b.getAttribute('name') || b.getAttribute('id');
+       }
 
        return aname < bname ? -1 : 1;
 }