Minor refactor of Actions

src/client/actions/ChannelCreate.js

@@ -4,9 +4,7 @@ class ChannelCreateAction extends Action {
   handle(data) {
     const client = this.client;
     const channel = client.dataManager.newChannel(data);
-    return {
+    return { channel };
-      channel,
-    };
   }
 }
 

src/client/actions/ChannelDelete.js

@@ -18,9 +18,7 @@ class ChannelDeleteAction extends Action {
       channel = this.deleted.get(data.id) || null;
     }
 
-    return {
+    return { channel };
-      channel,
-    };
   }
 
   scheduleForDeletion(id) {

src/client/actions/GuildChannelsPositionUpdate.js

@@ -12,9 +12,7 @@ class GuildChannelsPositionUpdate extends Action {
       }
     }
 
-    return {
+    return { guild };
-      guild,
-    };
   }
 }
 

src/client/actions/GuildDelete.js

@@ -36,9 +36,7 @@ class GuildDeleteAction extends Action {
       guild = this.deleted.get(data.id) || null;
     }
 
-    return {
+    return { guild };
-      guild,
-    };
   }
 
   scheduleForDeletion(id) {

src/client/actions/GuildEmojiCreate.js

@@ -4,9 +4,7 @@ class GuildEmojiCreateAction extends Action {
   handle(guild, createdEmoji) {
     const client = this.client;
     const emoji = client.dataManager.newEmoji(createdEmoji, guild);
-    return {
+    return { emoji };
-      emoji,
-    };
   }
 }
 

src/client/actions/GuildEmojiDelete.js

@@ -4,9 +4,7 @@ class GuildEmojiDeleteAction extends Action {
   handle(emoji) {
     const client = this.client;
     client.dataManager.killEmoji(emoji);
-    return {
+    return { emoji };
-      emoji,
-    };
   }
 }
 

src/client/actions/GuildEmojiUpdate.js

@@ -3,9 +3,7 @@ const Action = require('./Action');
 class GuildEmojiUpdateAction extends Action {
   handle(oldEmoji, newEmoji) {
     const emoji = this.client.dataManager.updateEmoji(oldEmoji, newEmoji);
-    return {
+    return { emoji };
-      emoji,
-    };
   }
 }
 

src/client/actions/GuildMemberGet.js

@@ -3,9 +3,7 @@ const Action = require('./Action');
 class GuildMemberGetAction extends Action {
   handle(guild, data) {
     const member = guild._addMember(data, false);
-    return {
+    return { member };
-      member,
-    };
   }
 }
 

src/client/actions/GuildMemberRemove.js

@@ -9,10 +9,10 @@ class GuildMemberRemoveAction extends Action {
 
   handle(data) {
     const client = this.client;
-
     const guild = client.guilds.get(data.guild_id);
+    let member = null;
     if (guild) {
-      let member = guild.members.get(data.user.id);
+      member = guild.members.get(data.user.id);
       if (member) {
         guild.memberCount--;
         guild._removeMember(member);
@@ -22,17 +22,8 @@ class GuildMemberRemoveAction extends Action {
       } else {
         member = this.deleted.get(guild.id + data.user.id) || null;
       }
-
-      return {
-        guild,
-        member,
-      };
     }
-
+    return { guild, member };
-    return {
-      guild,
-      member: null,
-    };
   }
 
   scheduleForDeletion(guildID, userID) {

src/client/actions/GuildRoleCreate.js

@@ -5,22 +5,15 @@ const Role = require('../../structures/Role');
 class GuildRoleCreate extends Action {
   handle(data) {
     const client = this.client;
-
     const guild = client.guilds.get(data.guild_id);
+    let role;
     if (guild) {
       const already = guild.roles.has(data.role.id);
-      const role = new Role(guild, data.role);
+      role = new Role(guild, data.role);
       guild.roles.set(role.id, role);
       if (!already) client.emit(Constants.Events.GUILD_ROLE_CREATE, role);
-
-      return {
-        role,
-      };
     }
-
+    return { role };
-    return {
-      role: null,
-    };
   }
 }
 

src/client/actions/GuildRoleDelete.js

@@ -9,10 +9,11 @@ class GuildRoleDeleteAction extends Action {
 
   handle(data) {
     const client = this.client;
-
     const guild = client.guilds.get(data.guild_id);
+    let role;
+
     if (guild) {
-      let role = guild.roles.get(data.role_id);
+      role = guild.roles.get(data.role_id);
       if (role) {
         guild.roles.delete(data.role_id);
         this.deleted.set(guild.id + data.role_id, role);
@@ -21,15 +22,9 @@ class GuildRoleDeleteAction extends Action {
       } else {
         role = this.deleted.get(guild.id + data.role_id) || null;
       }
-
-      return {
-        role,
-      };
     }
 
-    return {
+    return { role };
-      role: null,
-    };
   }
 
   scheduleForDeletion(guildID, roleID) {

src/client/actions/GuildRoleUpdate.js

@@ -5,8 +5,8 @@ const Util = require('../../util/Util');
 class GuildRoleUpdateAction extends Action {
   handle(data) {
     const client = this.client;
-
     const guild = client.guilds.get(data.guild_id);
+    
     if (guild) {
       const roleData = data.role;
       let oldRole = null;

src/client/actions/GuildRolesPositionUpdate.js

@@ -12,9 +12,7 @@ class GuildRolesPositionUpdate extends Action {
       }
     }
 
-    return {
+    return { guild };
-      guild,
-    };
   }
 }
 

src/client/actions/MessageDelete.js

@@ -8,11 +8,11 @@ class MessageDeleteAction extends Action {
 
   handle(data) {
     const client = this.client;
-
     const channel = client.channels.get(data.channel_id);
-    if (channel) {
+    let message;
-      let message = channel.messages.get(data.id);
 
+    if (channel) {
+      message = channel.messages.get(data.id);
       if (message) {
         channel.messages.delete(message.id);
         this.deleted.set(channel.id + message.id, message);
@@ -20,15 +20,9 @@ class MessageDeleteAction extends Action {
       } else {
         message = this.deleted.get(channel.id + data.id) || null;
       }
-
-      return {
-        message,
-      };
     }
 
-    return {
+    return { message };
-      message: null,
-    };
   }
 
   scheduleForDeletion(channelID, messageID) {

src/client/actions/MessageDeleteBulk.js

@@ -15,9 +15,7 @@ class MessageDeleteBulkAction extends Action {
     }
 
     if (messages.size > 0) client.emit(Constants.Events.MESSAGE_BULK_DELETE, messages);
-    return {
+    return { messages };
-      messages,
-    };
   }
 }
 

src/client/actions/MessageReactionAdd.js

@@ -12,26 +12,18 @@ class MessageReactionAdd extends Action {
   handle(data) {
     const user = this.client.users.get(data.user_id);
     if (!user) return false;
-
+    // Verify channel
     const channel = this.client.channels.get(data.channel_id);
     if (!channel || channel.type === 'voice') return false;
-
+    // Verify message
     const message = channel.messages.get(data.message_id);
     if (!message) return false;
-
     if (!data.emoji) return false;
-
+    // Verify reaction
     const reaction = message._addReaction(data.emoji, user);
+    if (reaction) this.client.emit(Constants.Events.MESSAGE_REACTION_ADD, reaction, user);
 
-    if (reaction) {
+    return { message, reaction, user };
-      this.client.emit(Constants.Events.MESSAGE_REACTION_ADD, reaction, user);
-    }
-
-    return {
-      message,
-      reaction,
-      user,
-    };
   }
 }
 

src/client/actions/MessageReactionRemove.js

@@ -12,26 +12,18 @@ class MessageReactionRemove extends Action {
   handle(data) {
     const user = this.client.users.get(data.user_id);
     if (!user) return false;
-
+    // Verify channel
     const channel = this.client.channels.get(data.channel_id);
     if (!channel || channel.type === 'voice') return false;
-
+    // Verify message
     const message = channel.messages.get(data.message_id);
     if (!message) return false;
-
     if (!data.emoji) return false;
-
+    // Verify reaction
     const reaction = message._removeReaction(data.emoji, user);
+    if (reaction) this.client.emit(Constants.Events.MESSAGE_REACTION_REMOVE, reaction, user);
 
-    if (reaction) {
+    return { message, reaction, user };
-      this.client.emit(Constants.Events.MESSAGE_REACTION_REMOVE, reaction, user);
-    }
-
-    return {
-      message,
-      reaction,
-      user,
-    };
   }
 }
 

src/client/actions/MessageReactionRemoveAll.js

@@ -12,9 +12,7 @@ class MessageReactionRemoveAll extends Action {
     message._clearReactions();
     this.client.emit(Constants.Events.MESSAGE_REACTION_REMOVE_ALL, message);
 
-    return {
+    return { message };
-      message,
-    };
   }
 }
 

src/client/actions/UserGet.js

@@ -4,9 +4,7 @@ class UserGetAction extends Action {
   handle(data) {
     const client = this.client;
     const user = client.dataManager.newUser(data);
-    return {
+    return { user };
-      user,
-    };
   }
 }