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

Make migration and immediate file saving somewhat more stable, actually dispose Framework.

Browse source
This commit is contained in:
Ottermandias 2022-10-20 14:25:28 +02:00
parent ccfc05f2b2
commit cb4f9f8131
3 changed files with 27 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Penumbra/Mods/Mod.Files.cs
View file

@ -139,7 +139,7 @@ public partial class Mod
foreach( var (group, index) in _groups.WithIndex() )
{
IModGroup.SaveDelayed( group, ModPath, index );
IModGroup.Save( group, ModPath, index );
}
}
}

38
Penumbra/Mods/Mod.Meta.Migration.cs
View file

@ -16,7 +16,16 @@ public sealed partial class Mod
private static class Migration
{
public static bool Migrate( Mod mod, JObject json )
=> MigrateV0ToV1( mod, json ) || MigrateV1ToV2( mod ) || MigrateV2ToV3( mod );
{
var ret = MigrateV0ToV1( mod, json ) || MigrateV1ToV2( mod ) || MigrateV2ToV3( mod );
if( ret )
{
// Immediately save on migration.
mod.SaveMetaFile();
}
return ret;
}
private static bool MigrateV2ToV3( Mod mod )
{
@ -27,10 +36,11 @@ public sealed partial class Mod
// Remove import time.
mod.FileVersion = 3;
mod.SaveMeta();
return true;
}
private static readonly Regex GroupRegex = new( @"group_\d{3}_", RegexOptions.Compiled );
private static bool MigrateV1ToV2( Mod mod )
{
if( mod.FileVersion > 1 )
@ -38,24 +48,26 @@ public sealed partial class Mod
return false;
}
foreach( var (group, index) in mod.GroupFiles.WithIndex().ToArray() )
if (!mod.GroupFiles.All( g => GroupRegex.IsMatch( g.Name )))
{
var newName = Regex.Replace( group.Name, "^group_", $"group_{index + 1:D3}_", RegexOptions.Compiled );
try
foreach( var (group, index) in mod.GroupFiles.WithIndex().ToArray() )
{
if( newName != group.Name )
var newName = Regex.Replace( group.Name, "^group_", $"group_{index + 1:D3}_", RegexOptions.Compiled );
try
{
group.MoveTo( Path.Combine( group.DirectoryName ?? string.Empty, newName ), false );
if( newName != group.Name )
{
group.MoveTo( Path.Combine( group.DirectoryName ?? string.Empty, newName ), false );
}
}
catch( Exception e )
{
Penumbra.Log.Error( $"Could not rename group file {group.Name} to {newName} during migration:\n{e}" );
}
}
catch( Exception e )
{
Penumbra.Log.Error( $"Could not rename group file {group.Name} to {newName} during migration:\n{e}" );
}
}
mod.FileVersion = 2;
mod.SaveMeta();
return true;
}
@ -128,7 +140,7 @@ public sealed partial class Mod
mod.FileVersion = 1;
mod.SaveDefaultMod();
mod.SaveMeta();
mod.SaveMetaFile();
return true;
}

2
Penumbra/Penumbra.cs
View file

@ -7,7 +7,6 @@ using System.Text;
using Dalamud.Game.Command;
using Dalamud.Interface.Windowing;
using Dalamud.Plugin;
using Dalamud.Utility;
using EmbedIO;
using EmbedIO.WebApi;
using ImGuiNET;
@ -291,6 +290,7 @@ public class Penumbra : IDalamudPlugin
ObjectReloader?.Dispose();
ModFileSystem?.Dispose();
CollectionManager?.Dispose();
Framework?.Dispose();
Dalamud.Commands.RemoveHandler( CommandName );