Thanks for the patch. When you see that kind of code, please try to improve it:
- you made twice the same modification, this means you can refactor the code to have just one single string with a couple of %s. - those _get_headerX functions are used to return a static value for _default; so they can be replaced by the value itself. I will do those changes when I merge the patch, but please try to do it on your own next time. -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-787586-ysa/+merge/62819 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-bug-787586-ysa. _______________________________________________ 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
