Add some safeguarding in case partition limit is less or equal to zero

This commit is contained in:
Oleksii Holub 2018-11-15 10:26:54 +02:00
parent beadc25e61
commit d620010e85

View file

@ -57,7 +57,7 @@ namespace DiscordChatExporter.Core.Services
} }
} }
private void ExportChatLogPartitions(IReadOnlyList<ChatLog> partitions, string filePath, ExportFormat format) private void ExportChatLogPartitioned(IReadOnlyList<ChatLog> partitions, string filePath, ExportFormat format)
{ {
// Split file path into components // Split file path into components
var dirPath = Path.GetDirectoryName(filePath); var dirPath = Path.GetDirectoryName(filePath);
@ -115,7 +115,7 @@ namespace DiscordChatExporter.Core.Services
int? partitionLimit = null) int? partitionLimit = null)
{ {
// If partitioning is disabled or there are fewer messages in chat log than the limit - process it without partitioning // If partitioning is disabled or there are fewer messages in chat log than the limit - process it without partitioning
if (partitionLimit == null || chatLog.Messages.Count <= partitionLimit) if (partitionLimit == null || partitionLimit <= 0 || chatLog.Messages.Count <= partitionLimit)
{ {
ExportChatLogSingle(chatLog, filePath, format); ExportChatLogSingle(chatLog, filePath, format);
} }
@ -123,7 +123,7 @@ namespace DiscordChatExporter.Core.Services
else else
{ {
var partitions = SplitIntoPartitions(chatLog, partitionLimit.Value); var partitions = SplitIntoPartitions(chatLog, partitionLimit.Value);
ExportChatLogPartitions(partitions, filePath, format); ExportChatLogPartitioned(partitions, filePath, format);
} }
} }
} }