Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp

Revision history for this message
qdp (OpenERP) (qdp) wrote :

i checked and for me it's solving all the known problems of the partner ledger. The only problem with this merge proposal is that it is creating error in yaml test because of the new constraint we've added.

[2011-05-04 14:46:25,524][trunk_mai_03] INFO:init:module account: loading test/account_report.yml
....
  File "/home/qdp/tinydev/addons/trunk/account/account_move_line.py", line 102, in _query_get
    raise osv.except_osv(_('Warning !'),_("You haven't supplied enough argument to compute the initial balance"))

Please check it,

thanks,
Quentin

review: Needs Fixing

« Back to merge proposal