Exception when trying to pass object as method parameter

[KubaZ2]

Cannot pass DiscordId as method parameter. It throws System.ArgumentException: 'Object of type 'System.String' cannot be converted to type 'System.UInt64'.'

[lahma]

Would you like to create a PR fixing the issue, accompanied with a test case? Might be enough to create custom ITypeConverter for your case or ideally if there's generic convertible pattern to adhere and then support it automatically in Jint.

[KubaZ2]

I'm new to jint and I don't really know how it works. I don't think I can fix it 😕

[lahma]

All good, thanks for the bug report. We'll try to investigate when we find the time.

[lahma]

@KubaZ2 there was a bug in resolution algorithm that caused Jint to choose wrong overload in this case. You also need to set AllowOperatorOverloading to true in order for implicit/explicit conversions to kick in.

jint/Jint.Tests/Runtime/InteropTests.cs

Lines 2770 to 2780 in 0231800

	[Fact]
	public void ShouldBeAbleToUseConvertibleStructAsMethodParameter()
	{
	var engine = new Engine(options => options.AllowOperatorOverloading());
	engine.SetValue("test", new DiscordTestClass());
	engine.SetValue("id", new DiscordId("12345"));
	Assert.Equal("12345", engine.Evaluate("String(id)").AsString());
	Assert.Equal("12345", engine.Evaluate("test.echo('12345')").AsString());
	Assert.Equal("12345", engine.Evaluate("test.create(12345)").AsString());
	}

There should be a build on MyGet feed soon (on NuGet later), see README on repo root how to access it.

[lahma]

And with #1099 merged you shouldn't even need to allow operator overloading, it's allowed by default with latest soon to be on MyGet version.