banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <1020688@xxxxxxxxxxxxxxxxxx>]

Hi Carlos,

a lot of voices in the OpenERP community have recently expressed that
the Banking Addons fulfil a certain need, but do so in a heavyweight,
monolithic manner. For a start, the Banking Addons should be split up
into separate modules for

- basic functionality (i.e. importing bank statements)
- integration with account_payment for exporting payment and direct debit orders
- automatic matching according to customizable or localized rules
- interactive matching
- account number validation and lookup

Your initiative fits perfectly within this goal. A merge proposal will
be more than welcome.

Cheers,
Stefan.

-- 
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