What is the NilableType construct for? about go-odoo

[jbguerraz]

Hello David,

Glad to see you've got interest in that tool :)

The NilableType is indeed kinda tricky.

The main issue is Odoo will send a false if a field is not set; so a field of type
int64 for instance won't be able to cope with such false value.

We could have decided to use only interface{} type and require go-odoo users to type their fields when they need to but instead we decided to go the strongly typed way (so one doesn't have to care much about fields types).

To do so we had to create a nilable structure which uses only interface{} typed fields (except for Booleans for which false isn't an issue) that is used for loading data from Odoo (

go-odoo/api/client.go

Line 81 in 1a704aa

func (c *Client) Read(model string, args []interface{}, options interface{}, elem interface{}) error {

-

go-odoo/api/client.go

Line 90 in 1a704aa

func (c *Client) SearchRead(model string, args []interface{}, options interface{}, elem interface{}) error {

) and a load method that would inspect the nilable variant of the object in order to create a properly typed one (

go-odoo/types/types.go

Line 39 in c83e3b8

func load(ns interface{}, s interface{}) interface{} {

) that will then replace the struct the user passed to the Read / SearchRead methods (

go-odoo/api/client.go

Line 85 in 1a704aa

reflect.ValueOf(elem).Elem().Set(reflect.ValueOf(ne.(types.NilableType).Type_()).Elem())

). All of that being transparent to the go-odoo user

Hope that helps :)

Thank you for your contrib efforts on other issues. We surely gonna adopt / contribute with you on those matters in a few (at the moment we're kinda busy).

[jbguerraz]

Done https://github.com/skilld-labs/go-odoo/wiki ;)

[blaggacao]

Wow, could not have been clearer! Copy past to the wiki it! 😉