memberId comparison should be a number always

Author: rootelement

src/services/ResourceService.js

@@ -214,7 +214,7 @@ async function createResource (currentUser, resource) {
     }
 
     if (_.reduce(resources,
-      (result, r) => r.memberId === memberId && r.roleId === resource.roleId ? true : result,
+      (result, r) => _.toNumber(r.memberId) === _.toNumber(memberId) && r.roleId === resource.roleId ? true : result,
       false)) {
       throw new errors.ConflictError(`User ${resource.memberHandle} already has resource with roleId: ${resource.roleId} in challenge: ${challengeId}`)
     }
@@ -263,7 +263,7 @@ async function deleteResource (currentUser, resource) {
     const { resources, memberId, handle } = await init(currentUser, challengeId, resource)
 
     const ret = _.reduce(resources,
-      (result, r) => r.memberId === memberId && r.roleId === resource.roleId ? r : result,
+      (result, r) => _.toNumber(r.memberId) === _.toNumber(memberId) && r.roleId === resource.roleId ? r : result,
       undefined)
 
     if (!ret) {