Code review comment for lp:~pedro.baeza/ocb-addons/6.1-fix-1223243
Revision history for this message
| Pedro Manuel Baeza (pedro.baeza) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Hi, Stefan, thanks for your prudence, and sorry for the mistakes on this MP. First of all, I thought that I linked the corresponding bug, but I wasn't. Now, it is linked. In this bug are explained all the changes, but I paste here again for reference:
- revision 9458: reduce the number of time validate() function is called on account move (worst case was one per line, now should be one per move)
- revision 9679: reduce the number of time analytic lines are created and dropped as mentioned Frederic (only end of validation, unbalanced move creates no analytic lines)
I'm going to set this as WIP until I have some time to fully tested.
Regards.