From 43f619ed761be3acd515fe953ca7848db0c86074 Mon Sep 17 00:00:00 2001 From: Martin Trigaux Date: Thu, 16 Jun 2016 15:07:01 +0200 Subject: [PATCH] [FIX] payment: correctly trigger the constraints The third parameter expects a list of field, not some random string. This made the constraints for the fields tagged with required_if_provider to be never checked. Closes #12458 --- addons/payment/models/payment_acquirer.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/payment/models/payment_acquirer.py b/addons/payment/models/payment_acquirer.py index bbcf6ca1bb1..7d44d2d16fa 100644 --- a/addons/payment/models/payment_acquirer.py +++ b/addons/payment/models/payment_acquirer.py @@ -129,7 +129,7 @@ class PaymentAcquirer(osv.Model): return True _constraints = [ - (_check_required_if_provider, 'Required fields not filled', ['required for this provider']), + (_check_required_if_provider, 'Required fields not filled', []), ] @openerp.api.model