Reordered timers and fixed filename overlay on reload

VideoPlayerPageViewModel.cs:
- fix: The filenameoverlay timer is now only started when not reloading
the current video. Previously, the filename was always shown.
- chg: Both timer intervals are reset upon playing media, but only the
reload timer is started.
- rem: Unneeded Start() and Stop() during reloading media.

Author: JohannesKauffmann

OneDrive-Cloud-Player/ViewModels/VideoPlayerPageViewModel.cs

@@ -28,9 +28,9 @@ public class VideoPlayerPageViewModel : ViewModelBase, INotifyPropertyChanged, I
         private readonly INavigationService _navigationService;
         private readonly GraphHelper graphHelper = GraphHelper.Instance();
         /// <summary>
-        /// Fires every two minutes to indicate the OneDrive download URL has expired.
+        /// Fires every time the OneDrive download URL has expired (two minutes).
         /// </summary>
-        private readonly Timer reloadIntervalTimer = new Timer(2 * 60 * 1000);
+        private readonly Timer reloadIntervalTimer = new Timer();
         /// <summary>
         /// Single-shot timer to hide the filename shortly after playing a video.
         /// </summary>
@@ -236,18 +236,19 @@ private async void InitializeLibVLC(InitializedEventArgs eventArgs)
             VideoLength = 0;
             PlayPauseButtonFontIcon = "\xE768";
 
-            // Create LibVLC instance and subscribe to events.
+            // Create LibVLC instance.
             LibVLC = new LibVLC(eventArgs.SwapChainOptions);
             MediaPlayer = new MediaPlayer(LibVLC);
 
+            // Subscribe to events only once.
             MediaPlayer.Playing += MediaPlayer_Playing;
             MediaPlayer.Paused += MediaPlayer_Paused;
             MediaPlayer.TimeChanged += MediaPlayer_TimeChanged;
-
-            // Subscribe to the timer events and start the reloadInterval timer.
-            fileNameOverlayTimer.Elapsed += FileNameOverlayTimer_Elapsed;
             reloadIntervalTimer.Elapsed += ReloadIntervalTimer_Elapsed;
-            reloadIntervalTimer.Start();
+            fileNameOverlayTimer.Elapsed += FileNameOverlayTimer_Elapsed;
+
+            // Start the filenameoverlay timer.
+            fileNameOverlayTimer.Start();
 
             // Finally, play the media.
             await PlayMedia();
@@ -331,14 +332,20 @@ private async Task PlayMedia(long startTime = 0)
         {
             CheckPreviousNextMediaInList();
 
-            FileName = MediaWrapper.CachedDriveItem.Name;
-
-            FileNameOverlayVisiblity = Visibility.Visible;
+            // If the starttime is not 0, a reload is performed, so the filename
+            // is already set and should stay hidden.
+            if (startTime == 0)
+            {
+                FileName = MediaWrapper.CachedDriveItem.Name;
+                FileNameOverlayVisiblity = Visibility.Visible;
+            }
 
-            // Reset the interval to 5 seconds again in case the next video is
-            // played within 5 seconds.
+            // Reset the timer intervals. Only restart the reloadIntervalTimer,
+            // since the filename shouldn't be displayed in case of a reload of
+            // the current video.
             fileNameOverlayTimer.Interval = 5 * 1000;
-            fileNameOverlayTimer.Start();
+            reloadIntervalTimer.Interval = 2 * 60 * 1000;
+            reloadIntervalTimer.Start();
 
             string mediaDownloadURL = await RetrieveDownloadURLMedia(MediaWrapper);
             // Play the OneDrive file.
@@ -451,14 +458,12 @@ private void SetVideoTime(double time)
 
         //TODO: Implement a Dialog system that shows a dialog when there is an error.
         /// <summary>
-        /// Tries to restart the media that is currently playing.
+        /// Restart the media from the currently timestamp.
         /// </summary>
         private async void ReloadCurrentMedia()
         {
             await PlayMedia(TimeLineValue);
-            reloadIntervalTimer.Stop(); //In case a user reloads with the reload button. Stop the timer so we dont get multiple running.
             InvalidOneDriveSession = false;
-            reloadIntervalTimer.Start();
         }
 
         /// <summary>
@@ -541,6 +546,7 @@ private async void PlayPreviousVideo()
 
             MediaWrapper.CachedDriveItem = App.Current.MediaItemList[--MediaListIndex];
             await PlayMedia();
+            fileNameOverlayTimer.Start();
         }
 
         /// <summary>
@@ -555,6 +561,7 @@ private async void PlayNextVideo()
 
             MediaWrapper.CachedDriveItem = App.Current.MediaItemList[++MediaListIndex];
             await PlayMedia();
+            fileNameOverlayTimer.Start();
         }
 
         /// <summary>