[open-ils-commits] r17498 - trunk/Open-ILS/xul/staff_client/server/patron (phasefx)

[svn at svn.open-ils.org]

Author: phasefx
Date: 2010-09-07 11:24:55 -0400 (Tue, 07 Sep 2010)
New Revision: 17498

Modified:
   trunk/Open-ILS/xul/staff_client/server/patron/display.js
Log:
Only modify msg when needed or we'll get spurious 'stop sign' pages

Modified: trunk/Open-ILS/xul/staff_client/server/patron/display.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/patron/display.js	2010-09-07 14:52:01 UTC (rev 17497)
+++ trunk/Open-ILS/xul/staff_client/server/patron/display.js	2010-09-07 15:24:55 UTC (rev 17498)
@@ -932,7 +932,7 @@
                     }
                 }
                 var penalties = obj.patron.standing_penalties();
-                msg += '<dl>';
+                if (penalties.length > 0) { msg += '<dl>'; }
                 for (var i = 0; i < penalties.length; i++) {
                     if (penalties[i].standing_penalty().block_list() || penalties[i].standing_penalty().id() == 20 /* ALERT_NOTE */) {
                         msg += '<dt>';
@@ -942,7 +942,7 @@
                         msg += '</dd>';
                     }
                 }
-                msg += '</dl>';
+                if (penalties.length > 0) { msg += '</dl>'; }
                 var holds = params.holds_summary;
                 if (holds.ready && holds.ready > 0) {
                     msg += $("patronStrings").getFormattedString('staff.patron.display.init.holds_ready', [holds.ready]);