mirror of
https://github.com/ppy/osu.git
synced 2024-12-13 08:32:57 +08:00
Replace existing cases with new helper method
This commit is contained in:
parent
eecf6ad558
commit
83b4625bd5
@ -49,7 +49,7 @@ namespace osu.Game.Tests.Visual.Editing
|
||||
public void TestCreateNewBeatmap()
|
||||
{
|
||||
AddStep("save beatmap", () => Editor.Save());
|
||||
AddAssert("new beatmap persisted", () => EditorBeatmap.BeatmapInfo.ID > 0);
|
||||
AddAssert("new beatmap persisted", () => EditorBeatmap.BeatmapInfo.IsManaged);
|
||||
AddAssert("new beatmap in database", () => beatmapManager.QueryBeatmapSet(s => s.ID == EditorBeatmap.BeatmapInfo.BeatmapSet.ID)?.DeletePending == false);
|
||||
}
|
||||
|
||||
|
@ -131,7 +131,7 @@ namespace osu.Game.Beatmaps
|
||||
var localRulesetInfo = rulesetInfo as RulesetInfo;
|
||||
|
||||
// Difficulty can only be computed if the beatmap and ruleset are locally available.
|
||||
if (localBeatmapInfo == null || localBeatmapInfo.ID == 0 || localRulesetInfo == null)
|
||||
if (localBeatmapInfo?.IsManaged != true || localRulesetInfo == null)
|
||||
{
|
||||
// If not, fall back to the existing star difficulty (e.g. from an online source).
|
||||
return Task.FromResult(new StarDifficulty(beatmapInfo.StarRating, (beatmapInfo as IBeatmapOnlineInfo)?.MaxCombo ?? 0));
|
||||
|
@ -516,7 +516,7 @@ namespace osu.Game.Database
|
||||
{
|
||||
Files.Dereference(file.FileInfo);
|
||||
|
||||
if (file.ID > 0)
|
||||
if (file.IsManaged)
|
||||
{
|
||||
// This shouldn't be required, but here for safety in case the provided TModel is not being change tracked
|
||||
// Definitely can be removed once we rework the database backend.
|
||||
@ -545,7 +545,7 @@ namespace osu.Game.Database
|
||||
});
|
||||
}
|
||||
|
||||
if (model.ID > 0)
|
||||
if (model.IsManaged)
|
||||
Update(model);
|
||||
}
|
||||
|
||||
@ -811,7 +811,7 @@ namespace osu.Game.Database
|
||||
/// <param name="items">The usable items present in the store.</param>
|
||||
/// <returns>Whether the <typeparamref name="TModel"/> exists.</returns>
|
||||
protected virtual bool CheckLocalAvailability(TModel model, IQueryable<TModel> items)
|
||||
=> model.ID > 0 && items.Any(i => i.ID == model.ID && i.Files.Any());
|
||||
=> model.IsManaged && items.Any(i => i.ID == model.ID && i.Files.Any());
|
||||
|
||||
/// <summary>
|
||||
/// Whether import can be skipped after finding an existing import early in the process.
|
||||
|
@ -45,7 +45,7 @@ namespace osu.Game.Overlays.Settings.Sections
|
||||
{
|
||||
get
|
||||
{
|
||||
int index = skinItems.FindIndex(s => s.ID > 0);
|
||||
int index = skinItems.FindIndex(s => s.IsManaged);
|
||||
if (index < 0)
|
||||
index = skinItems.Count;
|
||||
|
||||
@ -176,7 +176,7 @@ namespace osu.Game.Overlays.Settings.Sections
|
||||
Action = export;
|
||||
|
||||
currentSkin = skins.CurrentSkin.GetBoundCopy();
|
||||
currentSkin.BindValueChanged(skin => Enabled.Value = skin.NewValue.SkinInfo.ID > 0, true);
|
||||
currentSkin.BindValueChanged(skin => Enabled.Value = skin.NewValue.SkinInfo.IsManaged, true);
|
||||
}
|
||||
|
||||
private void export()
|
||||
|
@ -805,14 +805,14 @@ namespace osu.Game.Screens.Select
|
||||
|
||||
private void delete(BeatmapSetInfo beatmap)
|
||||
{
|
||||
if (beatmap == null || beatmap.ID <= 0) return;
|
||||
if (beatmap == null || !beatmap.IsManaged) return;
|
||||
|
||||
dialogOverlay?.Push(new BeatmapDeleteDialog(beatmap));
|
||||
}
|
||||
|
||||
private void clearScores(BeatmapInfo beatmapInfo)
|
||||
{
|
||||
if (beatmapInfo == null || beatmapInfo.ID <= 0) return;
|
||||
if (beatmapInfo == null || !beatmapInfo.IsManaged) return;
|
||||
|
||||
dialogOverlay?.Push(new BeatmapClearScoresDialog(beatmapInfo, () =>
|
||||
// schedule done here rather than inside the dialog as the dialog may fade out and never callback.
|
||||
|
@ -307,7 +307,7 @@ namespace osu.Game.Skinning
|
||||
|
||||
public void Save(Skin skin)
|
||||
{
|
||||
if (skin.SkinInfo.ID <= 0)
|
||||
if (!skin.SkinInfo.IsManaged)
|
||||
throw new InvalidOperationException($"Attempting to save a skin which is not yet tracked. Call {nameof(EnsureMutableSkin)} first.");
|
||||
|
||||
foreach (var drawableInfo in skin.DrawableComponentInfo)
|
||||
|
Loading…
Reference in New Issue
Block a user