Custom Marshalers and Unmarshalers not respected about ojg

[ohler55]

I'll add some more tests tomorrow. Just got back tonight from taking pictures of moose and feeding mosquitoes. :-)

[raidancampbell]

whoops, yes. I'm out of town and unable to give this a more thorough verification, but the diffs looks good to me!

[ohler55]

I completely missed that. Branch "marshalers" has an implementation and I will add unit tests over the next day or two.

[ohler55]

Unit tests added.

[raidancampbell]

hmmm. I can see that the MarshalJSON func is invoked in your test, but I'm not able to replicate it in my above code. It looks like the appendJSON type switch is passing over json.Marshaler and just hitting the default. The incoming data to that function is []TestStruct and the appendDefault code doesn't look to recurse back into the function to check for a json.Marshaler inside the struct.

Thanks for the prompt turnaround though, you've been extremely quick and helpful in the last couple issues!

[ohler55]

Pushed another update that adds the missing support to struct members. More unit tests are needed but a modified version of your test is included in the struct_test.go.

[ohler55]

OjG v1.11.1 has the fix and has been released. The https://github.com/ohler55/compare-go-json project has also been updated with the latest versions.

[ohler55]

Can this be closed?