Skip to content
Permalink
Browse files

[FIX] hr_holidays: use api model instead of multi for dashboard

Before 1ced3bf, the get_days_all_request method was called with the
context as an argument. As a consequence, because of the api.multi, the
first arg is used as the ids of the record. As the id is not used in
this method, it was not harmful. With the above mentioned fix, there is
no more argument. This causes a traceback when clicking on menu "My Time
Off/Dashboard".

With this commit, the api.model is used instead, that way, the first arg
is not considered as ids.
  • Loading branch information...
d-fence committed Mar 26, 2019
1 parent 9ea4fef commit 3e27ac39c72fa7d951b8165808991dac7febdd0f
Showing with 1 addition and 1 deletion.
  1. +1 −1 addons/hr_holidays/models/hr_leave_type.py
@@ -189,7 +189,7 @@ def get_days(self, employee_id):

return result

@api.multi
@api.model
def get_days_all_request(self):
employee_id = self._get_contextual_employee_id()

0 comments on commit 3e27ac3

Please sign in to comment.
You can’t perform that action at this time.