[open-ils-commits] r18401 - branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron (phasefx)

[svn at svn.open-ils.org]

Author: phasefx
Date: 2010-10-20 10:47:32 -0400 (Wed, 20 Oct 2010)
New Revision: 18401

Modified:
   branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue.js
   branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue_config.js
Log:
backport r18355 and r18399 for better phone-derived password behavior

Modified: branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue.js
===================================================================
--- branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue.js	2010-10-20 14:47:24 UTC (rev 18400)
+++ branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue.js	2010-10-20 14:47:32 UTC (rev 18401)
@@ -291,17 +291,16 @@
             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();
 }
 
 

Modified: branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue_config.js
===================================================================
--- branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue_config.js	2010-10-20 14:47:24 UTC (rev 18400)
+++ branches/rel_1_6_1/Open-ILS/xul/staff_client/server/patron/ue_config.js	2010-10-20 14:47:32 UTC (rev 18401)
@@ -296,7 +296,7 @@
                 type        : 'input',
                 regex        :  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 },
                 onpostchange: function(field, newval) {
@@ -320,7 +320,7 @@
                 type        : 'input',
                 regex        :  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 }
             }
@@ -335,7 +335,7 @@
                 type        : 'input',
                 regex        :  phoneRegex,
                 onblur      : function() {
-                    if(uEditUsePhonePw)
+                    if(uEditUsePhonePw && patron.isnew())
                         uEditMakePhonePw();
                 }
             }