banking-addons-team team mailing list archive

[Leonardo Pistone - camptocamp <leonardo.pistone@xxxxxxxxxxxxxx>]

Thanks for reviewing Pedro.

1. I know we could use lambdas. They both look correct to me, and readability also doesn't change much, IMHO.
2. That is a known weirdness of OpenERP (I put a comment for that reason). If you put a pdb in, you will see that self is the profile, non the statement, even though we are in the statement class.

But none of these points if worth fighting for: let's see if some other reviewer comes up.
-- 
https://code.launchpad.net/~camptocamp/banking-addons/bank-statement-reconcile-7.0-fix-related-balance-check/+merge/209723
Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons/bank-statement-reconcile-7.0.