On Fri, Sep 13, 2013 at 9:57 AM, Stefan Rijnhart (Therp) <[email protected]>wrote:
> l.1286: would it be alright with you to put the changes to the move line > model in a separate file, following the (relatively recent) conventions in > this series of modules? > I considered that too, but in the end I decided that since the two parts of the hack where so closely related that it would make the code much harder to understand if it was split in two files. After all, python never enforced a one-class-per-file rule like java, and in this case it made sense to keep them together. What do you think? -sbi -- https://code.launchpad.net/~acsone-openerp/banking-addons/ba-70-payment-export-refactoring/+merge/179543 Your team Banking Addons Core Editors is subscribed to branch lp:banking-addons. -- Mailing list: https://launchpad.net/~banking-addons-team Post to : [email protected] Unsubscribe : https://launchpad.net/~banking-addons-team More help : https://help.launchpad.net/ListHelp
