github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-30 06:15:32 -05:00
Code Issues Projects Releases Packages Wiki Activity

add param to skip affiliation creation on user create

Browse source
This commit is contained in:
Tim Alby 2018-08-17 15:08:59 +02:00
parent b9e63421a8
commit f5513f36ca
2 changed files with 18 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
services/web/app/coffee/Features/User/UserCreator.coffee
View file

@ -6,15 +6,18 @@ metrics = require('metrics-sharelatex')
module.exports = UserCreator = module.exports = UserCreator =
createNewUser: (opts, callback)-> createNewUser: (attributes, options, callback = (error, user) ->)->
logger.log opts:opts, "creating new user" if arguments.length == 2
callback = options
options = {}
logger.log user: attributes, "creating new user"
user = new User() user = new User()
username = opts.email.match(/^[^@]*/) username = attributes.email.match(/^[^@]*/)
if !opts.first_name? or opts.first_name == "" if !attributes.first_name? or attributes.first_name == ""
opts.first_name = username[0] attributes.first_name = username[0]
for key, value of opts for key, value of attributes
user[key] = value user[key] = value
user.ace.syntaxValidation = true user.ace.syntaxValidation = true
@ -27,6 +30,7 @@ module.exports = UserCreator =
user.save (err)-> user.save (err)->
callback(err, user) callback(err, user)
return if options?.skip_affiliation
# call addaffiliation after the main callback so it runs in the # call addaffiliation after the main callback so it runs in the
# background. There is no guaranty this will run so we must no rely on it # background. There is no guaranty this will run so we must no rely on it
addAffiliation user._id, user.email, (error) -> addAffiliation user._id, user.email, (error) ->

8
services/web/test/unit/coffee/User/UserCreatorTests.coffee
View file

@ -88,3 +88,11 @@ describe "UserCreator", ->
process.nextTick () => process.nextTick () =>
sinon.assert.calledWith(@addAffiliation, user._id, user.email) sinon.assert.calledWith(@addAffiliation, user._id, user.email)
done() done()
it "should not add affiliation if skipping", (done)->
attributes = email: @email
options = skip_affiliation: true
@UserCreator.createNewUser attributes, options, (err, user) =>
process.nextTick () =>
sinon.assert.notCalled(@addAffiliation)
done()