From e366e0bf55e4d7a2cf6171da9e86a97124af19e0 Mon Sep 17 00:00:00 2001 From: Marcos Spessatto Defendi Date: Fri, 14 Jun 2019 02:04:26 -0300 Subject: [PATCH] [FIX] Move the set Avatar call on user creation to make sure the user has username (#14665) --- server/lib/accounts.js | 29 ++++++++++++++--------------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/server/lib/accounts.js b/server/lib/accounts.js index c9a47e50d17..9773695e2cf 100644 --- a/server/lib/accounts.js +++ b/server/lib/accounts.js @@ -194,6 +194,20 @@ Accounts.insertUserDoc = _.wrap(Accounts.insertUserDoc, function(insertUserDoc, return callbacks.run('afterCreateUser', user); }); } + if (settings.get('Accounts_SetDefaultAvatar') === true) { + const avatarSuggestions = getAvatarSuggestionForUser(user); + Object.keys(avatarSuggestions).some((service) => { + const avatarData = avatarSuggestions[service]; + if (service !== 'gravatar') { + Meteor.runAsUser(_id, function() { + return Meteor.call('setAvatarFromService', avatarData.blob, '', service); + }); + return true; + } + + return false; + }); + } } if (roles.length === 0) { @@ -218,21 +232,6 @@ Accounts.insertUserDoc = _.wrap(Accounts.insertUserDoc, function(insertUserDoc, addUserRoles(_id, roles); - if (settings.get('Accounts_SetDefaultAvatar') === true) { - const avatarSuggestions = getAvatarSuggestionForUser(user); - Object.keys(avatarSuggestions).some((service) => { - const avatarData = avatarSuggestions[service]; - if (service !== 'gravatar') { - Meteor.runAsUser(_id, function() { - return Meteor.call('setAvatarFromService', avatarData.blob, '', service); - }); - return true; - } - - return false; - }); - } - return _id; });