Thanks, the code looks good. Just a few notes about the code: - return list([value]) could be simply return [value] - get_child_of_result could be named child_of_right_to_ids
More importantly: You have a modified the code w.r.t. a bug report in which a problem appeared using an *id*, not a string. And you have discovered the code should work in three cases: id, ids, string. Please augment the tests to include id and ids cases. This way, we make sure the bug won't reappear later. -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-740799-nch/+merge/62606 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-bug-740799-nch. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
