BlockState multi-variant serialization is completely incorrect

[Leo40Git]

Example:

import static net.devtech.arrp.json.blockstate.JState.*;

state().add(variant()
  .put("ready", false, model("yo:block/machine"))
  .put("ready", true, model("yo:block/machine_ready"))
)

What it currently outputs as:

{
  "variants": [
    {
      "ready=false": {
        "model": "yo:block/machine"
      }
    },
    {
      "ready=true": {
        "model": "yo:block/machine_ready"
      }
    }
  ]
}

What it should output:

{
  "variants": {
    "ready=false": {
      "model": "yo:block/machine"
    },
    "ready=true": {
      "model": "yo:block/machine_ready"
    }
}

[Devan-Kerman]

it's technically not incorrect because it works :tiny_potato:, however when there's only one variant it should use the non-list serializing, will do

[Devan-Kerman]

wait no, it already does that, are you using an old version of ARRP

[Leo40Git]

honestly this is a report from someone else and I can't believe I forgot to ask them what version of ARRP they were using so uh. I'll try to track them down and yell at them

[Devan-Kerman]

yea your example seems to work just fine in test

[Leo40Git]

Person has been yelled at, closing this. Sorry!

[Devan-Kerman]

np

[duplexsystem]

I am having this issue on 0.5.3.