misc: chore: Use 'foreach' statement in place of Gommon ForEach in "delete all" mod manager crash fix

* Simplify statement for valid mod folder checking
This commit is contained in:
KeatonTheBot 2025-06-18 00:00:00 -05:00
parent d2a532f971
commit f9780b33bb

View file

@ -4,7 +4,6 @@ using Avalonia.Controls.ApplicationLifetimes;
using Avalonia.Platform.Storage;
using Avalonia.Threading;
using DynamicData;
using Gommon;
using Ryujinx.Ava.Common.Locale;
using Ryujinx.Ava.UI.Helpers;
using Ryujinx.Ava.UI.Models;
@ -261,12 +260,7 @@ namespace Ryujinx.Ava.UI.ViewModels
var destinationDir = ModLoader.GetApplicationDir(ModLoader.GetSdModsBasePath(), _applicationId.ToString("x16"));
// TODO: More robust checking for valid mod folders
var isDirectoryValid = true;
if (directories.Length == 0)
{
isDirectoryValid = false;
}
var isDirectoryValid = directories.Length != 0;
if (!isDirectoryValid)
{
@ -324,7 +318,7 @@ namespace Ryujinx.Ava.UI.ViewModels
public void DeleteAll()
{
Mods.ForEach(it => Delete(it, false));
foreach (var it in Mods) Delete(it, false);
Mods.Clear();
OnPropertyChanged(nameof(ModCount));
Sort();