github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2025-04-05 00:27:17 +00:00
Code Issues Projects Releases Packages Wiki Activity

Update removeUserFromProject to account for token-access

Browse source
This commit is contained in:
Shane Kilkelly 2017-10-12 11:49:02 +01:00
parent 6feedf5520
commit 16416463c6
2 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
services/web/app/coffee/Features/Collaborators/CollaboratorsHandler.coffee
View file

@ -158,7 +158,11 @@ module.exports = CollaboratorsHandler =
logger.log user_id: user_id, project_id: project_id, "removing user"
conditions = _id:project_id
update = $pull:{}
update["$pull"] = collaberator_refs:user_id, readOnly_refs:user_id
update["$pull"] =
collaberator_refs:user_id,
readOnly_refs:user_id
tokenAccessReadOnly_refs:user_id
tokenAccessReadAndWrite_refs:user_id
Project.update conditions, update, (err)->
if err?
logger.error err: err, "problem removing user from project collaberators"

5
services/web/test/UnitTests/coffee/Collaborators/CollaboratorsHandlerTests.coffee
View file

@ -223,7 +223,10 @@ describe "CollaboratorsHandler", ->
.calledWith({
_id: @project_id
}, {
"$pull":{collaberator_refs:@user_id, readOnly_refs:@user_id}
"$pull":{
collaberator_refs:@user_id, readOnly_refs:@user_id,
tokenAccessReadOnly_refs:@user_id, tokenAccessReadAndWrite_refs:@user_id
}
})
.should.equal true