Refactor Model.fields_get
This method is very big and contains a lot of exception and test about the field type etc.
I think it will be more flexible, modular and readable that each field computes its definition.
Activity
assigned to @ced
added trytond type::performance + 1 deleted label
review74441002 updated at https://codereview.tryton.org/74441002/#ps1
review58501002 updated at https://codereview.tryton.org/58501002/#ps1
review58501002 updated at https://codereview.tryton.org/58501002/#ps20001
review58501002 updated at https://codereview.tryton.org/58501002/#ps40001
New changeset 1094d993d89c by Cédric Krier in branch 'default':
Move field definition from Model to Field
https://hg.tryton.org/trytond/rev/1094d993d89cNew changeset 0c2774c03e70 by Cédric Krier in branch 'default':
Add field definition
https://hg.tryton.org/trytond-gis/rev/0c2774c03e70New changeset 37433efe75f6 by Cédric Krier in branch 'default':
Move field definition from Model to Field
https://hg.tryton.org/tryton-env/rev/37433efe75f6New changeset 043347298837 by Cédric Krier in branch 'default':
Fill relation_field definition only with real field name
https://hg.tryton.org/trytond/rev/043347298837New changeset 7f7e8a0b86b0 by Cédric Krier in branch 'default':
Fill relation_field definition only with real field name
https://hg.tryton.org/tryton-env/rev/7f7e8a0b86b0New changeset 448e90dbd0ee by Cédric Krier in branch 'default':
Fill size definition only if it is not None
https://hg.tryton.org/trytond/rev/448e90dbd0eeNew changeset 252b5672bdc3 by Cédric Krier in branch 'default':
Fill size definition only if it is not None
https://hg.tryton.org/tryton-env/rev/252b5672bdc3New changeset 6d38b40b1e2d by Cédric Krier in branch 'default':
Fill add_remove definition only if it is not None
https://hg.tryton.org/trytond/rev/6d38b40b1e2dNew changeset 500e58292c07 by Cédric Krier in branch 'default':
Fill add_remove definition only if it is not None
https://hg.tryton.org/tryton-env/rev/500e58292c07
