Added role.delete()

src/client/actions/ActionsManager.js

@@ -16,6 +16,7 @@ class ActionsManager {
 		this.register('GuildUpdate');
 		this.register('GuildMemberRemove');
 		this.register('GuildRoleCreate');
+		this.register('GuildRoleDelete');
 		this.register('UserUpdate');
 	}
 

src/client/actions/GuildMemberRemove.js

@@ -20,6 +20,7 @@ class GuildMemberRemoveAction extends Action {
 			if (member) {
 				guild._removeMember(member);
 				this.deleted[guild.id + data.user.id] = member;
+				client.emit(Constants.Events.GUILD_MEMBER_REMOVE, guild, member);
 				this.scheduleForDeletion(guild.id, data.user.id);
 			}
 

src/client/actions/GuildRoleDelete.js

@@ -0,0 +1,50 @@
+'use strict';
+
+const Action = require('./Action');
+const Constants = require('../../util/Constants');
+const Message = require('../../structures/Message');
+
+class GuildRoleDeleteAction extends Action {
+
+	constructor(client) {
+		super(client);
+		this.timeouts = [];
+		this.deleted = {};
+	}
+
+	handle(data) {
+		let client = this.client;
+		let guild = client.store.get('guilds', data.guild_id);
+
+		if (guild) {
+			let exists = guild.store.get('roles', data.role_id);
+			if (exists) {
+				guild.store.remove('roles', data.role_id);
+				this.deleted[guild.id + data.role_id] = exists;
+				this.scheduleForDeletion(guild.id, data.role_id);
+				client.emit(Constants.Events.GUILD_ROLE_DELETE, guild, exists);
+			}
+
+			if (!exists) {
+				exists = this.deleted[guild.id + data.role_id];
+			}
+
+			return {
+				role: exists,
+			};
+		}
+
+		return {
+			role: null,
+		};
+	}
+
+	scheduleForDeletion(guildID, roleID) {
+		this.timeouts.push(
+			setTimeout(() => delete this.deleted[guildID + roleID],
+			this.client.options.rest_ws_bridge_timeout)
+		);
+	}
+};
+
+module.exports = GuildRoleDeleteAction;

src/client/rest/RESTMethods.js

@@ -248,8 +248,8 @@ class RESTMethods{
 			this.rest.makeRequest('del', Constants.Endpoints.GUILD_MEMBER(guild.id, member.id), true)
 				.then(() => {
 					resolve(this.rest.client.actions.GuildMemberRemove.handle({
-						guild_id : guild.id,
+						guild_id: guild.id,
-						user : member.user,
+						user: member.user,
 					}).m);
 				})
 				.catch(reject);
@@ -261,13 +261,26 @@ class RESTMethods{
 			this.rest.makeRequest('post', Constants.Endpoints.GUILD_ROLES(guild.id), true)
 				.then(role => {
 					resolve(this.rest.client.actions.GuildRoleCreate.handle({
-						guild_id : guild.id,
+						guild_id: guild.id,
 						role,
 					}).role);
 				})
 				.catch(reject);
 		});
 	}
+
+	DeleteGuildRole(role) {
+		return new Promise((resolve, reject) => {
+			this.rest.makeRequest('del', Constants.Endpoints.GUILD_ROLE(role.guild.id, role.id), true)
+				.then(() => {
+					resolve(this.rest.client.actions.GuildRoleDelete.handle({
+						guild_id: role.guild.id,
+						role_id: role.id,
+					}).role);
+				})
+				.catch(reject);
+		});
+	}
 }
 
 module.exports = RESTMethods;

src/client/websocket/packets/handlers/GuildMemberRemove.js

@@ -21,10 +21,6 @@ class GuildMemberRemoveHandler extends AbstractHandler {
 		let client = this.packetManager.client;
 
 		let response = client.actions.GuildMemberRemove.handle(data);
-
-		if (response.m) {
-			client.emit(Constants.Events.GUILD_MEMBER_REMOVE, response.g, response.m);
-		}
 	}
 
 };

src/client/websocket/packets/handlers/GuildRoleDelete.js

@@ -17,16 +17,7 @@ class GuildRoleDeleteHandler extends AbstractHandler {
 		let data = packet.d;
 		let client = this.packetManager.client;
 
-		let guild = client.store.get('guilds', data.guild_id);
+		let response = client.actions.GuildRoleDelete.handle(data);
-
-		if (guild) {
-			let exists = guild.store.get('roles', data.role_id);
-			if (exists) {
-				guild.store.remove('roles', data.role_id);
-				client.emit(Constants.Events.GUILD_ROLE_DELETE, guild, exists);
-			}
-		}
-
 	}
 
 };

src/structures/Role.js

@@ -21,6 +21,10 @@ class Role {
 		this.managed = data.managed;
 	}
 
+	delete() {
+		return this.client.rest.methods.DeleteGuildRole(this);
+	}
+
 	serialize() {
 		let serializedPermissions = {};
 		for (let permissionName in Constants.PermissionFlags) {

src/util/Constants.js

@@ -55,6 +55,7 @@ const Endpoints = exports.Endpoints = {
 	GUILD_EMBED:        (guildID) => `${Endpoints.GUILD(guildID)}/embed`,
 	GUILD_INVITES:      (guildID) => `${Endpoints.GUILD(guildID)}/invites`,
 	GUILD_ROLES:        (guildID) => `${Endpoints.GUILD(guildID)}/roles`,
+	GUILD_ROLE:        (guildID, roleID) => `${Endpoints.GUILD_ROLES(guildID)}/${roleID}`,
 	GUILD_BANS:         (guildID) => `${Endpoints.GUILD(guildID)}/bans`,
 	GUILD_INTEGRATIONS: (guildID) => `${Endpoints.GUILD(guildID)}/integrations`,
 	GUILD_MEMBERS:      (guildID) => `${Endpoints.GUILD(guildID)}/members`,

test/random.js

@@ -41,6 +41,7 @@ client.on('guildMemberRemove', (guild, user) => {
 
 client.on('guildRoleCreate', (guild, role) => {
 	console.log('new role', role.name, 'in', guild.name);
+	role.delete().then(role => console.log("deleted " + role.name)).catch(console.log);
 });
 
 client.on('guildRoleDelete', (guild, role) => {