summaryrefslogtreecommitdiff
path: root/addons/hr/models
diff options
context:
space:
mode:
authoryhu-odoo <yhu@odoo.com>2019-12-26 15:35:26 +0000
committerYannick Tivisse <yti@odoo.com>2020-01-29 10:05:12 +0000
commit32fc38ef58dc4b4f1ff4e176ff4fa03b9c427397 (patch)
tree0c1b87a8b507e0710c75f12e3632bcc54b599f21 /addons/hr/models
parent79ce907f664b63576a212a9e5652ce129210b6b5 (diff)
[IMP] hr_appraisal: Improve employee archiving flow
PURPOSE ======= When you close a payroll (employees), there are often a lot of document linked. Archiving an employee should archive the contract, cancel future leaves, archive the private address Specification ============= Add departure date (hr): - Add a Date field to both hr.departure.wizard and hr.employee. In the wizard, the field is required. - In toggle_active() method in hr.employee, set departure_date to false when unarchive the employee. - If user has a current running contract, a user error will raise if user enter a departure date earlier than the start date of the contract. Add checkbox to set a closing date on hr.contract (hr.contract): - In the hr.departure.wizard, set the departure date to be the end date of runing contract. Set the states of all draft contracts to "cancel". Add checkbox to free car (hr.fleet): - In the hr.departure.wizard, set end_date to fleet.vehicle.assignation.log, if there is no end_date or end_date > departure_date - Go through fleet.vehicle, find records with dirver_id to be the employee, set it to False. Add checkbox to archive private address (hr): - when the private address not link to a internel user, set employee.address_home_id.active to Flase - unarchive it after the employee unarchived Add checkbox to cancel future appraisals (hr.appraisal) - find all appraisals link to the employee and state in ['new', 'pending'], set their state to 'cancel'. Add checkbox to cancel future leaves (hr.holidays): - only consider leaves are not in state ['refuse', 'cancel'], find leaves with to_date > departure_date, set their state to 'cancel'. PR #42526 Task 2153106 Related: odoo/enterprise#7471 Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
Diffstat (limited to 'addons/hr/models')
-rw-r--r--addons/hr/models/hr_employee.py7
1 files changed, 6 insertions, 1 deletions
diff --git a/addons/hr/models/hr_employee.py b/addons/hr/models/hr_employee.py
index 4f2bdafd323..0324babaf23 100644
--- a/addons/hr/models/hr_employee.py
+++ b/addons/hr/models/hr_employee.py
@@ -114,6 +114,7 @@ class HrEmployeePrivate(models.Model):
('retired', 'Retired')
], string="Departure Reason", groups="hr.group_hr_user", copy=False, tracking=True)
departure_description = fields.Text(string="Additional Information", groups="hr.group_hr_user", copy=False, tracking=True)
+ departure_date = fields.Date(string="Departure Date", groups="hr.group_hr_user", copy=False, tracking=True)
message_main_attachment_id = fields.Many2one(groups="hr.group_hr_user")
_sql_constraints = [
@@ -273,10 +274,14 @@ class HrEmployeePrivate(models.Model):
def toggle_active(self):
res = super(HrEmployeePrivate, self).toggle_active()
- self.filtered(lambda employee: employee.active).write({
+ unarchived_employees = self.filtered(lambda employee: employee.active)
+ unarchived_employees.write({
'departure_reason': False,
'departure_description': False,
+ 'departure_date': False
})
+ archived_addresses = unarchived_employees.mapped('address_home_id').filtered(lambda addr: not addr.active)
+ archived_addresses.toggle_active()
if len(self) == 1 and not self.active:
return {
'type': 'ir.actions.act_window',