[open-ils-commits] [GIT] Evergreen ILS branch rel_2_3 updated. 3bde56c32c3de88112bfae364d992a40b1cc19ec

[Evergreen Git]

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_3 has been updated
       via  3bde56c32c3de88112bfae364d992a40b1cc19ec (commit)
      from  7c793f9cbf9423deb92842f41a4a2ab268090e17 (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 3bde56c32c3de88112bfae364d992a40b1cc19ec
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Thu Jul 26 16:45:24 2012 -0400

    Trying to update patron CSS color fails here, so don't try
    
    After modifying user standing penalties ("messages" in the staff client
    patron interface), you could get this error message in an alert() dialog:
        TypeError: patron.display is undefined.
    
    We were apparently trying to use patron.util.set_patron_css() wrongly,
    so the commented out changed verison of the call got farther, but stopped at
        ...penalties[i].standing_penalty().name is not a function.
    
    Which probably has something to do with unfleshed data.  Somebody can
    get to the bottom of this if they want, but for now it's easier just not
    to try to do this and make the alert() dialog go away.
    
    Signed-off-by: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
    Signed-off-by: Bill Erickson <berick at esilibrary.com>

diff --git a/Open-ILS/xul/staff_client/server/patron/standing_penalties.js b/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
index ce62e22..d36edf4 100644
--- a/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
+++ b/Open-ILS/xul/staff_client/server/patron/standing_penalties.js
@@ -226,7 +226,7 @@ function handle_remove_penalty(ev) {
                 */
                 document.getElementById('progress').hidden = true;
 
-                patron.util.set_penalty_css(xulG.patron, patron.display.w.document.documentElement);
+                //patron.util.set_penalty_css(xulG.patron);
             }
         );
         document.getElementById('progress').hidden = false;
@@ -311,7 +311,7 @@ function handle_edit_penalty(ev) {
                                     row_params.row.my.csp = p.standing_penalty();
                                     list.refresh_row( row_params );
 
-                                    patron.util.set_penalty_css(xulG.patron, patron.display.w.document.documentElement);
+                                    //patron.util.set_penalty_css(xulG.patron);
                                     document.getElementById('progress').hidden = true;
                                 } catch(E) {
                                     alert(E);
@@ -375,7 +375,7 @@ function handle_archive_penalty(ev) {
                             if (--outstanding_requests==0) {
                                 document.getElementById('progress').hidden = true;
 
-                                patron.util.set_penalty_css(xulG.patron, patron.display.w.document.documentElement);
+                                //patron.util.set_penalty_css(xulG.patron);
                             }
                         }
                     }(ids[i])

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

Summary of changes:
 .../server/patron/standing_penalties.js            |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
Evergreen ILS