Функция для проверки того, активированы ли некоторые поля — сообщество Odoo v8 — Odoo v10

рассмотрим эту функцию:

@api.multi
def create_invoice(self):
    """ Create a invoice refund
    """

    self.ensure_one()
    wizard_brw = self.browse() 
    inv_id = self._context.get('active_id')
    for wizard in wizard_brw:
        if not wizard.sure:
            raise UserError(
                _("Validation error!"),
                _("Please confirm that you know what you're doing by"
                  " checking the option bellow!"))
        if (wizard.invoice_id and wizard.invoice_id.company_id.jour_id and
                wizard.invoice_id and wizard.invoice_id.company_id.acc_id):
            inv_id = self.action_invoice_create(wizard,
                                                wizard.invoice_id) 
        else:
            raise UserError(
                _('Validation error!'),
                _("You must go to the company form and configure a journal"
                  " and an account for damaged invoices"))
    return self.new_open_window([inv_id], 'action_invoice_tree1', 'account')

Эта кнопка, среди прочего, должна проверять, выбрано ли в company_id поле jour_id или acc_id.

Эти поля на res.partner таковы:

class ResCompany(models.Model):
_inherit = 'res.company'

jour_id = fields.Many2one('account.journal', string='Journal', required=False,
    help="Default journal for damaged invoices")
acc_id = fields.Many2one('account.account', string='Account', 
    help="Default account used for invoices and lines from damaged invoices")
printer_fiscal = fields.Boolean(string='Manages fiscal printer',
    help='Indicates that the company can operate a fiscal printer')

Прямо сейчас эта функция не показывает никаких предупреждений или UserError, конечно, я импортировал from odoo.exceptions import UserError

Итак, я предполагаю, что что-то с функцией, кстати, это было вручную перенесено из Odoo v8, это из локализации.

Первоначальный метод выглядел так:

def create_invoice(self, cr, uid, ids, context=None):
    """ Create a invoice refund
    """
    context = context or {}
    wizard_brw = self.browse(cr, uid, ids, context=context)
    inv_id = context.get('active_id')
    for wizard in wizard_brw:
        if not wizard.sure:
            raise osv.except_osv(
                _("Validation error!"),
                _("Please confirm that you know what you're doing by"
                  " checking the option bellow!"))
        if (wizard.invoice_id and wizard.invoice_id.company_id.jour_id and
                wizard.invoice_id and wizard.invoice_id.company_id.acc_id):
            inv_id = self.action_invoice_create(cr, uid, ids, wizard,
                                                wizard.invoice_id, context)
        else:
            raise osv.except_osv(
                _('Validation error!'),
                _("You must go to the company form and configure a journal"
                  " and an account for damaged invoices"))
    return self.new_open_window(cr, uid, ids, [inv_id],
                                'action_invoice_tree1', 'account')

Я думаю, что эта функция могла бы быть намного проще (меньше кода), чем она есть, я просто старался уважать оригинал.

Как я могу добиться этого на «чистом» новом API (от сообщества Odoo v10)?


python odoo-10 odoo-8 openerp
person NeoVe    schedule 27.03.2017    source источник

Ответы (1)


arrow_upward
1
arrow_downward 

self.ensure_one()
wizard_brw = self.browse() 
inv_id = self._context.get('active_id')
for wizard in wizard_brw:

это неправильно по 2 причинам:

  1. вы используете ensure_one, поэтому нет необходимости зацикливаться
  2. вы зацикливаетесь на пустом наборе записей (вы ничего не просматриваете)

Вы должны просто сделать:

self.ensure_one()
if not self.sure:
     # do stuff

self уже твой волшебник ;)

person simahawk    schedule 27.03.2017