[FIX] hr_gamification: allow employees to grant each other badges

When creating a new wizard, the related field user_id = employee_id.user_id
forces a write on employee model

Make sure we actually have the same value (to avoid using the wizard to modify
employees) and use sudo to grant the badge.

In master, the field should not be stored
This commit is contained in:
Martin Trigaux
2018-08-28 17:32:56 +02:00
parent 42e41b191c
commit 0312c604f4
@@ -2,14 +2,27 @@
# Part of Odoo. See LICENSE file for full copyright and licensing details.
from odoo import api, fields, models, _
from odoo.exceptions import UserError
from odoo.exceptions import UserError, AccessError
class GamificationBadgeUserWizard(models.TransientModel):
_inherit = 'gamification.badge.user.wizard'
employee_id = fields.Many2one('hr.employee', string='Employee', required=True)
user_id = fields.Many2one('res.users', string='User', related='employee_id.user_id', store=True)
user_id = fields.Many2one('res.users', string='User',
related='employee_id.user_id', store=True, readonly=True)
# TODO 12.0/master remove this hack by changing the model
@api.model
def create(self, values):
employee = self.env['hr.employee'].browse(values['employee_id'])
values['user_id'] = employee.user_id.id
try:
return super(GamificationBadgeUserWizard, self).create(values)
except AccessError:
# an employee can not write on another employee
# force sudo because of related
return super(GamificationBadgeUserWizard, self.sudo()).create(values)
@api.multi
def action_grant_badge(self):