From c2915f9e1427853991828da1f9a210ea0a401c87 Mon Sep 17 00:00:00 2001 From: Berkeley Martinez Date: Wed, 10 Feb 2016 16:45:57 -0800 Subject: [PATCH] Convert model instance to POJO earilier in migration change --- server/middlewares/migrate-completed-challenges.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/server/middlewares/migrate-completed-challenges.js b/server/middlewares/migrate-completed-challenges.js index 7aec5e0b3c..a5f7110f86 100644 --- a/server/middlewares/migrate-completed-challenges.js +++ b/server/middlewares/migrate-completed-challenges.js @@ -22,10 +22,6 @@ const challengeTypeRegWithColon = /^(bonfire|checkpoint|waypoint|hike|zipline|basejump):\s+/i; function updateName(challenge) { - challenge = challenge && typeof challenge.toJSON === 'function' ? - challenge.toJSON() : - challenge; - if ( challenge.name && challenge.challengeType === 5 && @@ -57,6 +53,7 @@ function updateId(challenge) { if (idMap.hasOwnProperty(challenge.id)) { challenge.id = idMap[challenge.id]; } + return challenge; } @@ -72,6 +69,11 @@ function buildChallengeMap(userId, completedChallenges = [], User) { null, Scheduler.default ) + .map(challenge => { + return challenge && typeof challenge.toJSON === 'function' ? + challenge.toJSON() : + challenge; + }) .map(updateId) .filter(({ id, _id }) => ObjectID.isValid(id || _id)) .map(updateName)