From a7196dc969b98ed622efcfd1cce8740e739780ac Mon Sep 17 00:00:00 2001 From: Naiyar <137700126+imnaiyar@users.noreply.github.com> Date: Sat, 1 Nov 2025 03:35:51 +0600 Subject: [PATCH] fix: backport only passing relevant options to API when fetching (#11230) * fix: only pass relevant options to API when fetching * chore: update changes --- .../discord.js/src/managers/GuildBanManager.js | 6 +++--- packages/discord.js/src/managers/MessageManager.js | 14 +++++++------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/discord.js/src/managers/GuildBanManager.js b/packages/discord.js/src/managers/GuildBanManager.js index 97525e73a..74fde05ba 100644 --- a/packages/discord.js/src/managers/GuildBanManager.js +++ b/packages/discord.js/src/managers/GuildBanManager.js @@ -120,12 +120,12 @@ class GuildBanManager extends CachedManager { return this._add(data, cache); } - async _fetchMany(options = {}) { + async _fetchMany({ cache, ...apiOptions } = {}) { const data = await this.client.rest.get(Routes.guildBans(this.guild.id), { - query: makeURLSearchParams(options), + query: makeURLSearchParams(apiOptions), }); - return data.reduce((col, ban) => col.set(ban.user.id, this._add(ban, options.cache)), new Collection()); + return data.reduce((col, ban) => col.set(ban.user.id, this._add(ban, cache)), new Collection()); } /** diff --git a/packages/discord.js/src/managers/MessageManager.js b/packages/discord.js/src/managers/MessageManager.js index bd38a1a77..5e53d74be 100644 --- a/packages/discord.js/src/managers/MessageManager.js +++ b/packages/discord.js/src/managers/MessageManager.js @@ -111,12 +111,12 @@ class MessageManager extends CachedManager { return this._add(data, cache); } - async _fetchMany(options = {}) { + async _fetchMany({ cache, ...apiOptions } = {}) { const data = await this.client.rest.get(Routes.channelMessages(this.channel.id), { - query: makeURLSearchParams(options), + query: makeURLSearchParams(apiOptions), }); - return data.reduce((_data, message) => _data.set(message.id, this._add(message, options.cache)), new Collection()); + return data.reduce((_data, message) => _data.set(message.id, this._add(message, cache)), new Collection()); } /** @@ -158,11 +158,11 @@ class MessageManager extends CachedManager { * .then(messages => console.log(`Received ${messages.items.length} messages`)) * .catch(console.error); */ - async fetchPins(options = {}) { + async fetchPins({ cache, ...apiOptions } = {}) { const data = await this.client.rest.get(Routes.channelMessagesPins(this.channel.id), { query: makeURLSearchParams({ - ...options, - before: options.before && new Date(options.before).toISOString(), + ...apiOptions, + before: apiOptions.before && new Date(apiOptions.before).toISOString(), }), }); @@ -172,7 +172,7 @@ class MessageManager extends CachedManager { get pinnedAt() { return new Date(this.pinnedTimestamp); }, - message: this._add(item.message, options.cache), + message: this._add(item.message, cache), })), hasMore: data.has_more, };