From: erickson <erickson@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Mon, 10 Jan 2011 16:29:27 +0000 (+0000)
Subject: avoid fetching user settings for new user registration;  the result of fetching setti... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=e3a4af416d1b891c2fc84aeab63710284d18252e;p=evergreen%2Fequinox.git

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

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

diff --git a/Open-ILS/web/js/ui/default/actor/user/register.js b/Open-ILS/web/js/ui/default/actor/user/register.js
index 835129b933..f7eb1f5bdc 100644
--- a/Open-ILS/web/js/ui/default/actor/user/register.js
+++ b/Open-ILS/web/js/ui/default/actor/user/register.js
@@ -383,9 +383,11 @@ function uEditFetchUserSettings(userId) {
     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]});
+    }
 }