banking-addons-drivers team mailing list archive

[Pedro Manuel Baeza Romero <pedro.baeza@xxxxxxxxx>]

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

>  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
>
>  Regards.
>
>
> 2014-02-24 22:33 GMT+01:00 Christophe Combelles <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
>> Post to     : banking-addons-drivers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~banking-addons-drivers
>> More help   : https://help.launchpad.net/ListHelp
>>
>>
>
>