← Back to team overview

openerp-dev-web team mailing list archive

Re: [Merge] lp:~openerp-dev/openobject-client/trunk-bug-771801-rga into lp:openobject-client

 

Review: Needs Fixing
Seems Good but needs few improvements...

1 when you save a list of fields for export then those saved list is seen but not the fields. The fields appear if I uncheck the import compatible checkbox i.e when the model is refreshed where as it should be seen when I reopen the export form.

2: the rpc calls to "fields_get" are numerous .No doubt its for different model but the calls are duplicated for some models. for eg:
DEBUG_RPC:rpc.request:('execute', 'trunk', 1, '*', ('res.currency', 'fields_get', False, {'lang': 'en_US', 'project_id': False, 'tz': False, 'section_id': False, 'department_id': False}))
DEBUG_RPC:rpc.request:('execute', 'trunk', 1, '*', ('res.currency', 'fields_get', False, {'lang': 'en_US', 'project_id': False, 'tz': False, 'section_id': False, 'department_id': False}))
DEBUG_RPC:rpc.request:('execute', 'trunk', 1, '*', ('account.analytic.account', 'fields_get', False, {'lang': 'en_US', 'project_id': False, 'tz': False, 'section_id': False, 'department_id': False}))
DEBUG_RPC:rpc.request:('execute', 'trunk', 1, '*', ('account.analytic.account', 'fields_get', False, {'lang': 'en_US', 'project_id': False, 'tz': False, 'section_id': False, 'department_id': False}))
DEBUG_RPC:rpc.request:('execute', 'trunk', 1, '*', ('res.currency', 'fields_get', False, {'lang': 'en_US', 'project_id': False, 'tz': False, 'section_id': False, 'department_id': False}))


Please improve this....

Thanks !
nch
-- 
https://code.launchpad.net/~openerp-dev/openobject-client/trunk-bug-771801-rga/+merge/60368
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client/trunk-bug-771801-rga.


References