banking-addons-drivers team mailing list archive

[Christophe Combelles <ccomb@xxxxxxx>]

Sorry but I wonder how could I approve your MP, which was broken, you fixed it 
and you told « it works now », but you still have an enormous syntax error...

+ return self.get_import_type selection(cr, uid, context=context)

Christophe

Le 24/02/2014 23:10, Pedro Manuel Baeza Romero a écrit :
> Hi, Christophe,
>
> Sébastien MP and mine 
> (https://code.launchpad.net/~pedro.baeza/banking-addons/bank-statement-reconcile-70-import_imp/+merge/200022 
> <https://code.launchpad.net/%7Epedro.baeza/banking-addons/bank-statement-reconcile-70-import_imp/+merge/200022>) 
> both have a way to make inheritance of the selection method easier, so please 
> review both and approve them, so we can proceed with the merge.
>
> If you want to see some working examples of using this base for an statement 
> import, you can see /account_statement_ofx_import/ module on the same 
> repository, of this work in progress branch for spanish statement format:
>
> https://code.launchpad.net/~pedro.baeza/openerp-spain/7.0-l10n_es_bank_statement 
> <https://code.launchpad.net/%7Epedro.baeza/openerp-spain/7.0-l10n_es_bank_statement>
>
> Regards.
>
>
> 2014-02-24 22:33 GMT+01:00 Christophe Combelles <ccomb@xxxxxxx 
> <mailto:ccomb@xxxxxxx>>:
>
>     Hi,
>
>     I've spent some time today to understand how to use and extend the
>     /"account_statement_base_import"/ module.
>     I was trying to create another import filter for a french bank (credit
>     coop), but I realized that the list of filters cannot be extended at all.
>
>     I found a fix in a pending merge request from Sébastien here :
>     https://code.launchpad.net/~akretion-team/banking-addons/bank-statement-reconcile-70-api-improvement/+merge/197774
>     <https://code.launchpad.net/%7Eakretion-team/banking-addons/bank-statement-reconcile-70-api-improvement/+merge/197774>
>
>     what I don't understand is I also found numerous existing import filters
>     for bnp, lbp, paybox, atos, etc... some of them being quite old (august 2013).
>     How are all these import filters supposed to work, since the base module
>     cannot be extended?
>     Are they supposed to work with another branch or version?
>
>     thanks,
>     Christophe
>
>     --
>     Mailing list: https://launchpad.net/~banking-addons-drivers
>     <https://launchpad.net/%7Ebanking-addons-drivers>
>     Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
>     <mailto:banking-addons-drivers@xxxxxxxxxxxxxxxxxxx>
>     Unsubscribe : https://launchpad.net/~banking-addons-drivers
>     <https://launchpad.net/%7Ebanking-addons-drivers>
>     More help   : https://help.launchpad.net/ListHelp