Pass the full EF context rather than a legacy RulesetStore

This commit is contained in:
Dean Herbert 2021-11-23 16:27:28 +09:00
parent f264b128c2
commit 505fede44d
2 changed files with 14 additions and 6 deletions

View File

@ -14,7 +14,6 @@ using osu.Framework.Statistics;
using osu.Game.Configuration;
using osu.Game.Input.Bindings;
using osu.Game.Models;
using osu.Game.Rulesets;
using Realms;
#nullable enable
@ -33,7 +32,7 @@ namespace osu.Game.Database
/// </summary>
public readonly string Filename;
private readonly RulesetStore? rulesets;
private readonly IDatabaseContextFactory? efContextFactory;
/// <summary>
/// Version history:
@ -77,10 +76,10 @@ namespace osu.Game.Database
}
}
public RealmContextFactory(Storage storage, string filename, RulesetStore? rulesets = null)
public RealmContextFactory(Storage storage, string filename, IDatabaseContextFactory? efContextFactory = null)
{
this.storage = storage;
this.rulesets = rulesets;
this.efContextFactory = efContextFactory;
Filename = filename;
@ -252,7 +251,7 @@ namespace osu.Game.Database
var newItem = newSettings.ElementAt(i);
long rulesetId = oldItem.RulesetID;
string? rulesetName = rulesets?.GetRuleset((int)rulesetId)?.ShortName;
string? rulesetName = getRulesetShortNameFromLegacyID(rulesetId);
if (string.IsNullOrEmpty(rulesetName))
migration.NewRealm.Remove(newItem);
@ -264,6 +263,15 @@ namespace osu.Game.Database
}
}
private string? getRulesetShortNameFromLegacyID(long rulesetId)
{
if (efContextFactory == null)
return null;
using (var efContext = efContextFactory.Get())
return efContext.RulesetInfo.First(r => r.ID == rulesetId)?.ShortName;
}
/// <summary>
/// Flush any active contexts and block any further writes.
/// </summary>

View File

@ -192,7 +192,7 @@ namespace osu.Game
dependencies.Cache(RulesetStore = new RulesetStore(contextFactory, Storage));
dependencies.Cache(realmFactory = new RealmContextFactory(Storage, "client", RulesetStore));
dependencies.Cache(realmFactory = new RealmContextFactory(Storage, "client", contextFactory));
dependencies.CacheAs(Storage);