Merge pull request #521 from topcoder-platform/hotfix/checkIfExist

Hotfix/check if exist

Author: ThomasKranitsas

Diff for: src/controllers/ChallengeController.js

@@ -61,7 +61,7 @@ async function sendNotifications (req, res) {
  * @param {Object} res the response
  */
 async function getChallenge (req, res) {
-  const result = await service.getChallenge(req.authUser, req.params.challengeId)
+  const result = await service.getChallenge(req.authUser, req.params.challengeId, req.query.checkIfExists)
   res.send(result)
 }
 
@@ -116,6 +116,5 @@ module.exports = {
   partiallyUpdateChallenge,
   deleteChallenge,
   getChallengeStatistics,
-  sendNotifications,
-  getChallengeStatistics
+  sendNotifications
 }

Diff for: src/services/ChallengeService.js

@@ -1221,9 +1221,10 @@ async function getPhasesAndPopulate (data) {
  * Get challenge.
  * @param {Object} currentUser the user who perform operation
  * @param {String} id the challenge id
+ * @param {Boolean} checkIfExists flag to check if challenge exists
  * @returns {Object} the challenge with given id
  */
-async function getChallenge (currentUser, id) {
+async function getChallenge (currentUser, id, checkIfExists) {
   // get challenge from Elasticsearch
   let challenge
   // logger.warn(JSON.stringify({
@@ -1244,6 +1245,9 @@ async function getChallenge (currentUser, id) {
       throw e
     }
   }
+  if (checkIfExists) {
+    return _.pick(challenge, ['id', 'legacyId'])
+  }
   await helper.ensureUserCanViewChallenge(currentUser, challenge)
 
   // // FIXME: Temporarily hard coded as the migrad
@@ -1288,7 +1292,8 @@ async function getChallenge (currentUser, id) {
 
 getChallenge.schema = {
   currentUser: Joi.any(),
-  id: Joi.id()
+  id: Joi.id(),
+  checkIfExists: Joi.boolean()
 }
 
 /**