Skip to content

Commit 046e0ee

Browse files
committed
fix: add submitter
1 parent 29f6b75 commit 046e0ee

File tree

5 files changed

+31
-16
lines changed

5 files changed

+31
-16
lines changed

config/default.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ module.exports = {
7070
timeout: parseInt(process.env.TIMEOUT, 10) || 30000
7171
},
7272

73+
LEGACY_SUBMITTER_ROLE_ID: process.env.LEGACY_SUBMITTER_ROLE_ID || 1,
7374
LEGACY_REVIEWER_ROLE_ID: process.env.LEGACY_REVIEWER_ROLE_ID || 4,
7475
LEGACY_REVIEW_PHASE_ID: process.env.LEGACY_REVIEW_PHASE_ID || 4,
7576
LEGACY_REVIEWER_ITERATIVE_ROLE_ID: process.env.LEGACY_REVIEWER_ITERATIVE_ROLE_ID || 21,

src/common/helper.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,10 @@ function isReviewerRole (roleId) {
203203
return roleId === config.REVIEWER_ROLE_ID || roleId === config.ITERATIVE_REVIEWER_ROLE_ID || roleId === config.LEGACY_REVIEWER_ROLE_ID || roleId === config.LEGACY_REVIEWER_ITERATIVE_ROLE_ID
204204
}
205205

206+
function isSubmitterRole (roleId) {
207+
return roleId === config.LEGACY_SUBMITTER_ROLE_ID || roleId === config.SUBMITTER_ROLE_ID
208+
}
209+
206210
async function getUserId (handle) {
207211
const token = await getM2Mtoken()
208212
const result = await getRequest(`${config.MEMBER_API_V5_URL}/${handle}`, token)
@@ -223,5 +227,6 @@ module.exports = {
223227
executeSQLonDB,
224228
forceV4ESFeeder,
225229
isReviewerRole,
230+
isSubmitterRole,
226231
getUserId
227232
}

src/dao/RegistrationDAO.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ async function persistResourceWithRoleId (userId, challengeId, resourceId, roleI
5252
await persistResourceInfo(userId, resourceId, RESOURCE_TYPE_HANDLE_ID, handle)
5353
await persistResourceInfo(userId, resourceId, RESOURCE_TYPE_REG_DATE, regDate)
5454
await persistResourceInfo(userId, resourceId, RESOURCE_TYPE_APPEALS_COMPLETED, 'NO')
55+
5556
if (roleId === COPILOT_RESOURCE_ROLE_ID) {
5657
logger.debug(`Copilot Payment Amount Set on Resource Info Table: ${copilotPaymentAmount}`)
5758
await persistResourceInfo(userId, resourceId, RESOURCE_TYPE_COPILOT_PAYMENT_ID, copilotPaymentAmount || 0)

src/services/ProcessorService.js

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,12 +116,17 @@ async function _updateChallengeResource (message, isDelete) {
116116
await helper.forceV4ESFeeder(_.get(v5Challenge, 'legacyId'))
117117
await new Promise(resolve => setTimeout(resolve, config.INDEX_CHALLENGE_TIMEOUT * 1000))
118118
logger.debug('End v4 challenge reindexing to the elasticsearch service')
119+
119120
if (isDelete) {
120-
logger.debug(`v4 Unregistering Submitter ${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/unregister?userId=${userId} - ${JSON.stringify(body)}`)
121-
await helper.postRequest(`${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/unregister?userId=${userId}`, {}, m2mToken)
121+
// logger.debug(`v4 Unregistering Submitter ${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/unregister?userId=${userId} - ${JSON.stringify(body)}`)
122+
// await helper.postRequest(`${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/unregister?userId=${userId}`, {}, m2mToken)
123+
logger.debug(`Unregistering Submitter ${legacyChallengeID} with roleID ${config.LEGACY_SUBMITTER_ROLE_ID} and userId ${userId}`)
124+
await ResourceDirectManager.removeResource(legacyChallengeID, config.LEGACY_SUBMITTER_ROLE_ID, userId)
122125
} else {
123-
logger.debug(`v4 Registering Submitter ${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/register?userId=${userId}&v5ChallengeId=${challengeId} - ${JSON.stringify(body)}`)
124-
await helper.postRequest(`${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/register?userId=${userId}&v5ChallengeId=${challengeId}`, {}, m2mToken)
126+
// logger.debug(`v4 Registering Submitter ${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/register?userId=${userId}&v5ChallengeId=${challengeId} - ${JSON.stringify(body)}`)
127+
// await helper.postRequest(`${config.CHALLENGE_API_V4_URL}/${_.get(v5Challenge, 'legacyId')}/register?userId=${userId}&v5ChallengeId=${challengeId}`, {}, m2mToken)
128+
logger.debug(`Registering Submitter ${legacyChallengeID} with roleID ${config.LEGACY_SUBMITTER_ROLE_ID} and userId ${userId}`)
129+
await ResourceDirectManager.addResource(legacyChallengeID, config.LEGACY_SUBMITTER_ROLE_ID, userId, handle, {}, {})
125130
}
126131
} else {
127132
if (isDelete) {

src/services/ResourceDirectManager.js

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const ProjectPaymentDAO = require('../dao/ProjectPaymentDAO')
55
const logger = require('../common/logger')
66
const config = require('config')
77
const { find, toString } = require('lodash')
8-
const { isReviewerRole } = require('../common/helper')
8+
const { isReviewerRole, isSubmitterRole } = require('../common/helper')
99

1010
/**
1111
* Assign the given roleId to the specified userId in the given project.
@@ -26,19 +26,22 @@ async function assignRole (legacyChallengeId, roleId, userId, handle, copilotPay
2626
if (found) {
2727
throw new Error('User ' + userId + ' with role ' + roleId + ' already exists')
2828
}
29-
// if not found && user agreed terms (if any) && is eligible, add resource
30-
if (!found && termChecking && eligible) {
31-
const allroles = await ProjectServices.getAllResourceRoles()
32-
let roleToSet = null
33-
if (allroles && allroles.length > 0) {
34-
for (const role of allroles) {
35-
if (role.resource_role_id === roleId) {
36-
roleToSet = role
29+
30+
if (!isSubmitterRole(roleId)) {
31+
// if not found && user agreed terms (if any) && is eligible, add resource
32+
if (!found && termChecking && eligible) {
33+
const allroles = await ProjectServices.getAllResourceRoles()
34+
let roleToSet = null
35+
if (allroles && allroles.length > 0) {
36+
for (const role of allroles) {
37+
if (role.resource_role_id === roleId) {
38+
roleToSet = role
39+
}
3740
}
3841
}
39-
}
40-
if (!roleToSet) {
41-
throw new Error('Invalid role id ' + roleId)
42+
if (!roleToSet) {
43+
throw new Error('Invalid role id ' + roleId)
44+
}
4245
}
4346

4447
let projectPhaseId = null

0 commit comments

Comments
 (0)