diff --git a/addons/hr_evaluation/hr_evaluation.py b/addons/hr_evaluation/hr_evaluation.py index d4ee28aff68..612d91e4560 100644 --- a/addons/hr_evaluation/hr_evaluation.py +++ b/addons/hr_evaluation/hr_evaluation.py @@ -27,6 +27,7 @@ import time from openerp.osv import fields, osv from openerp.tools.translate import _ + class hr_evaluation_plan(osv.osv): _name = "hr_evaluation.plan" _description = "Appraisal Plan" @@ -42,11 +43,12 @@ class hr_evaluation_plan(osv.osv): 'active': True, 'month_first': 6, 'month_next': 12, - 'company_id': lambda s,cr,uid,c: s.pool.get('res.company')._company_default_get(cr, uid, 'account.account', context=c), + 'company_id': lambda s, cr, uid, c: s.pool.get('res.company')._company_default_get(cr, uid, 'account.account', context=c), } hr_evaluation_plan() + class hr_evaluation_plan_phase(osv.osv): _name = "hr_evaluation.plan.phase" _description = "Appraisal Plan Phase" @@ -55,13 +57,13 @@ class hr_evaluation_plan_phase(osv.osv): 'name': fields.char("Phase", size=64, required=True), 'sequence': fields.integer("Sequence"), 'company_id': fields.related('plan_id', 'company_id', type='many2one', relation='res.company', string='Company', store=True, readonly=True), - 'plan_id': fields.many2one('hr_evaluation.plan','Appraisal Plan', ondelete='cascade'), + 'plan_id': fields.many2one('hr_evaluation.plan', 'Appraisal Plan', ondelete='cascade'), 'action': fields.selection([ - ('top-down','Top-Down Appraisal Requests'), - ('bottom-up','Bottom-Up Appraisal Requests'), - ('self','Self Appraisal Requests'), - ('final','Final Interview')], 'Action', required=True), - 'survey_id': fields.many2one('survey','Appraisal Form',required=True), + ('top-down', 'Top-Down Appraisal Requests'), + ('bottom-up', 'Bottom-Up Appraisal Requests'), + ('self', 'Self Appraisal Requests'), + ('final', 'Final Interview')], 'Action', required=True), + 'survey_id': fields.many2one('survey', 'Appraisal Form', required=True), 'send_answer_manager': fields.boolean('All Answers', help="Send all answers to the manager"), 'send_answer_employee': fields.boolean('All Answers', @@ -73,15 +75,14 @@ class hr_evaluation_plan_phase(osv.osv): 'wait': fields.boolean('Wait Previous Phases', help="Check this box if you want to wait that all preceding phases " + "are finished before launching this phase."), - 'mail_feature': fields.boolean('Send mail for this phase', help="Check this box if you want to send mail to employees"+ - " coming under this phase"), + 'mail_feature': fields.boolean('Send mail for this phase', help="Check this box if you want to send mail to employees coming under this phase"), 'mail_body': fields.text('Email'), - 'email_subject':fields.text('char') + 'email_subject': fields.text('char') } _defaults = { 'sequence': 1, 'email_subject': _('''Regarding '''), - 'mail_body': lambda *a:_(''' + 'mail_body': lambda *a: _(''' Date: %(date)s Dear %(employee_name)s, @@ -100,24 +101,26 @@ Thanks, hr_evaluation_plan_phase() + class hr_employee(osv.osv): _name = "hr.employee" - _inherit="hr.employee" + _inherit = "hr.employee" _columns = { 'evaluation_plan_id': fields.many2one('hr_evaluation.plan', 'Appraisal Plan'), 'evaluation_date': fields.date('Next Appraisal Date', help="The date of the next appraisal is computed by the appraisal plan's dates (first appraisal + periodicity)."), } + def run_employee_evaluation(self, cr, uid, automatic=False, use_new_cursor=False, context=None): now = parser.parse(datetime.now().strftime('%Y-%m-%d')) obj_evaluation = self.pool.get('hr_evaluation.evaluation') - emp_ids =self.search(cr, uid, [ ('evaluation_plan_id','<>',False), ('evaluation_date','=', False)], context=context) + emp_ids = self.search(cr, uid, [('evaluation_plan_id', '<>', False), ('evaluation_date', '=', False)], context=context) for emp in self.browse(cr, uid, emp_ids, context=context): - first_date = (now+ relativedelta(months=emp.evaluation_plan_id.month_first)).strftime('%Y-%m-%d') + first_date = (now + relativedelta(months=emp.evaluation_plan_id.month_first)).strftime('%Y-%m-%d') self.write(cr, uid, [emp.id], {'evaluation_date': first_date}, context=context) - emp_ids =self.search(cr, uid, [ - ('evaluation_plan_id','<>',False), ('evaluation_date','<=', time.strftime("%Y-%m-%d")), - ], context=context) + emp_ids = self.search(cr, uid, [ + ('evaluation_plan_id', '<>', False), ('evaluation_date', '<=', time.strftime("%Y-%m-%d")), + ], context=context) for emp in self.browse(cr, uid, emp_ids, context=context): next_date = (now + relativedelta(months=emp.evaluation_plan_id.month_next)).strftime('%Y-%m-%d') self.write(cr, uid, [emp.id], {'evaluation_date': next_date}, context=context) @@ -125,6 +128,7 @@ class hr_employee(osv.osv): obj_evaluation.button_plan_in_progress(cr, uid, [plan_id], context=context) return True + class hr_evaluation(osv.osv): _name = "hr_evaluation.evaluation" _inherit = "mail.thread" @@ -135,28 +139,27 @@ class hr_evaluation(osv.osv): 'employee_id': fields.many2one('hr.employee', "Employee", required=True), 'note_summary': fields.text('Appraisal Summary'), 'note_action': fields.text('Action Plan', - help="If the evaluation does not meet the expectations, you can propose"+ - "an action plan"), + help="If the evaluation does not meet the expectations, you can propose an action plan"), 'rating': fields.selection([ - ('0','Significantly bellow expectations'), - ('1','Did not meet expectations'), - ('2','Meet expectations'), - ('3','Exceeds expectations'), - ('4','Significantly exceeds expectations'), + ('0', 'Significantly bellow expectations'), + ('1', 'Did not meet expectations'), + ('2', 'Meet expectations'), + ('3', 'Exceeds expectations'), + ('4', 'Significantly exceeds expectations'), ], "Appreciation", help="This is the appreciation on which the evaluation is summarized."), - 'survey_request_ids': fields.one2many('hr.evaluation.interview','evaluation_id','Appraisal Forms'), + 'survey_response_ids': fields.one2many('hr.evaluation.interview', 'evaluation_id', 'Appraisal Forms'), 'plan_id': fields.many2one('hr_evaluation.plan', 'Plan', required=True), 'state': fields.selection([ - ('draft','New'), - ('cancel','Cancelled'), - ('wait','Plan In Progress'), - ('progress','Waiting Appreciation'), - ('done','Done'), + ('draft', 'New'), + ('cancel', 'Cancelled'), + ('wait', 'Plan In Progress'), + ('progress', 'Waiting Appreciation'), + ('done', 'Done'), ], 'Status', required=True, readonly=True), 'date_close': fields.date('Ending Date', select=True), } _defaults = { - 'date': lambda *a: (parser.parse(datetime.now().strftime('%Y-%m-%d')) + relativedelta(months =+ 1)).strftime('%Y-%m-%d'), + 'date': lambda *a: (parser.parse(datetime.now().strftime('%Y-%m-%d')) + relativedelta(months=1)).strftime('%Y-%m-%d'), 'state': lambda *a: 'draft', } @@ -181,7 +184,6 @@ class hr_evaluation(osv.osv): return {'value': vals} def button_plan_in_progress(self, cr, uid, ids, context=None): - mail_message = self.pool.get('mail.message') hr_eval_inter_obj = self.pool.get('hr.evaluation.interview') if context is None: context = {} @@ -198,11 +200,15 @@ class hr_evaluation(osv.osv): children = [evaluation.employee_id] for child in children: + if (not child.user_id or not child.user_id.partner_id.id) and not child.work_email: + raise osv.except_osv(_('Warning!'), _('%s have not related user name and have no email address.' % child.name)) + int_id = hr_eval_inter_obj.create(cr, uid, { 'evaluation_id': evaluation.id, 'survey_id': phase.survey_id.id, - 'date_deadline': (parser.parse(datetime.now().strftime('%Y-%m-%d')) + relativedelta(months =+ 1)).strftime('%Y-%m-%d'), - 'user_id': child.user_id.id, + 'date_deadline': datetime.now() + relativedelta(months=1), + 'partner_id': child.user_id and child.user_id.partner_id.id or None, + 'email': child.work_email or None, 'user_to_review_id': evaluation.employee_id.id }, context=context) if phase.wait: @@ -212,7 +218,7 @@ class hr_evaluation(osv.osv): if (not wait) and phase.mail_feature: body = phase.mail_body % {'employee_name': child.name, 'user_signature': child.user_id.signature, - 'eval_name': phase.survey_id.title, 'date': time.strftime('%Y-%m-%d'), 'time': time } + 'eval_name': phase.survey_id.title, 'date': time.strftime('%Y-%m-%d'), 'time': time} sub = phase.email_subject if child.work_email: vals = {'state': 'outgoing', @@ -222,32 +228,32 @@ class hr_evaluation(osv.osv): 'email_from': evaluation.employee_id.work_email} self.pool.get('mail.mail').create(cr, uid, vals, context=context) - self.write(cr, uid, ids, {'state':'wait'}, context=context) + self.write(cr, uid, ids, {'state': 'wait'}, context=context) return True def button_final_validation(self, cr, uid, ids, context=None): request_obj = self.pool.get('hr.evaluation.interview') - self.write(cr, uid, ids, {'state':'progress'}, context=context) + self.write(cr, uid, ids, {'state': 'progress'}, context=context) for evaluation in self.browse(cr, uid, ids, context=context): if evaluation.employee_id and evaluation.employee_id.parent_id and evaluation.employee_id.parent_id.user_id: self.message_subscribe_users(cr, uid, [evaluation.id], user_ids=[evaluation.employee_id.parent_id.user_id.id], context=context) - if len(evaluation.survey_request_ids) != len(request_obj.search(cr, uid, [('evaluation_id', '=', evaluation.id),('state', 'in', ['done','cancel'])], context=context)): - raise osv.except_osv(_('Warning!'),_("You cannot change state, because some appraisal(s) are in waiting answer or draft state.")) + if len(evaluation.survey_response_ids) != len(request_obj.search(cr, uid, [('evaluation_id', '=', evaluation.id), ('state', 'in', ['done', 'cancel'])], context=context)): + raise osv.except_osv(_('Warning!'), _("You cannot change state, because some appraisal(s) are in waiting answer or draft state.")) return True def button_done(self, cr, uid, ids, context=None): - self.write(cr, uid, ids,{'state':'done', 'date_close': time.strftime('%Y-%m-%d')}, context=context) + self.write(cr, uid, ids, {'state': 'done', 'date_close': time.strftime('%Y-%m-%d')}, context=context) return True def button_cancel(self, cr, uid, ids, context=None): - interview_obj=self.pool.get('hr.evaluation.interview') + interview_obj = self.pool.get('hr.evaluation.interview') evaluation = self.browse(cr, uid, ids[0], context) - interview_obj.survey_req_cancel(cr, uid, [r.id for r in evaluation.survey_request_ids]) - self.write(cr, uid, ids,{'state':'cancel'}, context=context) + interview_obj.survey_req_cancel(cr, uid, [r.id for r in evaluation.survey_response_ids]) + self.write(cr, uid, ids, {'state': 'cancel'}, context=context) return True def button_draft(self, cr, uid, ids, context=None): - self.write(cr, uid, ids,{'state': 'draft'}, context=context) + self.write(cr, uid, ids, {'state': 'draft'}, context=context) return True def copy(self, cr, uid, id, default=None, context=None): @@ -256,9 +262,9 @@ class hr_evaluation(osv.osv): if context is None: context = {} default = default.copy() - default['survey_request_ids'] = [] + default['survey_response_ids'] = [] return super(hr_evaluation, self).copy(cr, uid, id, default, context=context) - + def write(self, cr, uid, ids, vals, context=None): if vals.get('employee_id'): employee_id = self.pool.get('hr.employee').browse(cr, uid, vals.get('employee_id'), context=context) @@ -268,47 +274,55 @@ class hr_evaluation(osv.osv): new_vals = {'date_deadline': vals.get('date')} obj_hr_eval_iterview = self.pool.get('hr.evaluation.interview') for evalutation in self.browse(cr, uid, ids, context=context): - for survey_req in evalutation.survey_request_ids: + for survey_req in evalutation.survey_response_ids: obj_hr_eval_iterview.write(cr, uid, [survey_req.id], new_vals, context=context) return super(hr_evaluation, self).write(cr, uid, ids, vals, context=context) hr_evaluation() + class survey_request(osv.osv): - _inherit = "survey.request" + _inherit = "survey.response" _columns = { 'is_evaluation': fields.boolean('Is Appraisal?'), } survey_request() + class hr_evaluation_interview(osv.osv): _name = 'hr.evaluation.interview' - _inherits = {'survey.request': 'request_id'} - _inherit = 'mail.thread' - _rec_name = 'request_id' + _inherits = {'survey.response': 'response_id'} + _inherit = ['mail.thread', 'ir.needaction_mixin'] + _rec_name = 'response_id' _description = 'Appraisal Interview' _columns = { - 'request_id': fields.many2one('survey.request','Request_id', ondelete='cascade', required=True), + 'response_id': fields.many2one('survey.response', 'Request_id', ondelete='cascade', required=True), 'user_to_review_id': fields.many2one('hr.employee', 'Employee to Interview'), 'evaluation_id': fields.many2one('hr_evaluation.evaluation', 'Appraisal Form'), + 'user_id': fields.many2one('res.users', 'Interviewer', readonly=1), } _defaults = { 'is_evaluation': True, } + def _needaction_domain_get(self, cr, uid, context=None): + return ['|', + '&', ('state', '=', 'done'), ('user_id', '=', uid), + '&', ('state', 'in', ['new', 'skip']), ('user_to_review_id.user_id', '=', uid)] + def name_get(self, cr, uid, ids, context=None): if not ids: return [] reads = self.browse(cr, uid, ids, context=context) res = [] for record in reads: - name = record.request_id.survey_id.title + name = record.response_id.survey_id.title res.append((record['id'], name)) return res def survey_req_waiting_answer(self, cr, uid, ids, context=None): - self.write(cr, uid, ids, { 'state': 'waiting_answer'}, context=context) + self.write(cr, uid, ids, {'state': 'new'}, context=context) return True def survey_req_done(self, cr, uid, ids, context=None): @@ -317,25 +331,21 @@ class hr_evaluation_interview(osv.osv): flag = False wating_id = 0 if not id.evaluation_id.id: - raise osv.except_osv(_('Warning!'),_("You cannot start evaluation without Appraisal.")) - records = hr_eval_obj.browse(cr, uid, [id.evaluation_id.id], context=context)[0].survey_request_ids + raise osv.except_osv(_('Warning!'), _("You cannot start evaluation without Appraisal.")) + records = hr_eval_obj.browse(cr, uid, [id.evaluation_id.id], context=context)[0].survey_response_ids for child in records: - if child.state == "draft": + if child.state == "new": wating_id = child.id continue if child.state != "done": flag = True if not flag and wating_id: self.survey_req_waiting_answer(cr, uid, [wating_id], context=context) - self.write(cr, uid, ids, { 'state': 'done'}, context=context) - return True - - def survey_req_draft(self, cr, uid, ids, context=None): - self.write(cr, uid, ids, { 'state': 'draft'}, context=context) + self.write(cr, uid, ids, {'state': 'done'}, context=context) return True def survey_req_cancel(self, cr, uid, ids, context=None): - self.write(cr, uid, ids, { 'state': 'cancel'}, context=context) + self.write(cr, uid, ids, {'state': 'cancel'}, context=context) return True def action_print_survey(self, cr, uid, ids, context=None): @@ -353,7 +363,7 @@ class hr_evaluation_interview(osv.osv): context = {} record = self.browse(cr, uid, ids, context=context) record = record and record[0] - context.update({'survey_id': record.survey_id.id, 'response_id': [record.response.id], 'response_no':0,}) + context.update({'survey_id': record.survey_id.id, 'response_id': [record.response_id.id], 'response_no': 0}) value = self.pool.get("survey").action_print_survey(cr, uid, ids, context=context) return value diff --git a/addons/hr_evaluation/hr_evaluation_data.xml b/addons/hr_evaluation/hr_evaluation_data.xml index 2afb5200091..8899c745aa4 100644 --- a/addons/hr_evaluation/hr_evaluation_data.xml +++ b/addons/hr_evaluation/hr_evaluation_data.xml @@ -19,11 +19,9 @@ Self Appraisal 20 - - 5 - open + restricted Employee Appraisal Form @@ -606,10 +604,8 @@ Once the form had been filled, the employee send it to his supervisor. Employee Opinion 20 - - 5 Process diff --git a/addons/hr_evaluation/hr_evaluation_view.xml b/addons/hr_evaluation/hr_evaluation_view.xml index 52b288fdad5..3298453ea4e 100644 --- a/addons/hr_evaluation/hr_evaluation_view.xml +++ b/addons/hr_evaluation/hr_evaluation_view.xml @@ -175,27 +175,26 @@ - +
-
- + -
-
@@ -285,13 +284,13 @@