From 31d914e44b77ffd0d4511b5159e6869c04e8b1ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aura=20Rom=C3=A1n?= <kyradiscord@gmail.com>
Date: Sun, 5 Nov 2023 19:23:32 +0100
Subject: [PATCH] fix(Webhook): do not call `client.deleteWebhook` in `delete`
 (#9786)

* fix(Webhook): do not call `client.deleteWebhook` in `delete`

Partially reverts #9777 which caused a regression (#9785) when using `WebhookClient`

* chore: add comment

Co-Authored-By: Souji <timoqueezle@gmail.com>

* fix: move `deleteWebhook` from `Client` to `BaseClient`

---------

Co-authored-by: Souji <timoqueezle@gmail.com>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
---
 packages/discord.js/src/client/BaseClient.js | 18 ++++++++++++++++++
 packages/discord.js/src/client/Client.js     | 17 -----------------
 2 files changed, 18 insertions(+), 17 deletions(-)

diff --git a/packages/discord.js/src/client/BaseClient.js b/packages/discord.js/src/client/BaseClient.js
index 631748c92..5f8e492f0 100644
--- a/packages/discord.js/src/client/BaseClient.js
+++ b/packages/discord.js/src/client/BaseClient.js
@@ -2,6 +2,7 @@
 
 const EventEmitter = require('node:events');
 const { REST } = require('@discordjs/rest');
+const { Routes } = require('discord-api-types/v10');
 const { DiscordjsTypeError, ErrorCodes } = require('../errors');
 const Options = require('../util/Options');
 const { mergeDefault, flatten } = require('../util/Util');
@@ -48,6 +49,23 @@ class BaseClient extends EventEmitter {
     this.rest.clearHandlerSweeper();
   }
 
+  /**
+   * Options used for deleting a webhook.
+   * @typedef {Object} WebhookDeleteOptions
+   * @property {string} [token] Token of the webhook
+   * @property {string} [reason] The reason for deleting the webhook
+   */
+
+  /**
+   * Deletes a webhook.
+   * @param {Snowflake} id The webhook's id
+   * @param {WebhookDeleteOptions} [options] Options for deleting the webhook
+   * @returns {Promise<void>}
+   */
+  async deleteWebhook(id, { token, reason } = {}) {
+    await this.rest.delete(Routes.webhook(id, token), { auth: !token, reason });
+  }
+
   /**
    * Increments max listeners by one, if they are not zero.
    * @private
diff --git a/packages/discord.js/src/client/Client.js b/packages/discord.js/src/client/Client.js
index 92b23cbf6..20fdff4f4 100644
--- a/packages/discord.js/src/client/Client.js
+++ b/packages/discord.js/src/client/Client.js
@@ -255,23 +255,6 @@ class Client extends BaseClient {
     this.rest.setToken(null);
   }
 
-  /**
-   * Options used for deleting a webhook.
-   * @typedef {Object} WebhookDeleteOptions
-   * @property {string} [token] Token of the webhook
-   * @property {string} [reason] The reason for deleting the webhook
-   */
-
-  /**
-   * Deletes a webhook.
-   * @param {Snowflake} id The webhook's id
-   * @param {WebhookDeleteOptions} [options] Options for deleting the webhook
-   * @returns {Promise<void>}
-   */
-  async deleteWebhook(id, { token, reason } = {}) {
-    await this.rest.delete(Routes.webhook(id, token), { auth: !token, reason });
-  }
-
   /**
    * Options used when fetching an invite from Discord.
    * @typedef {Object} ClientFetchInviteOptions