feat(MessageStore): add remove() (#2468)

* MessageStore#remove()

* typings

src/stores/MessageStore.js

@@ -82,6 +82,16 @@ class MessageStore extends DataStore {
     });
   }
 
+  /**
+   * Deletes a message, even if it's not cached.
+   * @param {MessageResolvable} message The message to delete
+   * @param {string} [reason] Reason for deleting this message, if it does not belong to the client user
+   */
+  async remove(message, reason) {
+    message = this.resolveID(message);
+    if (message) await this.client.api.channels(this.channel.id).messages(message).delete({ reason });
+  }
+
   async _fetchId(messageID, cache) {
     const existing = this.get(messageID);
     if (existing && !existing.partial) return existing;

src/structures/Message.js

@@ -454,13 +454,11 @@ class Message extends Base {
    */
   delete({ timeout = 0, reason } = {}) {
     if (timeout <= 0) {
-      return this.client.api.channels(this.channel.id).messages(this.id)
-        .delete({ reason })
-        .then(() =>
-          this.client.actions.MessageDelete.handle({
-            id: this.id,
-            channel_id: this.channel.id,
-          }).message);
+      return this.channel.messages.remove(this.id, reason).then(() =>
+        this.client.actions.MessageDelete.handle({
+          id: this.id,
+          channel_id: this.channel.id,
+        }).message);
     } else {
       return new Promise(resolve => {
         this.client.setTimeout(() => {

typings/index.d.ts

@@ -1448,6 +1448,7 @@ declare module 'discord.js' {
 		public fetch(message: Snowflake, cache?: boolean): Promise<Message>;
 		public fetch(options?: ChannelLogsQueryOptions, cache?: boolean): Promise<Collection<Snowflake, Message>>;
 		public fetchPinned(cache?: boolean): Promise<Collection<Snowflake, Message>>;
+		public remove(message: MessageResolvable, reason?: string): Promise<void>;
 	}
 
 	export class PresenceStore extends DataStore<Snowflake, Presence, typeof Presence, PresenceResolvable> {