Fixed: Completed downloads in Qbit missing import path

(cherry picked from commit 35365665cfd436ac276dd9591e23333bd26cf789)

Closes #4162
pull/4172/head
Mark McDowall 8 months ago committed by Bogdan
parent 269ea5597f
commit 2bfc6ae5d6

@ -137,7 +137,7 @@ namespace NzbDrone.Core.Test.Download.DownloadClientTests.QBittorrentTests
{
Mocker.GetMock<IQBittorrentProxy>()
.Setup(s => s.GetTorrentProperties(torrent.Hash.ToLower(), It.IsAny<QBittorrentSettings>()))
.Returns(new QBittorrentTorrentProperties { ContentPath = torrent.ContentPath, SavePath = torrent.SavePath });
.Returns(new QBittorrentTorrentProperties { SavePath = torrent.SavePath });
Mocker.GetMock<IQBittorrentProxy>()
.Setup(s => s.GetTorrentFiles(torrent.Hash.ToLower(), It.IsAny<QBittorrentSettings>()))

@ -302,6 +302,19 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
break;
}
if (version >= new Version("2.6.1") && item.Status == DownloadItemStatus.Completed)
{
if (torrent.ContentPath != torrent.SavePath)
{
item.OutputPath = _remotePathMappingService.RemapRemoteToLocal(Settings.Host, new OsPath(torrent.ContentPath));
}
else
{
item.Status = DownloadItemStatus.Warning;
item.Message = "Unable to Import. Path matches client base download directory, it's possible 'Keep top-level folder' is disabled for this torrent or 'Torrent Content Layout' is NOT set to 'Original' or 'Create Subfolder'?";
}
}
queueItems.Add(item);
}
@ -315,22 +328,9 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
public override DownloadClientItem GetImportItem(DownloadClientItem item, DownloadClientItem previousImportAttempt)
{
var properties = Proxy.GetTorrentProperties(item.DownloadId.ToLower(), Settings);
var savePath = new OsPath(properties.SavePath);
var version = Proxy.GetApiVersion(Settings);
if (version >= new Version("2.6.1"))
// On API version >= 2.6.1 this is already set correctly
if (!item.OutputPath.IsEmpty)
{
if (properties.ContentPath != savePath.ToString())
{
item.OutputPath = _remotePathMappingService.RemapRemoteToLocal(Settings.Host, new OsPath(properties.ContentPath));
}
else
{
item.Status = DownloadItemStatus.Warning;
item.Message = "Unable to Import. Path matches client base download directory, it's possible 'Keep top-level folder' is disabled for this torrent or 'Torrent Content Layout' is NOT set to 'Original' or 'Create Subfolder'?";
}
return item;
}
@ -341,6 +341,11 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
return item;
}
var properties = Proxy.GetTorrentProperties(item.DownloadId.ToLower(), Settings);
var savePath = new OsPath(properties.SavePath);
var result = item.Clone();
// get the first subdirectory - QBittorrent returns `/` path separators even on windows...
var relativePath = new OsPath(files[0].Name);
while (!relativePath.Directory.IsEmpty)
@ -349,9 +354,10 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
}
var outputPath = savePath + relativePath.FileName;
item.OutputPath = _remotePathMappingService.RemapRemoteToLocal(Settings.Host, outputPath);
return item;
result.OutputPath = _remotePathMappingService.RemapRemoteToLocal(Settings.Host, outputPath);
return result;
}
public override DownloadClientInfo GetStatus()

@ -48,9 +48,6 @@ namespace NzbDrone.Core.Download.Clients.QBittorrent
[JsonProperty(PropertyName = "seeding_time")]
public long SeedingTime { get; set; } // Torrent seeding time (in seconds)
[JsonProperty(PropertyName = "content_path")]
public string ContentPath { get; set; } // Torrent save path
}
public class QBittorrentTorrentFile

Loading…
Cancel
Save