banking-addons-drivers team mailing list archive

[Stefan <stefan@xxxxxxxx>]

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.