fix: solve some errors 🚑

Author: mateof

TelegramDownloader/Pages/Partials/SpeedChart.razor

@@ -46,7 +46,10 @@
 
     private async Task UpdateChartData(bool isInitial = false)
     {
-        var downloadData = tis.downloadSpeedsHistory
+        var downloadHistory = tis.GetDownloadSpeedsHistoryCopy();
+        var uploadHistory = tis.GetUploadSpeedsHistoryCopy();
+
+        var downloadData = downloadHistory
             .Select(x => new {
                 time = x.time.ToString("HH:mm:ss"),
                 datetime = x.time.ToString("dd/MM/yyyy HH:mm:ss"),
@@ -55,7 +58,7 @@
             })
             .ToList();
 
-        var uploadData = tis.uploadSpeedsHistory
+        var uploadData = uploadHistory
             .Select(x => new {
                 time = x.time.ToString("HH:mm:ss"),
                 datetime = x.time.ToString("dd/MM/yyyy HH:mm:ss"),

TelegramDownloader/Pages/Partials/impl/FileManagerImpl.razor

@@ -100,6 +100,7 @@
             await LoadDataAsync();
             if (_initialized && fm != null)
             {
+                fm.Path = "/";
                 await fm.RefreshFilesAsync();
             }
             _initialized = true;
@@ -275,11 +276,11 @@
     {
         BsonFileManagerModel item = null;
         if (isShared)
-            item = await fs.getSharedItemById(id: idFile, collection: id);
+            item = await fs.getSharedItemById(id: idFile, collection: bsi.CollectionId);
         else
             item = await fs.getItemById(id, idFile);
         string localdir = MyNavigationManager.BaseUri;
-        if (forcePreDownload || (HelperService.bytesToMegaBytes(size) <= GeneralConfigStatic.config.MaxPreloadFileSizeInMb && item.MessageId != null))
+        if (forcePreDownload || (HelperService.bytesToMegaBytes(size) <= GeneralConfigStatic.config.MaxPreloadFileSizeInMb && item != null && item.MessageId != null))
         {
             return new System.Uri(Path.Combine(localdir, "api/file/GetFile", name).Replace("\\", "/") + $"?idChannel={(isShared ? bsi.ChannelId : id)}&idFile={item.MessageId}").AbsoluteUri;
         }

TelegramDownloader/Pages/Partials/impl/LocalFileManagerImpl.razor

@@ -88,6 +88,7 @@
             await LoadDataAsync();
             if (_initialized && FileManager != null)
             {
+                FileManager.Path = "/";
                 await FileManager.RefreshFilesAsync();
             }
             _initialized = true;

TelegramDownloader/Services/TransactionInfoService.cs

@@ -25,6 +25,7 @@ public class TransactionInfoService
         public long bytesDownloaded = 0;
         public List<SpeedHistory> downloadSpeedsHistory = new List<SpeedHistory>();
         public List<SpeedHistory> uploadSpeedsHistory = new List<SpeedHistory>();
+        private readonly object _speedHistoryLock = new object();
         private int speedHistoryCount = 0;
 
         private static Mutex PendingDownloadMutex = new Mutex();
@@ -123,22 +124,44 @@ private void OnTimedEvent(Object source, ElapsedEventArgs e)
 
         private bool canStopTimer()
         {
-            return uploadSpeedsHistory.Sum(x => x.speed) + downloadSpeedsHistory.Sum(x => x.speed) == 0;
+            lock (_speedHistoryLock)
+            {
+                return uploadSpeedsHistory.Sum(x => x.speed) + downloadSpeedsHistory.Sum(x => x.speed) == 0;
+            }
         }
 
         private void setHistorySpeed()
         {
             DateTime now = DateTime.Now;
             var activeDownloads = downloadModels.Where(x => x.state == StateTask.Working).Select(x => x.name).ToList();
             var activeUploads = uploadModels.Where(x => x.state == StateTask.Working).Select(x => x.name).ToList();
-            downloadSpeedsHistory.Add(new SpeedHistory() { time = now, speed = bytesDownloaded, speedString = downloadSpeed, activeFiles = activeDownloads });
-            uploadSpeedsHistory.Add(new SpeedHistory() { time = now, speed = bytesUploaded, speedString = uploadSpeed, activeFiles = activeUploads });
-            downloadSpeedsHistory.RemoveAll(x => (now - x.time).TotalSeconds > MAX_SPEED_HISTORY_SECONDS);
-            uploadSpeedsHistory.RemoveAll(x => (now - x.time).TotalSeconds > MAX_SPEED_HISTORY_SECONDS);
+            lock (_speedHistoryLock)
+            {
+                downloadSpeedsHistory.Add(new SpeedHistory() { time = now, speed = bytesDownloaded, speedString = downloadSpeed, activeFiles = activeDownloads });
+                uploadSpeedsHistory.Add(new SpeedHistory() { time = now, speed = bytesUploaded, speedString = uploadSpeed, activeFiles = activeUploads });
+                downloadSpeedsHistory.RemoveAll(x => (now - x.time).TotalSeconds > MAX_SPEED_HISTORY_SECONDS);
+                uploadSpeedsHistory.RemoveAll(x => (now - x.time).TotalSeconds > MAX_SPEED_HISTORY_SECONDS);
+            }
             if (HistorykEventChanged != null)
                 HistorykEventChanged.Invoke(this, EventArgs.Empty);
         }
 
+        public List<SpeedHistory> GetDownloadSpeedsHistoryCopy()
+        {
+            lock (_speedHistoryLock)
+            {
+                return downloadSpeedsHistory.ToList();
+            }
+        }
+
+        public List<SpeedHistory> GetUploadSpeedsHistoryCopy()
+        {
+            lock (_speedHistoryLock)
+            {
+                return uploadSpeedsHistory.ToList();
+            }
+        }
+
         public async Task addDownloadBytes(long bytes)
         {
             DownloadBytesMutex.WaitOne();