Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-722832-psi

Revision history for this message
Purnendu Singh (OpenERP) (purnendu-singh) wrote :

Hello,

Here are the list of improvements my merge proposal contains:
1) Added code for "Net Profit" and "Net Loss" accounts.
2) Remove "reserve and surplus account" as decided by QDP and MRA.
3) Improve Report rml accourdingly.
4) Balance sheet report:
     a) In case of Net Profit set amount as negative to avoid any confusion.
     b) Change in the logic of sum_dr() and sum_cr() methods to get correct output in both case (Net profit as well as Net Loss), previously in case of Loss we were getting wrong total on Asset side.

Thanks,
Purnendu Singh

review: Needs Resubmitting

« Back to merge proposal