Skip to content
This repository was archived by the owner on Dec 14, 2023. It is now read-only.

Commit 55f3ff4

Browse files
authored
Force user_id to what's known (#299)
* Force user_id to what's known * Linting
1 parent 6af6d21 commit 55f3ff4

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

profiles.js

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -252,17 +252,22 @@ module.exports = function (options) {
252252

253253
profile = _.omit(profile, fieldsToBeRemoved);
254254
var cleanedProfile = _.omit(profile, 'user');
255-
seneca.act({role: plugin, cmd: 'save', profile: cleanedProfile}, function (err, savedProfile) {
256-
if (err) {
257-
return done(err);
258-
}
259-
// save mutates the profile values, like "name" from "firstName" + "lastName" and need to be ported to the synced sys_user
260-
savedProfile = _.omit(savedProfile, fieldsToBeRemoved);
261-
savedProfile.user = profile.user;
262-
syncUserObj(savedProfile, function (err, res) {
263-
if (err) return done(err);
255+
seneca.act({ role: plugin, cmd: 'load', id: cleanedProfile.id }, function (err, requestedProfile) {
256+
if (err) return done(err);
257+
258+
seneca.act({role: plugin, cmd: 'save', profile: cleanedProfile}, function (err, savedProfile) {
259+
if (err) {
260+
return done(err);
261+
}
262+
// save mutates the profile values, like "name" from "firstName" + "lastName" and need to be ported to the synced sys_user
263+
savedProfile = _.omit(savedProfile, fieldsToBeRemoved);
264+
savedProfile.user = profile.user;
265+
savedProfile.userId = requestedProfile.userId;
266+
syncUserObj(savedProfile, function (err, res) {
267+
if (err) return done(err);
264268

265-
return done(null, profile);
269+
return done(null, profile);
270+
});
266271
});
267272
});
268273
}

0 commit comments

Comments
 (0)