[open-ils-commits] r15635 - trunk/Open-ILS/web/js/ui/default/actor/user (erickson)

[svn at svn.open-ils.org]

Author: erickson
Date: 2010-02-24 14:30:44 -0500 (Wed, 24 Feb 2010)
New Revision: 15635

Modified:
   trunk/Open-ILS/web/js/ui/default/actor/user/register.js
Log:
fetch user groups non-async to prevent clobbering expire date when group dijit updates with async data.  set required var before it's used.  don't require values in the password field for existing users

Modified: trunk/Open-ILS/web/js/ui/default/actor/user/register.js
===================================================================
--- trunk/Open-ILS/web/js/ui/default/actor/user/register.js	2010-02-24 18:03:41 UTC (rev 15634)
+++ trunk/Open-ILS/web/js/ui/default/actor/user/register.js	2010-02-24 19:30:44 UTC (rev 15635)
@@ -425,6 +425,12 @@
             break;
     }
 
+    var required = row.getAttribute('required') == 'required';
+
+    // password data is not fetched/required/displayed for existing users
+    if(!patron.isnew() && 'passwd' == fmfield)
+        required = false;
+
     var dijitArgs = {
         style: wstyle, 
         required : required,
@@ -436,8 +442,12 @@
     if(value !== null)
         dijitArgs.value = value;
 
-    var required = row.getAttribute('required') == 'required';
+    // fetch profile groups non-async so existing expire_date is
+    // not overwritten when the profile groups arrive and update
+    var sync = (fmfield == 'profile') ? true : false;
+
     var widget = new openils.widget.AutoFieldWidget({
+        forceSync : sync,
         idlField : fieldIdl,
         fmObject : fmObject,
         fmClass : fmcls,