Review: Needs Fixing
Points to improve:
- position of the new boolean field is not proper and string should be "Made
Payment Order?" + improve the tooltip on it.
- No need to write: if run.payment_advice == False: => just raise error at the
beginning of the for loop of run.
- No need of _defaults = {} for False value.
Thanks,
Mustufa
--
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-india-add-boolean-field-mtr/+merge/115285
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/trunk-payroll-india-mra.
_______________________________________________
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
