Refactor method access to use AccessTools for Harmony

[twobit97]

No description provided.

[datvm]

Hi, what's the point of this PR? Surely typeof() is faster than using AccessTools to get the type since it's already known? Isn't it essentially the same?

[twobit97]

Hi, what's the point of this PR? Surely typeof() is faster than using AccessTools to get the type since it's already known? Isn't it essentially the same?

The extension call .Method(...) relies on HarmonyLib.AccessToolsExtensions, and that was causing a crash on my setup. This change keeps the same target type and only changes how the method is resolved. The crash may be specific to my environment

[datvm]

Do you have Beaver Buddies? I heard that they are incompatible with Harmony and I have seen stack trace like that when people had that mod. Anyway, if the extension method does not work, this will not either. It's calling the same method. When compiled by C# compiler, it will be the same.

[twobit97]

Got it, thanks for clarifying. No, I don’t have Beaver Buddies installed. I’ll go through the mods I have and see what’s causing the issue, I'll close the request.

[datvm]

Then it's because you have BepExIn. Which Timberborn no longer uses.