From dac90e79f7ea5b86e7082c921910babb46584a88 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= Date: Mon, 10 Apr 2017 14:01:29 +0200 Subject: [PATCH] [FIX] payment_transfer: hack the broken hack Put bank data back in transfer providers. Otherwise users are very very sad to not have a default message containing bank accounts data. --- addons/payment_transfer/models/payment.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/addons/payment_transfer/models/payment.py b/addons/payment_transfer/models/payment.py index 996294bf287..00a34ac0ad4 100644 --- a/addons/payment_transfer/models/payment.py +++ b/addons/payment_transfer/models/payment.py @@ -46,6 +46,13 @@ class TransferPaymentAcquirer(models.Model): values['post_msg'] = self._format_transfer_data() return super(TransferPaymentAcquirer, self).create(values) + @api.multi + def write(self, values): + """ Hook in write to create a default post_msg. See create(). """ + if all(not acquirer.post_msg and acquirer.provider != 'transfer' for acquirer in self) and values.get('provider') == 'transfer': + values['post_msg'] = self._format_transfer_data() + return super(TransferPaymentAcquirer, self).write(values) + class TransferPaymentTransaction(models.Model): _inherit = 'payment.transaction'