RepoMirrors/osu
1
0
Fork 0
mirror of https://github.com/ppy/osu synced 2024-12-17 20:35:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #10640 from bdach/fix-music-paused-on-multi-exit

Browse Source
This commit is contained in:
Dean Herbert 2020-11-02 15:46:19 +09:00 committed by GitHub
commit 8adf146f53
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 22 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
osu.Game.Tests/Visual/Navigation/TestSceneScreenNavigation.cs
View File

@ -56,7 +56,7 @@ namespace osu.Game.Tests.Visual.Navigation
AddUntilStep("wait for selected", () => !Game.Beatmap.IsDefault);
if (withUserPause)
AddStep("pause", () => Game.Dependencies.Get<MusicController>().Stop());
AddStep("pause", () => Game.Dependencies.Get<MusicController>().Stop(true));
AddStep("press enter", () => pressAndRelease(Key.Enter));

24
osu.Game/Overlays/MusicController.cs
View File

@ -166,10 +166,17 @@ namespace osu.Game.Overlays
/// <summary>
/// Start playing the current track (if not already playing).
/// </summary>
/// <param name="restart">Whether to restart the track from the beginning.</param>
/// <param name="requestedByUser">
/// Whether the request to play was issued by the user rather than internally.
/// Specifying <c>true</c> will ensure that other methods like <see cref="EnsurePlayingSomething"/>
/// will resume music playback going forward.
/// </param>
/// <returns>Whether the operation was successful.</returns>
public bool Play(bool restart = false)
public bool Play(bool restart = false, bool requestedByUser = false)
{
IsUserPaused = false;
if (requestedByUser)
IsUserPaused = false;
if (restart)
CurrentTrack.Restart();
@ -182,9 +189,14 @@ namespace osu.Game.Overlays
/// <summary>
/// Stop playing the current track and pause at the current position.
/// </summary>
public void Stop()
/// <param name="requestedByUser">
/// Whether the request to stop was issued by the user rather than internally.
/// Specifying <c>true</c> will ensure that other methods like <see cref="EnsurePlayingSomething"/>
/// will not resume music playback until the next explicit call to <see cref="Play"/>.
/// </param>
public void Stop(bool requestedByUser = false)
{
IsUserPaused = true;
IsUserPaused |= requestedByUser;
if (CurrentTrack.IsRunning)
CurrentTrack.Stop();
}
@ -196,9 +208,9 @@ namespace osu.Game.Overlays
public bool TogglePause()
{
if (CurrentTrack.IsRunning)
Stop();
Stop(true);
else
Play();
Play(requestedByUser: true);
return true;
}

3
osu.Game/Screens/Select/SongSelect.cs
View File

@ -579,7 +579,8 @@ namespace osu.Game.Screens.Select
updateComponentFromBeatmap(Beatmap.Value);
// restart playback on returning to song select, regardless.
music.Play();
// not sure this should be a permanent thing (we may want to leave a user pause paused even on returning)
music.Play(requestedByUser: true);
}
this.FadeIn(250);

2
osu.Game/Tests/Visual/OsuTestScene.cs
View File

@ -189,7 +189,7 @@ namespace osu.Game.Tests.Visual
rulesetDependencies?.Dispose();
if (MusicController?.TrackLoaded == true)
MusicController.CurrentTrack.Stop();
MusicController.Stop();
if (contextFactory?.IsValueCreated == true)
contextFactory.Value.ResetDatabase();