From fffa95ca09748a7b6a8dde86e9a9267e1ee87068 Mon Sep 17 00:00:00 2001
From: Blade Barringer <blade@crookedneighbor.com>
Date: Tue, 12 Jan 2016 08:11:19 -0600
Subject: [PATCH] refactor: Test against party not being private, instead of
 public

---
 website/src/controllers/api-v3/groups.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/website/src/controllers/api-v3/groups.js b/website/src/controllers/api-v3/groups.js
index 8604337648..13ddf76d64 100644
--- a/website/src/controllers/api-v3/groups.js
+++ b/website/src/controllers/api-v3/groups.js
@@ -31,7 +31,6 @@ api.createGroup = {
   middlewares: [authWithHeaders(), cron],
   async handler (req, res) {
     let user = res.locals.user;
-
     let group = new Group(Group.sanitize(req.body)); // TODO validate empty req.body
     group.leader = user._id;
 
@@ -43,7 +42,7 @@ api.createGroup = {
       user.balance--;
       user.guilds.push(group._id);
     } else {
-      if (group.privacy === 'public') throw new NotAuthorized(res.t('partyMustbePrivate'));
+      if (group.privacy !== 'private') throw new NotAuthorized(res.t('partyMustbePrivate'));
       if (user.party._id) throw new NotAuthorized(res.t('messageGroupAlreadyInParty'));
 
       user.party._id = group._id;
@@ -54,6 +53,7 @@ api.createGroup = {
 
     firebase.updateGroupData(savedGroup);
     firebase.addUserToGroup(savedGroup._id, user._id);
+
     return res.respond(201, savedGroup); // TODO populate
   },
 };