Merge pull request #167 from CoderDojo/ckiss-fixes

fixed issues caused by load_dojo_admins_for_user

Author: ckiss

profiles.js

@@ -495,9 +495,12 @@ module.exports = function (options) {
           return champion.id === args.user ? args.user.id : null;
         });
 
-        seneca.act({role: 'cd-users', cmd: 'load_dojo_admins_for_user', userId: profile.userId, user: args.user}, function (err, dojoAdmins) {
+        seneca.act({role: 'cd-users', cmd: 'load_dojo_admins_for_user', userId: profile.userId}, function (err, dojoAdmins) {
           if (err) return done(err);
           profile.requestingUserIsDojoAdmin = _.find(dojoAdmins, function (dojoAdmin) {
+            if(!dojoAdmin.ok){
+              return false
+            }
             return dojoAdmin.id === args.user ? args.user.id : null;
           });
 

users.js

@@ -495,7 +495,7 @@ module.exports = function (options) {
 
   function cmd_load_dojo_admins_for_user (args, done) {
     var seneca = this;
-    var userId = args.userId || args.user ? args.user.id : null;
+    var userId = args.userId;
 
     seneca.act({role: 'cd-dojos', cmd: 'dojos_for_user', id: userId}, function (err, dojos) {
       if (err) return done(err);