Uff, how can I miss that! My checker doesn't complain and I didn't run it.
Sorry for the inconveniences. Now it's fixed and tested. But please, don't put
Disapprove if you don't have strong reasons to do that. Put only Needs fixing
until the issue is solved.
Regards.
2014-02-24 23:41 GMT+01:00 Christophe Combelles <ccomb@xxxxxxx
<mailto: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
