[open-ils-commits] r19147 - branches/rel_2_0/Open-ILS/web/js/ui/default/actor/user (erickson)

[svn at svn.open-ils.org]

Author: erickson
Date: 2011-01-10 11:29:27 -0500 (Mon, 10 Jan 2011)
New Revision: 19147

Modified:
   branches/rel_2_0/Open-ILS/web/js/ui/default/actor/user/register.js
Log:
avoid fetching user settings for new user registration;  the result of fetching settings with a null user id is that settings of the staff/editor were fetched instead, leading to settings that appeared to selected, but were not saved during submit

Modified: branches/rel_2_0/Open-ILS/web/js/ui/default/actor/user/register.js
===================================================================
--- branches/rel_2_0/Open-ILS/web/js/ui/default/actor/user/register.js	2011-01-10 16:27:54 UTC (rev 19146)
+++ branches/rel_2_0/Open-ILS/web/js/ui/default/actor/user/register.js	2011-01-10 16:29:27 UTC (rev 19147)
@@ -383,9 +383,11 @@
     var names = userSettingTypes.map(function(obj) { return obj.name() });
 
     /* fetch any values set for this user */
-    userSettings = fieldmapper.standardRequest(
-        ['open-ils.actor', 'open-ils.actor.patron.settings.retrieve.authoritative'],
-        {params : [openils.User.authtoken, userId, names]});
+    if(userId) {
+        userSettings = fieldmapper.standardRequest(
+            ['open-ils.actor', 'open-ils.actor.patron.settings.retrieve.authoritative'],
+            {params : [openils.User.authtoken, userId, names]});
+    }
 }