h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-01-28 09:02:58 +08:00
Code Issues Releases Wiki Activity

Make placeComponent resilient to missing dependencies

Browse Source
This commit is contained in:
Dean Herbert 2023-02-20 20:00:12 +09:00
parent a9c7edd087
commit b68562b033
1 changed files with 18 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
osu.Game/Overlays/SkinEditor/SkinEditor.cs
View File

@ -353,12 +353,18 @@ namespace osu.Game.Overlays.SkinEditor
placeComponent(component);
}
private void placeComponent(ISerialisableDrawable component, bool applyDefaults = true)
/// <summary>
/// Attempt to place a given component in the current target.
/// </summary>
/// <param name="component">The component to be placed.</param>
/// <param name="applyDefaults">Whether to apply default anchor / origin / position values.</param>
/// <returns>Whether placement succeeded. Could fail if no target is available, or if the current target has missing dependency requirements for the component.</returns>
private bool placeComponent(ISerialisableDrawable component, bool applyDefaults = true)
{
var targetContainer = getFirstTarget();
if (targetContainer == null)
return;
return false;
var drawableComponent = (Drawable)component;
@ -370,10 +376,19 @@ namespace osu.Game.Overlays.SkinEditor
drawableComponent.Y = targetContainer.DrawSize.Y / 2;
}
targetContainer.Add(component);
try
{
targetContainer.Add(component);
}
catch
{
// May fail if dependencies are not available, for instance.
return false;
}
SelectedComponents.Clear();
SelectedComponents.Add(component);
return true;
}
private void populateSettings()