Subscribe to download
Better Advanced Tablet [StationeersMods]

[Better Advanced Tablet [StationeersMods] - 0Harmony]: Patching exception in method null
at HarmonyLib.PatchClassProcessor.ReportException (System.Exception exception, System.Reflection.MethodBase original) [0x0006c] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at HarmonyLib.PatchClassProcessor.Patch () [0x00095] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at BetterAdvancedTablet.BetterAdvancedTabletPlugin+<>c__DisplayClass6_0.<Patch>b__0 (System.Type type) [0x00024] in <a47f360a5e1d4f9690fd526ca1ee261f>:0
Undefined target method for patch method static void BetterAdvancedTablet.Patches+MainMenuWindowManagerPatches::Prefix()
at HarmonyLib.PatchClassProcessor.PatchWithAttributes (System.Reflection.MethodBase& lastOriginal) [0x00047] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at HarmonyLib.PatchClassProcessor.Patch () [0x0006a] in <474744d65d8e460fa08cd5fd82b5d65f>:0

That looks like an error that can be ignored. Due to the logic I added with PR #4, it will now continue down the chain even if one patch fails. As long as that is the only error, you will be fine. (Context, the old method `NewOrContinue` was deleted in beta (now main)). It is expected that that patch will fail on the new version but multiple patches were needed to maintain backwards compatibility.

BetterAdvancedTablet.Patches+MainMenuWindowManagerPatches
[[Invalid About.xml] 3134483503 - 0Harmony]: Patching exception in method null
at HarmonyLib.PatchClassProcessor.ReportException (System.Exception exception, System.Reflection.MethodBase original) [0x0006c] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at HarmonyLib.PatchClassProcessor.Patch () [0x00095] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at BetterAdvancedTablet.BetterAdvancedTabletPlugin+<>c__DisplayClass6_0.<Patch>b__0 (System.Type type) [0x00024] in <a47f360a5e1d4f9690fd526ca1ee261f>:0
Undefined target method for patch method static void BetterAdvancedTablet.Patches+MainMenuWindowManagerPatches::Prefix()
at HarmonyLib.PatchClassProcessor.PatchWithAttributes (System.Reflection.MethodBase& lastOriginal) [0x00047] in <474744d65d8e460fa08cd5fd82b5d65f>:0
at HarmonyLib.PatchClassProcessor.Patch () [0x0006a] in <474744d65d8e460fa08cd5fd82b5d65f>:0
[[Invalid About.xml] 3134483503 - Information]: Patching

https://github.com/serialtasted/betteradvancedtablet/pull/4 has been tested on both versions and it will continue to patch if one of the patch classes fail.

Replacing `harmony.PatchAll();` with:
AccessTools.GetTypesFromAssembly(typeof(Patches).Assembly).Do(type =>
{
try
{
Debug.Log($"Patching {type.FullName}");
harmony.CreateClassProcessor(type).Patch();
}
catch (System.Exception e)
{
Debug.LogException(e);
}
});

Does work on both beta and main, so that might be a better option.

I had a change to test it, seems I was wrong. It looks like PatchAll does not continue on error. As a result, you will need to remove the old patch (commit 2b8992) and wait until after beta is released to publish to steam.

Looks like beyond having a custom patching method instead of PatchAll, there is no way to have patching continue on error.

Sounds good. I think it should work on both versions, but I haven't tested that part yet. Based on how it is coded, it should of course work since 1 patch will fail on beta, and another on mainline. The patch on `LoadWorld` will probaly result in AdvancedTabletPrefabPatch() running twice but since there is logic in that to only run once, it should work.

I need to do some testing. But I just merged it

Btw, @Serialtasted, I was wondering on the status of https://github.com/serialtasted/betteradvancedtablet/pull/3 that I submitted a few weeks ago to fix the mod on beta?

For all, Reinstall StationeersMods and Bepinex then it works. sry for my English!