diff --git a/odex25_hr/exp_official_mission/models/hr_official_mission.py b/odex25_hr/exp_official_mission/models/hr_official_mission.py index 67ab41295..d941ba906 100644 --- a/odex25_hr/exp_official_mission/models/hr_official_mission.py +++ b/odex25_hr/exp_official_mission/models/hr_official_mission.py @@ -15,6 +15,7 @@ class HrOfficialMission(models.Model): _rec_name = 'mission_type' _description = 'Official mission' _inherit = ['mail.thread', 'mail.activity.mixin'] + _order = 'date_from desc' date = fields.Date(default=lambda self: fields.Date.today()) date_from = fields.Date() diff --git a/odex25_hr/exp_payroll_custom/models/employee_reward.py b/odex25_hr/exp_payroll_custom/models/employee_reward.py index 4483d0c75..3ba2cf5ff 100644 --- a/odex25_hr/exp_payroll_custom/models/employee_reward.py +++ b/odex25_hr/exp_payroll_custom/models/employee_reward.py @@ -22,7 +22,7 @@ class EmployeeReward(models.Model): percentage = fields.Float(string="Percentage%", default=100) amount = fields.Float(string="Amount") account_id = fields.Many2one('account.account') - journal_id = fields.Many2one('account.journal', string='Payment Method', domain=[('type', 'in', ('bank', 'cash'))]) + journal_id = fields.Many2one('account.journal', string='Payment Method') next_approve = fields.Text(string="Next Required Approval", compute="_get_nxt_approve") reward_type = fields.Selection( [('allowance', 'Allowance'), ('amount', 'Amount')], default='allowance') @@ -121,19 +121,20 @@ class EmployeeReward(models.Model): debit_line_vals = { 'name': record.employee_id.name, 'debit': record.amount, - 'account_id': record.account_id.id, - 'partner_id': record.employee_id.user_id.partner_id.id + 'account_id': item.account_id.id, + 'partner_id': record.employee_id.user_id.partner_id.id, + 'analytic_account_id': record.employee_id.department_id.analytic_account_id.id } credit_line_vals = { 'name': record.employee_id.name, 'credit': record.amount, - 'account_id': record.journal_id.default_account_id.id, - 'partner_id': record.employee_id.user_id.partner_id.id + 'account_id': item.journal_id.default_account_id.id + #'partner_id': record.employee_id.user_id.partner_id.id } if not record.move_id: move = record.env['account.move'].create({ 'state': 'draft', - 'journal_id': record.journal_id.id, + 'journal_id': item.journal_id.id, 'date': item.date, 'ref': record.employee_id.name, 'line_ids': [(0, 0, debit_line_vals), (0, 0, credit_line_vals)], @@ -205,23 +206,23 @@ class EmployeeReward(models.Model): # record.move_id.write({'state': 'canceled'}) record.move_id.unlink() record.write({'move_id': False, }) - record.account_id = False - record.journal_id = False + #record.account_id = False + #record.journal_id = False record.reward_state = "draft" self.write({'state': 'draft'}) - self.account_id = False - self.journal_id = False + #self.account_id = False + #self.journal_id = False else: self.write({ 'state': 'draft', - 'account_id': False, - 'journal_id': False + #'account_id': False, + #'journal_id': False }) for record in self.line_ids_reward: record.write({ 'move_id': False, - 'account_id': False, - 'journal_id': False, + #'account_id': False, + #'journal_id': False, 'reward_state': 'draft' }) @@ -275,7 +276,7 @@ class HrEmployee(models.Model): domain=lambda self: self._domain_get_employee()) amount = fields.Float(string="Amount", compute='_compute_calculate_amount', store=True) account_id = fields.Many2one('account.account', string='Account') - journal_id = fields.Many2one('account.journal', string='Payment Method', domain=[('type', 'in', ('bank', 'cash'))]) + journal_id = fields.Many2one('account.journal', string='Payment Method') percentage = fields.Float(string="Percentage%") move_id = fields.Many2one('account.move') contract_advantage_id = fields.Many2one('hr.contract') diff --git a/odex25_hr/exp_payroll_custom/views/employee_reward_view.xml b/odex25_hr/exp_payroll_custom/views/employee_reward_view.xml index d1aa4d94e..991d73f10 100644 --- a/odex25_hr/exp_payroll_custom/views/employee_reward_view.xml +++ b/odex25_hr/exp_payroll_custom/views/employee_reward_view.xml @@ -55,9 +55,9 @@ 'required': [('reward_type', '=', 'allowance')]}"/> - - @@ -90,12 +90,12 @@ attrs="{'readonly':['|','|',('state','=','submitted'),('state','=','done'),('state','=','refused')], 'required': [('state', '=', 'hrm')]}"> - + attrs="{'readonly':[('reward_state','!=','hrm')],'invisible':[('parent.transfer_type','!=','accounting')]}"/--> diff --git a/odex25_hr/hr_loans_salary_advance/views/employee_reward_view.xml b/odex25_hr/hr_loans_salary_advance/views/employee_reward_view.xml index 5c61e6959..95505ff09 100644 --- a/odex25_hr/hr_loans_salary_advance/views/employee_reward_view.xml +++ b/odex25_hr/hr_loans_salary_advance/views/employee_reward_view.xml @@ -25,12 +25,13 @@ force_save="1" attrs="{'column_invisible':[('parent.reward_type','!=','allowance')]}" sum="Totals"/> - + attrs="{'readonly':[('reward_state','!=','hrm')],'column_invisible':[('parent.transfer_type','!=','accounting')]}"/> + attrs="{'readonly':[('reward_state','!=','hrm')],'column_invisible':[('parent.transfer_type','!=','accounting')]}"/--> + - + attrs="{'readonly':[('reward_state','!=','hrm')]}"/--> diff --git a/odex25_hr/hr_termination/models/hr_termination.py b/odex25_hr/hr_termination/models/hr_termination.py index f5d61c821..69cfa4d6a 100644 --- a/odex25_hr/hr_termination/models/hr_termination.py +++ b/odex25_hr/hr_termination/models/hr_termination.py @@ -19,6 +19,7 @@ class HrTermination(models.Model): _inherit = ['mail.thread', 'mail.activity.mixin'] _rec_name = 'employee_id' _description = 'Termination' + _order = 'last_work_date asc' # default compute function def _get_employee_id(self): @@ -436,7 +437,7 @@ class HrTermination(models.Model): self.allowance_deduction_ids = False rule_line = self.env['hr.salary.rule.line'].search([('allowance_deduction_inverse_id', '=', False)]) if rule_line: - rule_line.unlink() + rule_line.sudo().unlink() # Get all advantages from contract if self.sudo().contract_id: