Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

error #49

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

error #49

wants to merge 26 commits into from

Conversation

vitalikbyrevich
Copy link

I decided to upgrade and got this error in the logs.
[Info :PlanBuild.Patches] Applying Gizmo patches
[Warning: HarmonyX] AccessTools.DeclaredMethod: Could not find method for type Gizmo.ComfyGizmo and name HandleAxisInput and parameters
[Error : Unity Log] ArgumentException: Undefined target method for patch method static bool PlanBuild.ModCompat.PatcherGizmo::ComfyGizmo_HandleAxisInput_Prefix()
Stack trace:
HarmonyLib.PatchClassProcessor.PatchWithAttributes (System.Reflection.MethodBase& lastOriginal) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.PatchClassProcessor.Patch () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
Rethrow as HarmonyException: Patching exception in method null
HarmonyLib.PatchClassProcessor.ReportException (System.Exception exception, System.Reflection.MethodBase original) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.PatchClassProcessor.Patch () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Harmony.PatchAll (System.Type type) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
PlanBuild.Patches.Apply () (at <7e9cb6e4ba5a4f6cbc248ba09441ef5b>:0)
PlanBuild.PlanBuildPlugin.Awake () (at <7e9cb6e4ba5a4f6cbc248ba09441ef5b>:0)
UnityEngine.GameObject:AddComponent(Type)
BepInEx.Bootstrap.Chainloader:Start()
UnityEngine.Application:.cctor()

[Info :Server Devcommands] "ComfyGizmo" detected. Patching "HandleAxisInput" for mouse wheel binding.
[Warning: HarmonyX] AccessTools.Method: Could not find method for type Gizmo.ComfyGizmo and name HandleAxisInput and parameters
[Error : Unity Log] NullReferenceException: Null method for valheim.jerekuusela.server_devcommand.comfygizmo
Stack trace:
HarmonyLib.PatchProcessor.Patch () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Harmony.Patch (System.Reflection.MethodBase original, HarmonyLib.HarmonyMethod prefix, HarmonyLib.HarmonyMethod postfix, HarmonyLib.HarmonyMethod transpiler, HarmonyLib.HarmonyMethod finalizer, HarmonyLib.HarmonyMethod ilmanipulator) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
ServerDevcommands.ComfyGizmoPatcher.DoPatching (System.Reflection.Assembly assembly) (at <7da8fb1e734b429da91d4ccc1356b42b>:0)
ServerDevcommands.ServerDevcommands.Start () (at <7da8fb1e734b429da91d4ccc1356b42b>:0)

There are no such errors when using "GizmoReloaded".

redseiko and others added 26 commits September 18, 2021 21:14
Added manifest/icon/README from existing Thunderstore original Gizmo.
  * Added configuration option for a 'reset all rotations' hot-key (default to un-set).
  * Cleaned-up the UpdatePlacementGhost transpiler.
  * Try to add compatability with other mods that also transpile `UpdatePlacementGhost`.
  * Create a new GameObject `ComfyGizmo` to maintain the current Quaternion rotation state.
  * Re-ordered the mapping/assignment of original Gizmo's prefab XYZ roots/transforms per request by Jere.
  * Increased the snap-angles maximum from 128 to 256.
  * Moved plugin configuration logic into its own class `PluginConfig`.
… between rotation modes. Rotations reset on switching rotation modes.
…ion. Disabled by default. Configurable in F1.
…zmo reloaded. Not fixed to 8/16/32 snap divisions. Will decrease snap divisions until 2 or an odd number hit. Maximum snap divisions maintained at 256.
…ent off grid divisions when quick toggling between snap divisions.
…atterns and using the 2-1-0 fixed rotation system.
…ith axes rotations on gizmo being misaligned with piece rotation. Updated to v1.5.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants