Noi/BirthdayBot
1
0
Fork 0
mirror of https://github.com/NoiTheCat/BirthdayBot.git synced 2024-10-16 15:59:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Slightly improve birthday update task handling

Browse source
This commit is contained in:
Noi 2020-04-03 20:58:02 -07:00
parent 670309715b
commit c5cab92899
1 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
BackgroundServices/BirthdayRoleUpdate.cs
View file

@ -25,25 +25,25 @@ namespace BirthdayBot.BackgroundServices
var tasks = new List<Task>(); var tasks = new List<Task>();
foreach (var guild in BotInstance.DiscordClient.Guilds) foreach (var guild in BotInstance.DiscordClient.Guilds)
{ {
var t = ProcessGuildAsync(guild); tasks.Add(ProcessGuildAsync(guild));
tasks.Add(t);
} }
var alltasks = Task.WhenAll(tasks);
try try
{ {
await Task.WhenAll(tasks); await alltasks;
} }
catch (Exception ex) catch (Exception ex)
{ {
// TODO does this not actually work as might be expected? var exs = alltasks.Exception;
var exs = from task in tasks if (exs != null)
where task.Exception != null
select task.Exception;
Log($"Encountered {exs.Count()} errors during bulk guild processing.");
foreach (var iex in exs)
{ {
// TODO probably not a good idea Log($"{exs.InnerExceptions.Count} exception(s) during bulk processing!");
Log(iex.ToString()); // TODO needs major improvements. output to file?
}
else
{
Log(ex.ToString());
} }
} }