From c874b92af340467e773ca994d7ed5f9f03c28cfb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?St=C3=A9phane=20Bidoul?= Date: Sun, 14 Dec 2014 21:34:19 +0100 Subject: [PATCH 1/2] [IMP] set default value for payment.mode.type Since the field is required and payment modes may exist when account_banking_payment_export is installed, a default value is necessary to let Odoo set the required database constraint. --- account_banking_payment_export/models/payment_mode.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/account_banking_payment_export/models/payment_mode.py b/account_banking_payment_export/models/payment_mode.py index df59d37c7..b585932b3 100644 --- a/account_banking_payment_export/models/payment_mode.py +++ b/account_banking_payment_export/models/payment_mode.py @@ -43,9 +43,14 @@ class PaymentMode(models.Model): res = [t.code for t in payment_mode.type.suitable_bank_types] return res + def _default_type(self): + return self.env.ref('account_banking_payment_export.' + 'manual_bank_tranfer') + type = fields.Many2one( 'payment.mode.type', string='Export type', required=True, - help='Select the Export Payment Type for the Payment Mode.') + help='Select the Export Payment Type for the Payment Mode.', + default=_default_type) payment_order_type = fields.Selection( related='type.payment_order_type', readonly=True, string="Order Type", selection=[('payment', 'Payment'), ('debit', 'Debit')], From 7826e346a9d5671777eab4f03cfd427773cfd75e Mon Sep 17 00:00:00 2001 From: "Laurent Mignon (aka lmi)" Date: Tue, 20 Jan 2015 10:21:18 +0100 Subject: [PATCH 2/2] [FIX] default method should return a recordset.. (even if it is empty) --- account_banking_payment_export/models/payment_mode.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/account_banking_payment_export/models/payment_mode.py b/account_banking_payment_export/models/payment_mode.py index b585932b3..d8333339a 100644 --- a/account_banking_payment_export/models/payment_mode.py +++ b/account_banking_payment_export/models/payment_mode.py @@ -23,7 +23,7 @@ # ############################################################################## -from openerp import models, fields +from openerp import models, fields, api class PaymentMode(models.Model): @@ -43,9 +43,12 @@ class PaymentMode(models.Model): res = [t.code for t in payment_mode.type.suitable_bank_types] return res + @api.model def _default_type(self): - return self.env.ref('account_banking_payment_export.' - 'manual_bank_tranfer') + return self.env.ref( + 'account_banking_payment_export.' + 'manual_bank_tranfer', raise_if_not_found=False)\ + or self.env['payment.mode.type'] type = fields.Many2one( 'payment.mode.type', string='Export type', required=True,