Rename TrackRequest and TrackRequestOptions to DownloadRequest …

…and `DownloadRequestOptions`
DJDoubleD · Oct 21, 2024 · 68e12a8 · 68e12a8
1 parent 566529e
commit 68e12a8
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 15 deletions.
diff --git a/...wnloaderX/Models/Download/TrackRequest.cs → QobuzDownloaderX/Shared/DownloadRequest.cs b/...wnloaderX/Models/Download/TrackRequest.cs → QobuzDownloaderX/Shared/DownloadRequest.cs
@@ -5,9 +5,9 @@
 using QobuzApiSharp.Exceptions;
 using QobuzApiSharp.Models.Content;
 using QobuzApiSharp.Service;
+using QobuzDownloaderX.Models;
 using QobuzDownloaderX.Models.Content;
 using QobuzDownloaderX.Properties;
-using QobuzDownloaderX.Shared;
 using Requests;
 using Requests.Options;
 using System;
@@ -18,9 +18,9 @@
 using System.Linq;
 using System.Threading.Tasks;
 
-namespace QobuzDownloaderX.Models.Download
+namespace QobuzDownloaderX.Shared
 {
-    internal class TrackRequest : Request<TrackRequestOptions, DownloadItem, DownloadItem>, ISpeedReportable
+    internal class DownloadRequest : Request<DownloadRequestOptions, DownloadItem, DownloadItem>, ISpeedReportable
     {
         private readonly DownloadLogger _logger;
         private readonly DownloadItem _downloadItem;
@@ -35,7 +35,7 @@ internal class TrackRequest : Request<TrackRequestOptions, DownloadItem, Downloa
 
         public SpeedReporter<long> SpeedReporter => _requestContainer.SpeedReporter;
 
-        public TrackRequest(TrackRequestOptions options)
+        public DownloadRequest(DownloadRequestOptions options)
             : base(options)
         {
             _logger = options.Logger;
@@ -188,7 +188,7 @@ private bool DownloadTrack(Track qobuzTrack, string basePath, bool isPartOfTrack
             if (removeTagArtFileAfterDownload)
             {
                 Notify<IRequest, DownloadItem> x = (_, _) => RemoveTempTaggingArtFile(key);
-                Options.RequestCompleated = (Notify<IRequest, DownloadItem>)(Delegate.Combine(Options.RequestCompleated, x));
+                Options.RequestCompleated = (Notify<IRequest, DownloadItem>)Delegate.Combine(Options.RequestCompleated, x);
             }
             return true;
         }
@@ -304,15 +304,15 @@ private bool DownloadAlbum(Album qobuzAlbum, string basePath, string albumPathSu
             {
                 if (State != RequestState.Running && State != RequestState.Idle)
                     return false;
-                bool isLastTrackOfAlbum = (i + tracksPageOffset) == (tracksTotal - 1);
+                bool isLastTrackOfAlbum = i + tracksPageOffset == tracksTotal - 1;
                 Track qobuzTrack = qobuzAlbum.Tracks.Items[i];
                 qobuzTrack.Album = qobuzAlbum;
 
                 if (!DownloadTrack(qobuzTrack, basePath, false, isLastTrackOfAlbum, albumPathSuffix))
                     noErrorsOccured = false;
 
                 i++;
-                if (i == tracksLoaded && tracksTotal > (i + tracksPageOffset))
+                if (i == tracksLoaded && tracksTotal > i + tracksPageOffset)
                 {
                     tracksPageOffset += tracksLimit;
                     qobuzAlbum = ExecuteApiCall(apiService => apiService.GetAlbum(qobuzAlbum.Id, true, null, tracksLimit, tracksPageOffset));

diff --git a/...rX/Models/Download/TrackRequestOptions.cs → ...nloaderX/Shared/DownloadRequestOptions.cs b/...rX/Models/Download/TrackRequestOptions.cs → ...nloaderX/Shared/DownloadRequestOptions.cs
@@ -1,9 +1,9 @@
-using QobuzDownloaderX.Shared;
+using QobuzDownloaderX.Models;
 using Requests.Options;
 
-namespace QobuzDownloaderX.Models.Download
+namespace QobuzDownloaderX.Shared
 {
-    public record TrackRequestOptions : RequestOptions<DownloadItem, DownloadItem>
+    public record DownloadRequestOptions : RequestOptions<DownloadItem, DownloadItem>
     {
         public DownloadLogger Logger { get; init; }
         public DownloadItem DownloadItem { get; init; }
@@ -14,14 +14,14 @@ public record TrackRequestOptions : RequestOptions<DownloadItem, DownloadItem>
         /// <summary>
         /// Main Constructor
         /// </summary>
-        public TrackRequestOptions()
+        public DownloadRequestOptions()
         { }
 
         /// <summary>
         /// Copy Constructor
         /// </summary>
         /// <param name="options">Copied object</param>
-        protected TrackRequestOptions(TrackRequestOptions options) : base(options)
+        protected DownloadRequestOptions(DownloadRequestOptions options) : base(options)
         {
             Logger = options.Logger;
             DownloadItem = options.DownloadItem;

diff --git a/QobuzDownloaderX/View/MainForm.cs b/QobuzDownloaderX/View/MainForm.cs
@@ -1,7 +1,6 @@
 using DownloadAssistant.Requests;
 using Newtonsoft.Json.Linq;
 using QobuzDownloaderX.Models;
-using QobuzDownloaderX.Models.Download;
 using QobuzDownloaderX.Properties;
 using QobuzDownloaderX.Shared;
 using QobuzDownloaderX.View;
@@ -39,7 +38,7 @@ public QobuzDownloaderX()
             };
         }
 
-        private readonly ExtendedContainer<TrackRequest> _requests = [];
+        private readonly ExtendedContainer<DownloadRequest> _requests = [];
         public string DownloadLogPath { get; set; }
 
         public int DevClickEggThingValue { get; set; }
@@ -299,7 +298,7 @@ public void StartLinkItemDownload(params string[] downloadLinks)
                     output.Invoke(new Action(() => output.AppendText($"URL >{downloadItem.Url}< not understood. Is there a typo?\r\n")));
 
                 else
-                    _requests.Add(new TrackRequest(new()
+                    _requests.Add(new DownloadRequest(new()
                     {
                         DownloadItem = downloadItem,
                         UpdateAlbumTagsUi = UpdateAlbumTagsUI,