banking-addons-team team mailing list archive

["Ronald Portier \(Therp\)" <ronald@xxxxxxxx>]

Just to say I support Virgil on #2, and sparse better then dense. In my experience this also makes not only for better reading and comprehension, but also for much easier debugging.

And I do not see the point of adding return True to functions where this does not have any meaning: 
if a function contains (or detects) errors, it should throw an exception. No exception thrown means function did the work it was supposed to do.


-- 
https://code.launchpad.net/~savoirfairelinux-openerp/banking-addons/loose-coupling/+merge/185033
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~savoirfairelinux-openerp/banking-addons/loose-coupling into lp:banking-addons/bank-statement-reconcile-7.0.