pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] hr_holidays: improve display name

Browse Source 
Before this commit, the date range was not known without open the record in backend.
It is really painfull to don't know when a guy of your team ping you on a new leave
the date from the current leave. (today ? next week ? ...)

Now we have the date into the name_get and so the subject of the email.

task-2001439

Nothing related to next activity ;) but idea was the same.

closes odoo/odoo#44462

Signed-off-by: Jérémy Kersten (jke) <jke@openerp.com>
This commit is contained in:
Jeremy Kersten
2020-02-03 11:29:50 +00:00
parent 355cb45603
commit e3fa264d10
1 changed files with 21 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/hr_holidays/models/hr_leave.py
+21 -10
View File
@@ -593,17 +593,28 @@ class HolidaysRequest(models.Model):
else:
target = leave.employee_id.name
if leave.leave_type_request_unit == 'hour':
res.append(
(leave.id,
_("%s on %s : %.2f hours") %
(target, leave.holiday_status_id.name, leave.number_of_hours_display))
)
res.append((
leave.id,
_("%s on %s: %.2f hours on %s") % (
target,
leave.holiday_status_id.name,
leave.number_of_hours_display,
fields.Date.to_string(leave.date_from),
)
))
else:
res.append(
(leave.id,
_("%s on %s: %.2f days") %
(target, leave.holiday_status_id.name, leave.number_of_days))
)
display_date = fields.Date.to_string(leave.date_from)
if leave.number_of_days > 1:
display_date += ' ⇨ %s' % fields.Date.to_string(leave.date_to)
res.append((
leave.id,
_("%s on %s: %.2f days (%s)") % (
target,
leave.holiday_status_id.name,
leave.number_of_days,
display_date,
)
))
return res
def add_follower(self, employee_id):