Skip to content

Commit 751e3ea

Browse files
Add a ton of logging
1 parent 595848f commit 751e3ea

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

src/services/ChallengeService.js

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -645,15 +645,20 @@ async function update (currentUser, challengeId, data, userToken, isFull) {
645645
helper.ensureNoDuplicateOrNullElements(data.groups, 'groups')
646646
helper.ensureNoDuplicateOrNullElements(data.gitRepoURLs, 'gitRepoURLs')
647647

648+
console.log('Before fetching challenge')
648649
const challenge = await helper.getById('Challenge', challengeId)
649-
650+
console.log('After fetching challenge')
650651
// check groups authorization
652+
console.log('Before checking group access')
651653
await ensureAccessibleByGroupsAccess(currentUser, challenge)
654+
console.log('After checking group access')
652655

656+
console.log('before fetching attachments')
653657
let newAttachments
654658
if (isFull || !_.isUndefined(data.attachmentIds)) {
655659
newAttachments = await helper.getByIds('Attachment', data.attachmentIds || [])
656660
}
661+
console.log('after fetching attachments')
657662

658663
if (!currentUser.isMachine && !helper.hasAdminRole(currentUser) && challenge.createdBy.toLowerCase() !== currentUser.handle.toLowerCase()) {
659664
throw new errors.ForbiddenError(`Only M2M, admin or challenge's copilot can perform modification.`)
@@ -669,26 +674,33 @@ async function update (currentUser, challengeId, data, userToken, isFull) {
669674
})
670675
}
671676

677+
console.log('before validateChallengeData(data)')
672678
await validateChallengeData(data)
673679
if ((challenge.status === constants.challengeStatuses.Completed || challenge.status === constants.challengeStatuses.Canceled) && data.status && data.status !== challenge.status) {
674680
throw new errors.BadRequestError(`Cannot change ${challenge.status} challenge status to ${data.status} status`)
675681
}
682+
console.log('after validateChallengeData(data)')
676683

677684
if (data.winners && (challenge.status !== constants.challengeStatuses.Completed && data.status !== constants.challengeStatuses.Completed)) {
678685
throw new errors.BadRequestError(`Cannot set winners for challenge with non-completed ${challenge.status} status`)
679686
}
680687

688+
console.log('before validatePhases(data.phases)')
681689
if (data.phases) {
682690
await helper.validatePhases(data.phases)
683691
// populate phases
684692
await populatePhases(data.phases, data.startDate || challenge.startDate, data.timelineTemplateId || challenge.timelineTemplateId)
685693
data.endDate = helper.calculateChallengeEndDate(challenge, data)
686694
}
695+
console.log('after validatePhases(data.phases)')
687696

697+
console.log('before validateWinners(data.winners)')
688698
if (data.winners && data.winners.length) {
689699
await validateWinners(data.winners)
690700
}
701+
console.log('after validateWinners(data.winners)')
691702

703+
console.log('before constructing the query')
692704
data.updated = new Date()
693705
data.updatedBy = currentUser.handle || currentUser.sub
694706
const updateDetails = {}
@@ -884,15 +896,17 @@ async function update (currentUser, challengeId, data, userToken, isFull) {
884896
// send null to Elasticsearch to clear the field
885897
data.winners = null
886898
}
899+
console.log('after constructing the query')
887900

888-
console.log('------------------')
889-
console.log(updateDetails)
890-
console.log('------------------')
891-
901+
console.log('before update')
892902
await models.Challenge.update({ id: challengeId }, updateDetails)
903+
console.log('after update')
904+
console.log('before creating audit log')
905+
893906
if (auditLogs.length > 0) {
894907
await models.AuditLog.batchPut(auditLogs)
895908
}
909+
console.log('after creating audit log')
896910

897911
delete data.attachmentIds
898912
_.assign(challenge, data)

0 commit comments

Comments
 (0)