github/Discord.Net
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix flawed bulk message deletion (#872)

Browse Source 
* Fix flawed bulk message deletion

https://github.com/RogueException/Discord.Net/issues/871, consider changing DeleteMessagesParams.MessageIds type to I(Readonly)List<ulong> or IEnumerable<ulong> to avoid unnecessary copying (batch.ToArray())

* Update code formatting
This commit is contained in:
BinkanSalaryman
2018-01-07 04:33:00 +01:00
committed by Christopher F
parent cd82a0f70c
commit 804d9188e7
1 changed files with 17 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/Discord.Net.Rest/Entities/Channels/ChannelHelper.cs
View File

@@ -188,21 +188,27 @@ namespace Discord.Rest
public static async Task DeleteMessagesAsync(ITextChannel channel, BaseDiscordClient client,
IEnumerable<ulong> messageIds, RequestOptions options)
{
const int BATCH_SIZE = 100;
var msgs = messageIds.ToArray();
if (msgs.Length < 100)
int batches = msgs.Length / BATCH_SIZE;
for (int i = 0; i <= batches; i++)
{
var args = new DeleteMessagesParams(msgs);
await client.ApiClient.DeleteMessagesAsync(channel.Id, args, options).ConfigureAwait(false);
}
else
{
var batch = new ulong[100];
for (int i = 0; i < (msgs.Length + 99) / 100; i++)
ArraySegment<ulong> batch;
if (i < batches)
{
Array.Copy(msgs, i * 100, batch, 0, Math.Min(msgs.Length - (100 * i), 100));
var args = new DeleteMessagesParams(batch);
await client.ApiClient.DeleteMessagesAsync(channel.Id, args, options).ConfigureAwait(false);
batch = new ArraySegment<ulong>(msgs, i * BATCH_SIZE, BATCH_SIZE);
}
else
{
batch = new ArraySegment<ulong>(msgs, i * BATCH_SIZE, msgs.Length - batches * BATCH_SIZE);
if (batch.Count == 0)
{
break;
}
}
var args = new DeleteMessagesParams(batch.ToArray());
await client.ApiClient.DeleteMessagesAsync(channel.Id, args, options).ConfigureAwait(false);
}
}