From 42895e27b6f1188e6e623d5a921c99f4b4f0038f Mon Sep 17 00:00:00 2001
From: Salman Ahmed <email@iisalman.me>
Date: Fri, 4 Sep 2020 10:10:14 +0300
Subject: [PATCH] Expose track change results on the methods

---
 osu.Game/Overlays/MusicController.cs | 26 ++++++++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

diff --git a/osu.Game/Overlays/MusicController.cs b/osu.Game/Overlays/MusicController.cs
index 119aad5226..ea72ef0b84 100644
--- a/osu.Game/Overlays/MusicController.cs
+++ b/osu.Game/Overlays/MusicController.cs
@@ -207,7 +207,18 @@ namespace osu.Game.Overlays
         /// <summary>
         /// Play the previous track or restart the current track if it's current time below <see cref="restart_cutoff_point"/>.
         /// </summary>
-        public void PreviousTrack() => Schedule(() => prev());
+        /// <param name="onSuccess">
+        /// Invoked when the operation has been performed successfully.
+        /// The result isn't returned directly to the caller because
+        /// the operation is scheduled and isn't performed immediately.
+        /// </param>
+        /// <returns>A <see cref="ScheduledDelegate"/> of the operation.</returns>
+        public ScheduledDelegate PreviousTrack(Action<PreviousTrackResult> onSuccess = null) => Schedule(() =>
+        {
+            PreviousTrackResult res = prev();
+            if (res != PreviousTrackResult.None)
+                onSuccess?.Invoke(res);
+        });
 
         /// <summary>
         /// Play the previous track or restart the current track if it's current time below <see cref="restart_cutoff_point"/>.
@@ -243,7 +254,18 @@ namespace osu.Game.Overlays
         /// <summary>
         /// Play the next random or playlist track.
         /// </summary>
-        public void NextTrack() => Schedule(() => next());
+        /// <param name="onSuccess">
+        /// Invoked when the operation has been performed successfully.
+        /// The result isn't returned directly to the caller because
+        /// the operation is scheduled and isn't performed immediately.
+        /// </param>
+        /// <returns>A <see cref="ScheduledDelegate"/> of the operation.</returns>
+        public ScheduledDelegate NextTrack(Action onSuccess = null) => Schedule(() =>
+        {
+            bool res = next();
+            if (res)
+                onSuccess?.Invoke();
+        });
 
         private bool next()
         {