github/lidarr
1
0
Fork 0
mirror of https://github.com/lidarr/lidarr.git synced 2025-08-21 05:53:33 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fixed timing issue allowing multiple instances of the same command to be queued

Browse source
This commit is contained in:
Taloth Saldono 2020-05-03 16:33:52 +02:00 • committed by Qstick
commit 9f6e3d388b
1 changed files with 26 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/NzbDrone.Core/Messaging/Commands/CommandQueueManager.cs
View file

@ -100,8 +100,10 @@ namespace NzbDrone.Core.Messaging.Commands
_logger.Trace("Publishing {0}", command.Name); _logger.Trace("Publishing {0}", command.Name);
_logger.Trace("Checking if command is queued or started: {0}", command.Name); _logger.Trace("Checking if command is queued or started: {0}", command.Name);
lock (_commandQueue)
{
var existingCommands = QueuedOrStarted(command.Name); var existingCommands = QueuedOrStarted(command.Name);
var existing = existingCommands.SingleOrDefault(c => CommandEqualityComparer.Instance.Equals(c.Body, command)); var existing = existingCommands.FirstOrDefault(c => CommandEqualityComparer.Instance.Equals(c.Body, command));
if (existing != null) if (existing != null)
{ {
@ -127,6 +129,7 @@ namespace NzbDrone.Core.Messaging.Commands
return commandModel; return commandModel;
} }
}
public CommandModel Push(string commandName, DateTime? lastExecutionTime, DateTime? lastStartTime, CommandPriority priority = CommandPriority.Normal, CommandTrigger trigger = CommandTrigger.Unspecified) public CommandModel Push(string commandName, DateTime? lastExecutionTime, DateTime? lastStartTime, CommandPriority priority = CommandPriority.Normal, CommandTrigger trigger = CommandTrigger.Unspecified)
{ {