Skip to content

Commit 1812347

Browse files
committed
Merge branch 'develop'
2 parents f6b0de9 + cab341c commit 1812347

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/services/ResourceService.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ async function createResource (currentUser, resource) {
262262
}
263263

264264
if (_.reduce(resources,
265-
(result, r) => _.toNumber(r.memberId) === _.toNumber(memberId) && r.roleId === resource.roleId ? true : result,
265+
(result, r) => _.toString(r.memberId) === _.toString(memberId) && r.roleId === resource.roleId ? true : result,
266266
false)) {
267267
throw new errors.ConflictError(`User ${resource.memberHandle} already has resource with roleId: ${resource.roleId} in challenge: ${challengeId}`)
268268
}
@@ -311,7 +311,7 @@ async function deleteResource (currentUser, resource) {
311311
const { resources, memberId, handle } = await init(currentUser, challengeId, resource)
312312

313313
const ret = _.reduce(resources,
314-
(result, r) => _.toNumber(r.memberId) === _.toNumber(memberId) && r.roleId === resource.roleId ? r : result,
314+
(result, r) => _.toString(r.memberId) === _.toString(memberId) && r.roleId === resource.roleId ? r : result,
315315
undefined)
316316

317317
if (!ret) {
@@ -410,7 +410,7 @@ async function listChallengesByMember (memberId, criteria) {
410410
}
411411

412412
listChallengesByMember.schema = {
413-
memberId: Joi.number().integer().positive().required(),
413+
memberId: Joi.string().required(),
414414
criteria: Joi.object().keys({
415415
resourceRoleId: Joi.string().uuid(),
416416
page: Joi.page(),

0 commit comments

Comments
 (0)