banking-addons-team team mailing list archive
-
banking-addons-team team
-
Mailing list archive
-
Message #01240
Re: lp:~gdgellatly/banking-addons/banking-addons-reconcile-minor-import-fixes into lp:banking-addons/bank-statement-reconcile-7.0
seeing no one reviewed yet I also added in another quick fix from error installing branch on trunk. Changed method get_default_pay_receive_accounts of account_statement_ext to use orm built in get. Is actually an improvement as get ensures the result is the most specific property whereas the original method gave no such guarentee in a multicompany environment.
Without this this method will fail on trunk and am surprised if it worked on 7.0, however the change is backward compatible with 7.0 and more efficient and less code.
--
https://code.launchpad.net/~gdgellatly/banking-addons/banking-addons-reconcile-minor-import-fixes/+merge/200475
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~gdgellatly/banking-addons/banking-addons-reconcile-minor-import-fixes into lp:banking-addons/bank-statement-reconcile-7.0.
References