v3 and adapted v2: bugs fixes for groups, challenges and tasks

common/script/fns/randomDrop.js

@@ -3,7 +3,7 @@ import content from '../content/index';
 import i18n from '../i18n';
 import { daysSince } from '../cron';
 import { diminishingReturns } from '../statHelpers';
-import { predictableRandom } from './predictableRandom';
+import predictableRandom from './predictableRandom';
 import randomVal from './randomVal';
 
 // Clone a drop object maintaining its functions so that we can change it without affecting the original item

website/src/controllers/api-v2/challenges.js

@@ -52,7 +52,9 @@ api.list = async function(req, res, next) {
       let obj = challenge.toJSON();
 
       obj._isMember = user.challenges.indexOf(challenge._id) !== -1;
+      return obj;
     });
+
     // TODO Instead of populate we make a find call manually because of https://github.com/Automattic/mongoose/issues/3833
     await Q.all(resChals.map((chal, index) => {
       return Q.all([
@@ -195,7 +197,8 @@ api.create = async function(req, res, next){
     req.body.rewards = req.body.rewards || [];
 
     var chalTasks = req.body.habits.concat(req.body.rewards)
-                  .concat(req.body.dailys).concat(req.body.todos);
+                  .concat(req.body.dailys).concat(req.body.todos)
+                  .map(v2Task => Tasks.Task.fromJSONV2(v2Task));
 
     chalTasks = chalTasks.map(function(task) {
       var newTask = new Tasks[task.type](Tasks.Task.sanitize(task));
@@ -318,7 +321,7 @@ api.selectWinner = async function(req, res, next) {
     if (!challenge) return next('Challenge ' + req.params.cid + ' not found');
     if (!challenge.canModify(res.locals.user)) return next(shared.i18n.t('noPermissionCloseChallenge'));
 
-    let winner = await User.findOne({_id: req.params.uid}).exec();
+    let winner = await User.findOne({_id: req.query.uid}).exec();
     if (!winner || winner.challenges.indexOf(challenge._id) === -1) return next('Winner ' + req.query.uid + ' not found.');
 
     // Close channel in background, some ops are run in the background without `await`ing

website/src/controllers/api-v2/user.js

@@ -676,7 +676,14 @@ api.cast = async function(req, res, next) {
 
         if (!partyMembers) throw new NotFound(res.t('userWithIDNotFound', {userId: targetId}));
         spell.cast(user, partyMembers, req);
+        if (partyMembers === user) {
           await partyMembers.save();
+        } else {
+          await Q.all([
+            await partyMembers.save(),
+            await user.save(),
+          ]);
+        }
       }
 
       if (party && !spell.silent) {

website/src/controllers/api-v3/user.js

@@ -350,7 +350,15 @@ api.castSpell = {
       if (task.challenge.id) throw new BadRequest(res.t('challengeTasksNoCast'));
 
       spell.cast(user, task, req);
+      if (user.isModified()) {
+        await Q.all([
+          user.save(),
+          task.save(),
+        ]);
+      } else {
         await task.save();
+      }
+
       res.respond(200, task);
     } else if (targetType === 'self') {
       spell.cast(user, null, req);
@@ -370,7 +378,8 @@ api.castSpell = {
 
       let toSave = tasks.filter(t => t.isModified());
       let isUserModified = user.isModified();
-      toSave.unshift(user.save());
+
+      if (isUserModified) toSave.unshift(user.save());
       let saved = await Q.all(toSave);
 
       let response = {
@@ -403,8 +412,15 @@ api.castSpell = {
 
         if (!partyMembers) throw new NotFound(res.t('userWithIDNotFound', {userId: targetId}));
         spell.cast(user, partyMembers, req);
+        if (user.isModified()) {
+          await Q.all([
+            user.save(),
+            partyMembers.save(),
+          ]);
+        } else {
           await partyMembers.save();
         }
+      }
       res.respond(200, partyMembers);
 
       if (party && !spell.silent) {

website/src/models/group.js

@@ -631,19 +631,17 @@ schema.methods.leave = async function leaveGroup (user, keep = 'keep-all') {
 
   let promises = [];
 
-  // If user is the last one in group and group is private, delete it
+  // remove the group from the user's groups
-  if (group.memberCount <= 1 && group.privacy === 'private') {
-    return await group.remove();
-  }
-
-  // otherwise just remove a member TODO create User.methods.removeFromGroup?
   if (group.type === 'guild') {
     promises.push(User.update({_id: user._id}, {$pull: {guilds: group._id}}).exec());
   } else {
     promises.push(User.update({_id: user._id}, {$set: {party: {}}}).exec());
   }
 
-  // If the leader is leaving (or if the leader previously left, and this wasn't accounted for)
+  // If user is the last one in group and group is private, delete it
+  if (group.memberCount <= 1 && group.privacy === 'private') {
+    return await group.remove();
+  } else { // otherwise If the leader is leaving (or if the leader previously left, and this wasn't accounted for)
     let update = {
       $inc: {memberCount: -1},
     };
@@ -657,6 +655,8 @@ schema.methods.leave = async function leaveGroup (user, keep = 'keep-all') {
       if (seniorMember) update.$set = {leader: seniorMember._id};
     }
     promises.push(group.update(update).exec());
+  }
+
   firebase.removeUserFromGroup(group._id, user._id);
 
   return Q.all(promises);
@@ -730,7 +730,7 @@ if (!nconf.get('IS_TEST')) {
     new model({ // eslint-disable-line babel/new-cap
       _id: TAVERN_ID,
       leader: '7bde7864-ebc5-4ee2-a4b7-1070d464cdb0', // Siena Leslie
-      name: 'HabitRPG',
+      name: 'Tavern',
       type: 'guild',
       privacy: 'public',
     }).save();