banking-addons-team team mailing list archive

[Carlos Vásquez (CLEARCORP) <carlos.vasquez@xxxxxxxxxxxxxxx>]

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.

carlos.vasquez@xxxxxxxxxxxxxxx
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