OK. Glad to hear this. We will be working on this and hopefully we will have improvements to share with everyone.
We will keep in mind what you propose as module distribution for the project. Thank you, -- PS: escribí este email desde mi teléfono, por favor disculpe la brevedad y cualquier error de escritura. Carlos Vásquez CTO · Director de Ingeniería CLEARCORP S.A. [email protected] CR: +(506) 4000-CORP (4000-2677) US: +1 (786) 472-4267 Cel: +(506) 8351-4484 skype: crvasquez twitter: cvclearcorp 300 m. Este de la Escuela Calle de Platanares 11402 San Jerónimo, Moravia San José, Costa Rica http://www.clearcorp.co.cr -- You received this bug notification because you are a member of Banking Addons Team, which is the registrant for Banking Addons. https://bugs.launchpad.net/bugs/1020688 Title: The code for matching st. lines to move lines is not expandable and difficult to maintain Status in Banking Addons: New Bug description: Hi, We are starting to use the banking-addons modules to implement better banking capabilities in OpenERP for Costa RIca. We need to expand the functionality in order to accomplish our objective. We are very happy to see that the code is well documented. But the way that the match method and it's dependent methods are implemented doesn't allow us to localize the banking-addons without having to rewrite several methods completely. Before we do this and duplicate the work between the teams, I would like to know if you are open to a code refactor, only in form. We can reorganize the code doing more small methods, without changing it's current functionality. That way we can inherit the objects and change only some of those methods, or even inherit them (with a super call). We need to invest time in this either way, but we rather do it in the right way getting these addons better. I will wait for your feedback while we start designing the changes and start localizing some parts of it. To manage notifications about this bug go to: https://bugs.launchpad.net/banking-addons/+bug/1020688/+subscriptions -- 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
