Bug: Odin Inspector 3 compatibility error about fastscriptreload

[GhatSmith]

My bad! Sorry guys for the issue, should be very easy to fix. More details here.

[949886]

Source of Odin 3.1.14.3 in Sirenix.OdinInspector.Editor.dll:

/// <summary>Draws the default Odin inspector.</summary>
public void DrawDefaultInspector() => this.DrawOdinInspector();

/// <summary>Draws the default Unity inspector.</summary>
public void DrawUnityInspector() => base.DrawDefaultInspector();

private void DrawOdinInspector()
{
    ...
}

maybe help.

[handzlikchris]

You can try to remove those odin specific calls for now. Likely they are using older version

[mgmhunt]

Same issue. Moving the package from
.../Library/PackageCache/com.handzlikchris.fastscriptreload@cc5dd52158
to
.../Library/Packages

and commenting out 32-37 lines in the file
.../Library/Packages/com.handzlikchris.fastscriptreload@cc5dd52158/Scripts/Editor/NewFields/NewFieldsRendererDefaultEditorPatch.cs

will allow compile. not sure of the side effects of this..!

[handzlikchris]

shouldn't be much downside - it's editor for fields added at runtime. Without Odin it should still render normal fields

[Zarpyk]

Source of Odin 3.1.14.3 in Sirenix.OdinInspector.Editor.dll:

/// <summary>Draws the default Odin inspector.</summary>
public void DrawDefaultInspector() => this.DrawOdinInspector();

/// <summary>Draws the default Unity inspector.</summary>
public void DrawUnityInspector() => base.DrawDefaultInspector();

private void DrawOdinInspector()
{
    ...
}

maybe help.

Probably change line 35/36 (of the script NewFieldsRendererDefaultEditorPatch) fix it (I don't know what these methods do)?

var renderAdditionalFieldsDrawOdinInspectorPostfix = AccessTools.Method(typeof(NewFieldsRendererDefaultEditorPatch), nameof(DrawDefaultInspector));
var customOdinEditorRenderingMethod = AccessTools.Method("Sirenix.OdinInspector.Editor.OdinEditor:DrawDefaultInspector");

[builder-main]

Related to this issue : #77

[handzlikchris]

Compile error as @GhatSmith mentioned, should be fixed with latest push, @949886 - feel free to reopen if still doesn't work after pulling latest