ToxicMushroom/Dalamud
1
0
Fork 0
mirror of https://github.com/goatcorp/Dalamud.git synced 2025-12-12 18:27:23 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix plugin disable during updates failing due to concurrent file modification; fix plugin updates failing to load config due to assembly version issues

Browse source
This commit is contained in:
meli 2020-03-22 09:41:38 -07:00
parent 760dee2322
commit 27c873dbe9
3 changed files with 47 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

20
Dalamud/Configuration/PluginConfigurations.cs
View file

@ -16,6 +16,10 @@ namespace Dalamud.Configuration
this.configDirectory.Create();
}
// NOTE: Save/Load are still using Type information for now, despite LoadForType<> superseding Load
// and not requiring or using it. It might be worth removing the Type info from Save, to strip it from
// all future saved configs, and then Load() can probably be removed entirey.
public void Save(IPluginConfiguration config, string pluginName) {
File.WriteAllText(GetPath(pluginName).FullName, JsonConvert.SerializeObject(config, Formatting.Indented, new JsonSerializerSettings
{
@ -38,6 +42,22 @@ namespace Dalamud.Configuration
});
}
// Parameterized deserialization
// Currently this is called via reflection from DalamudPluginInterface.GetPluginConfig()
// Eventually there may be an additional pluginInterface method that can call this directly
// without reflection - for now this is in support of the existing plugin api
public T LoadForType<T>(string pluginName) where T : IPluginConfiguration
{
var path = GetPath(pluginName);
if (!path.Exists)
return default(T);
// intentionally no type handling - it will break when updating a plugin at runtime
// and turns out to be unnecessary when we fully qualify the object type
return JsonConvert.DeserializeObject<T>(File.ReadAllText(path.FullName));
}
private FileInfo GetPath(string pluginName) => new FileInfo(Path.Combine(this.configDirectory.FullName, $"{pluginName}.json"));
}
}

18
Dalamud/Plugin/DalamudPluginInterface.cs
View file

@ -94,6 +94,24 @@ namespace Dalamud.Plugin
/// </summary>
/// <returns>A previously saved config or null if none was saved before.</returns>
public IPluginConfiguration GetPluginConfig() {
// This is done to support json deserialization of plugin configurations
// even after running an in-game update of plugins, where the assembly version
// changes.
// Eventually it might make sense to have a separate method on this class
// T GetPluginConfig<T>() where T : IPluginConfiguration
// that can invoke LoadForType() directly instead of via reflection
// This is here for now to support the current plugin API
foreach (var type in Assembly.GetCallingAssembly().GetTypes())
{
if (type.GetInterface(typeof(IPluginConfiguration).FullName) != null)
{
var mi = this.configs.GetType().GetMethod("LoadForType");
var fn = mi.MakeGenericMethod(type);
return (IPluginConfiguration)fn.Invoke(this.configs, new object[] { this.pluginName });
}
}
// this shouldn't be a thing, I think, but just in case
return this.configs.Load(this.pluginName);
}

14
Dalamud/Plugin/PluginInstallerWindow.cs
View file

@ -99,15 +99,19 @@ namespace Dalamud.Plugin
{
Log.Information("Eligible for update: {0}", remoteInfo.InternalName);
foreach (var sortedVersion in sortedVersions) {
File.Create(Path.Combine(sortedVersion.FullName, ".disabled"));
}
// DisablePlugin() below immediately creates a .disabled file anyway, but will fail
// with an exception if we try to do it twice in row like this
// TODO: not sure if doing this for all versions is really necessary, since the
// others really needed to be disabled before anyway
//foreach (var sortedVersion in sortedVersions) {
// File.Create(Path.Combine(sortedVersion.FullName, ".disabled"));
//}
// Try to disable plugin if it is loaded
try {
this.manager.DisablePlugin(info);
} catch {
// ignored
} catch (Exception ex) {
Log.Error(ex, "Plugin disable failed");
}
InstallPlugin(remoteInfo);