Techsystech
/
server-tools
mirror of https://github.com/OCA/server-tools.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[FIX] base_exception: '_check_exception' is 'api.multi' since it's called by constraint methods 

'detect_exception' can be called on an empty recordset.
pull/2463/head
Laurent Mignon (ACSONE) 2017-06-09 14:09:08 +02:00 committed by matiasperalta1
parent 89af146fd4
commit 401b114771
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
base_exception/models/base_exception.py
View File

@ -108,7 +108,7 @@ class BaseException(models.AbstractModel):
action = self.env.ref('base_exception.action_exception_rule_confirm') action = self.env.ref('base_exception.action_exception_rule_confirm')
return action return action
@api.model @api.multi
def _check_exception(self): def _check_exception(self):
""" """
This method must be used in a constraint that must be created in the This method must be used in a constraint that must be created in the
@ -138,6 +138,8 @@ class BaseException(models.AbstractModel):
def detect_exceptions(self): def detect_exceptions(self):
"""returns the list of exception_ids for all the considered base.exceptions """returns the list of exception_ids for all the considered base.exceptions
""" """
if not self:
return []
exception_obj = self.env['exception.rule'] exception_obj = self.env['exception.rule']
all_exceptions = exception_obj.sudo().search( all_exceptions = exception_obj.sudo().search(
[('rule_group', '=', self[0].rule_group)]) [('rule_group', '=', self[0].rule_group)])