← Back to team overview

banking-addons-drivers team mailing list archive

Re: account_payment_extension

 

On Fri, Feb 21, 2014 at 11:46 AM, Stefan <stefan@xxxxxxxx> wrote:

> to do it. One request: we currently have an API hook
> extend_payment_order_domain(). If you leave something similar in, then
> we can keep the filter on payment term in a legacy module for current
> setups.
>

In the latest incarnation of the fosdem branch, the payment term filter is
indeed done with the extend_payment_order_domain hook.

-sbi


On Fri, Feb 21, 2014 at 11:46 AM, Stefan <stefan@xxxxxxxx> wrote:

> On 02/21/2014 11:00 AM, Alexis de Lattre wrote:
> >
> > Another related question : in the module account_banking_payment, in
> > the file "model/payment_mode.py", there is a field "payment_term_ids"
> > which is designed to filter out the "invoices to pay" to invoices that
> > have these payment terms. I understand that this field could be used
> > in a way that brings the same kind of feature. For example, instead of
> > having one payment term "30 days net", we could create 2 payment terms
> > "30 days net by wire transfer" and "30 days net by direct debit", and
> > associate each payment term to the related payment mode... wouldn't it
> > do the job ? But we could argue that we are mixing 2 different
> > concepts : the concept of payment term (=delay of the payment) and the
> > concept of payment mode (= how the payment transaction is carried out).
> >
>
> Hi Alexis,
>
> yes, this is how it is used and yes, we are mixing two different
> concepts. I would applaud compatibility between te various modules, and
> for the payment export functionality of banking addons to switch to a
> situation in which a dedicated payment mode is used seems the proper way
> to do it. One request: we currently have an API hook
> extend_payment_order_domain(). If you leave something similar in, then
> we can keep the filter on payment term in a legacy module for current
> setups.
>
> Thanks,
> Stefan.
>
>
> --
> 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
>

References