The discussion about JSON Marshalers was quite a while ago but after taking a look at them in the Boostrap.groovy I remembered one thing and wanted to bring it up.

If I remember correctly, we decided that we wanted to define the JSON structure we return in a class method toJson() and call the it in the Bootsrap.groovy file to register it as an object marshaler like

class ExampleDomain {
  ...
  Map toJson() {
    [ .... ]
  } 

}

and then in Bootsrap.groovy register it like

JSON.registerObjectMarshaller(ExampleDomain) { ExampleDomain obj ->
    return obj.toJson()
}

So I was wondering if we are going to proceed with that convention or are we going to go some other route.
If we are sticking with it I suggest creating a simple ticket where we clean up the Bootstrap.groovy file by transferring all of the defined Marshalers to Domain > Map toJson().

Another idea/suggestion would be to not define the marshalers manually since the code is repetitive and instead do something like:

[
  Order,
  Location,
  StockMovement,
  Person,
   ....
].each { obj -> 
  JSON.registerObjectMarshaller(obj) { it ->
    return it.toJson()
  }
}

This is sudocode and I am not sure this would work, but just wanted to throw this idea out.