From: phasefx <phasefx@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Wed, 20 Oct 2010 14:47:24 +0000 (+0000)
Subject: backport r18355 and r18399 for better phone-derived password behavior
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=a08bfdbaf05da64c10b2432721a9e709058c3c14;p=Evergreen.git

backport r18355 and r18399 for better phone-derived password behavior

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_6_0@18400 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/xul/staff_client/server/patron/ue.js b/Open-ILS/xul/staff_client/server/patron/ue.js
index d39e931282..de4838d424 100644
--- a/Open-ILS/xul/staff_client/server/patron/ue.js
+++ b/Open-ILS/xul/staff_client/server/patron/ue.js
@@ -291,17 +291,16 @@ function uEditResetPw(pw) {
             if( (pw = patron.day_phone()) ||
                 (pw = patron.evening_phone()) || (pw = patron.other_phone()) ) {
                     pw = pw.substring(pw.length - 4); // this is iffy
-                    uEditResetPw(pw);
-                        appendClear($('ue_password_plain'), text(pw));
-                        unHideMe($('ue_password_gen'));
+                    appendClear($('ue_password_plain'), text(pw));
+                    unHideMe($('ue_password_gen'));
              }
         } else {
             pw = uEditMakeRandomPw(patron);
         }
-        $('ue_password1').value = pw;
-        $('ue_password2').value = pw;
-        $('ue_password1').onchange();
     }
+    $('ue_password1').value = pw;
+    $('ue_password2').value = pw;
+    $('ue_password1').onchange();
 }
 
 
diff --git a/Open-ILS/xul/staff_client/server/patron/ue_config.js b/Open-ILS/xul/staff_client/server/patron/ue_config.js
index aec5f7cf9b..6fe6c524c1 100644
--- a/Open-ILS/xul/staff_client/server/patron/ue_config.js
+++ b/Open-ILS/xul/staff_client/server/patron/ue_config.js
@@ -296,7 +296,7 @@ function uEditDefineData(patron) {
 				type		: 'input',
 				regex		:  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 },
                 onpostchange: function(field, newval) {
@@ -320,7 +320,7 @@ function uEditDefineData(patron) {
 				type		: 'input',
 				regex		:  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 }
 			}
@@ -335,7 +335,7 @@ function uEditDefineData(patron) {
 				type		: 'input',
 				regex		:  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 }
 			}