fix: fix for saving complex instead of simple roles

WFCD · Aug 11, 2018 · 56e4990 · 56e4990
1 parent 04459d7
commit 56e4990
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/commands/Roles/AddRole.js b/src/commands/Roles/AddRole.js
@@ -81,7 +81,7 @@ class AddRole extends Command {
       return this.messageManager.statuses.FAILURE;
     }
 
-    const rolesToCommit = roles.map(innerRole => innerRole);
+    const rolesToCommit = roles.map(innerRole => innerRole.getSimple());
     const newRole = new JoinableRole(role);
 
     if (reqRoleRegex.test(message.strippedContent)) {

diff --git a/src/commands/Roles/RemoveRole.js b/src/commands/Roles/RemoveRole.js
@@ -54,7 +54,6 @@ class RemoveRole extends Command {
       return this.messageManager.statuses.FAILURE;
     }
     const roles = await this.settings.getRolesForGuild(message.guild);
-    this.logger.debug(`roles: ${JSON.stringify(roles.map(somerole => somerole.getSimple()))}`);
     const filteredRoles = roles.filter(storedRole => role.id === storedRole.id);
     if (filteredRoles.length > 0) {
       const deleteRole = deleteRegex.test(message.strippedContent);