Skip to content
This repository was archived by the owner on Apr 13, 2023. It is now read-only.

Commit 4da4f97

Browse files
authored
Merge pull request #37 from SpringRoll/bug/160066769-game-group-delete
Fix bug where deleted groups could still be referenced by games
2 parents 5c91944 + 38ecf48 commit 4da4f97

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

app/routes/groups/group.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,12 +99,20 @@ router.post('/:slug', access.isAdmin, function(req, res)
9999
}
100100
case "deleteGroup":
101101
{
102+
// Remove reference to group from game
103+
Game.removeGroup(null, group._id, function(err)
104+
{
105+
if(err) {
106+
done(err);
107+
return
108+
}
102109
// Remove the group
103110
group.remove(function(err)
104111
{
105112
req.flash('success', 'Deleted ' + group.name + ' successfully.');
106113
res.redirect('/groups');
107-
});
114+
})
115+
});
108116
break;
109117
}
110118
case "updateGroup":

npm-shrinkwrap.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)