Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

[14.0][IMP] jsonifier with field name - #6

Closed
thomaspaulb wants to merge 1 commit into
14.0from
14.0-jsonifier-with-name
Closed

thomaspaulb wants to merge 1 commit into
14.0from
14.0-jsonifier-with-name

Conversation

@thomaspaulb

Copy link
Copy Markdown
Member

No description provided.

@thomaspaulb
thomaspaulb force-pushed the 14.0-jsonifier-with-name branch from 1afddb1 to 16def21 Compare February 28, 2023 08:43
@thomaspaulb

Copy link
Copy Markdown
Member Author

@gfcapalbo I've opened this one on sunflowerit, maybe later you can open it in OCA.

@gfcapalbo

Copy link
Copy Markdown

@thomaspaulb I was busy with lara last afternoon, this morning I had an idea ,
perhaps _(field.string) is not the best way to fetch the string, i am afraid that if string attribute is not explicitly defined it will return nothing.... I think not , but i must verify 15 minute.

..... I will quickly look at how odoo/model.py or fields.py does it..

@thomaspaulb

Copy link
Copy Markdown
Member Author

@gfcapalbo It's currently on production and working well though. But yes,

--> Please check if .string is always there

--> One thing i noticed is that when I tried to get the translation of order_line.name in sale.order in fr_FR language, I got the field name Description, while I expected something French. Maybe you can check why that is

@gfcapalbo

Copy link
Copy Markdown

@gfcapalbo OCA version has conflict -- note.

@gfcapalbo

Copy link
Copy Markdown

@thomaspaulb

replaced by OCA#2668 , now that FIT/server-tools will be updated and not used as base in projects anymore.
OCA#2668 needs new review and testing.

@gfcapalbo gfcapalbo closed this Jun 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants