backport: TextChannel#bulkDelete accepts Snowflake[]

This commit is contained in:
Lewdcario
2018-08-08 17:14:52 -05:00
parent af6d649510
commit c76f3048af
2 changed files with 11 additions and 11 deletions

View File

@@ -179,13 +179,12 @@ class RESTMethods {
} }
bulkDeleteMessages(channel, messages) { bulkDeleteMessages(channel, messages) {
const ids = messages.map(m => m.id);
return this.rest.makeRequest('post', Endpoints.Channel(channel).messages.bulkDelete, true, { return this.rest.makeRequest('post', Endpoints.Channel(channel).messages.bulkDelete, true, {
messages: ids, messages: messages,
}).then(() => }).then(() =>
this.client.actions.MessageDeleteBulk.handle({ this.client.actions.MessageDeleteBulk.handle({
channel_id: channel.id, channel_id: channel.id,
ids, ids: messages,
}).messages }).messages
); );
} }

View File

@@ -452,7 +452,8 @@ class TextBasedChannel {
/** /**
* Bulk delete given messages that are newer than two weeks. * Bulk delete given messages that are newer than two weeks.
* <warn>This is only available when using a bot account.</warn> * <warn>This is only available when using a bot account.</warn>
* @param {Collection<Snowflake, Message>|Message[]|number} messages Messages or number of messages to delete * @param {Collection<Snowflake, Message>|Message[]|Snowflake[]|number} messages
* Messages or number of messages to delete
* @param {boolean} [filterOld=false] Filter messages to remove those which are older than two weeks automatically * @param {boolean} [filterOld=false] Filter messages to remove those which are older than two weeks automatically
* @returns {Promise<Collection<Snowflake, Message>>} Deleted messages * @returns {Promise<Collection<Snowflake, Message>>} Deleted messages
* @example * @example
@@ -462,16 +463,16 @@ class TextBasedChannel {
* .catch(console.error); * .catch(console.error);
*/ */
bulkDelete(messages, filterOld = false) { bulkDelete(messages, filterOld = false) {
if (messages instanceof Collection) messages = [...messages.values()]; if (messages instanceof Array || messages instanceof Collection) {
if (messages instanceof Array) { let messageIDs = messages instanceof Collection ? messages.keyArray() : messages.map(m => m.id || m);
if (filterOld) { if (filterOld) {
messages = messages.filter(m => Date.now() - Snowflake.deconstruct(m.id).date.getTime() < 1209600000); messageIDs = messageIDs.filter(id => Date.now() - Snowflake.deconstruct(id).date.getTime() < 1209600000);
} }
if (messages.length === 0) return Promise.resolve(new Collection()); if (messageIDs.length === 0) return Promise.resolve(new Collection());
if (messages.length === 1) { if (messageIDs.length === 1) {
return messages[0].delete().then(() => new Collection([[messages[0].id, messages[0]]])); return this.fetchMessage(messageIDs[0]).then(m => m.delete()).then(m => new Collection([[m.id, m]]));
} }
return this.client.rest.methods.bulkDeleteMessages(this, messages); return this.client.rest.methods.bulkDeleteMessages(this, messageIDs);
} }
if (!isNaN(messages)) return this.fetchMessages({ limit: messages }).then(msgs => this.bulkDelete(msgs, filterOld)); if (!isNaN(messages)) return this.fetchMessages({ limit: messages }).then(msgs => this.bulkDelete(msgs, filterOld));
throw new TypeError('The messages must be an Array, Collection, or number.'); throw new TypeError('The messages must be an Array, Collection, or number.');