diff --git a/addons/project/project.py b/addons/project/project.py index d75a2692019..5de70c12597 100644 --- a/addons/project/project.py +++ b/addons/project/project.py @@ -500,6 +500,7 @@ class task(osv.osv): _description = "Task" _log_create = True _date_name = "date_start" + _inherit = ['ir.needaction', 'mail.thread'] def _resolve_project_id_from_context(self, cr, uid, context=None): @@ -888,8 +889,7 @@ class task(osv.osv): if not task.date_end: vals.update({ 'date_end':time.strftime('%Y-%m-%d %H:%M:%S')}) self.write(cr, uid, [task.id],vals, context=context) - message = _("The task '%s' is done") % (task.name,) - self.log(cr, uid, task.id, message) + self.do_close_send_note(cr, uid, [task.id], context) return True def do_reopen(self, cr, uid, ids, context=None): @@ -909,6 +909,7 @@ class task(osv.osv): }, context=context) self.write(cr, uid, [task.id], {'state': 'open'}, context=context) + self.do_open_send_note(cr, uid, [task.id], context) return True def do_cancel(self, cr, uid, ids, context={}): @@ -927,9 +928,8 @@ class task(osv.osv): 'ref_doc1': 'project.task,%d' % task.id, 'ref_doc2': 'project.project,%d' % project.id, }, context=context) - message = _("The task '%s' is cancelled.") % (task.name,) - self.log(cr, uid, task.id, message) self.write(cr, uid, [task.id], {'state': 'cancelled', 'remaining_hours':0.0}, context=context) + self.do_cancel_send_note(cr, uid, [task.id], context) return True def do_open(self, cr, uid, ids, context={}): @@ -940,12 +940,12 @@ class task(osv.osv): if not t.date_start: data['date_start'] = time.strftime('%Y-%m-%d %H:%M:%S') self.write(cr, uid, [t.id], data, context=context) - message = _("The task '%s' is opened.") % (t.name,) - self.log(cr, uid, t.id, message) + self.do_open_send_note(cr, uid, [t.id], context) return True def do_draft(self, cr, uid, ids, context={}): self.write(cr, uid, ids, {'state': 'draft'}, context=context) + self.do_draft_send_note(cr, uid, ids, context) return True @@ -956,7 +956,7 @@ class task(osv.osv): for attachment_id in attachment_ids: new_attachment_ids.append(attachment.copy(cr, uid, attachment_id, default={'res_id': delegated_task_id}, context=context)) return new_attachment_ids - + def do_delegate(self, cr, uid, ids, delegate_data={}, context=None): """ @@ -984,20 +984,16 @@ class task(osv.osv): 'name': newname, }, context=context) if delegate_data['state'] == 'pending': - self.do_pending(cr, uid, task.id, context=context) + self.do_pending(cr, uid, [task.id], context=context) elif delegate_data['state'] == 'done': - self.do_close(cr, uid, task.id, context=context) - - message = _("The task '%s' has been delegated to %s.") % (delegate_data['name'], delegate_data['user_id'][1]) - self.log(cr, uid, task.id, message) + self.do_close(cr, uid, [task.id], context=context) + self.do_delegation_send_note(cr, uid, [task.id], context) delegated_tasks[task.id] = delegated_task_id return delegated_tasks def do_pending(self, cr, uid, ids, context={}): self.write(cr, uid, ids, {'state': 'pending'}, context=context) - for (id, name) in self.name_get(cr, uid, ids): - message = _("The task '%s' is pending.") % name - self.log(cr, uid, id, message) + self.do_pending_send_note(cr, uid, ids, context) return True def set_remaining_time(self, cr, uid, ids, remaining_time=1.0, context=None): @@ -1028,7 +1024,7 @@ class task(osv.osv): def set_kanban_state_done(self, cr, uid, ids, context=None): self.write(cr, uid, ids, {'kanban_state': 'done'}, context=context) - def _change_type(self, cr, uid, ids, next, *args): + def _change_type(self, cr, uid, ids, next, context=None): """ go to the next stage if next is False, go to previous stage @@ -1049,13 +1045,14 @@ class task(osv.osv): elif typeid and typeid in sorted_types and sorted_types.index(typeid) != len(sorted_types)-1: index = sorted_types.index(typeid) self.write(cr, uid, task.id, {'type_id': sorted_types[index+1]}) + self.state_change_send_note(cr, uid, [task.id], context) return True - def next_type(self, cr, uid, ids, *args): - return self._change_type(cr, uid, ids, True, *args) + def next_type(self, cr, uid, ids, context=None): + return self._change_type(cr, uid, ids, True, context=context) - def prev_type(self, cr, uid, ids, *args): - return self._change_type(cr, uid, ids, False, *args) + def prev_type(self, cr, uid, ids, context=None): + return self._change_type(cr, uid, ids, False, context=context) def _store_history(self, cr, uid, ids, context=None): for task in self.browse(cr, uid, ids, context=context): @@ -1072,9 +1069,10 @@ class task(osv.osv): return True def create(self, cr, uid, vals, context=None): - result = super(task, self).create(cr, uid, vals, context=context) - self._store_history(cr, uid, [result], context=context) - return result + task_id = super(task, self).create(cr, uid, vals, context=context) + self._store_history(cr, uid, [task_id], context=context) + self.create_send_note(cr, uid, [task_id], context=context) + return task_id # Overridden to reset the kanban_state to normal whenever # the stage (type_id) of the task changes. @@ -1085,13 +1083,14 @@ class task(osv.osv): new_stage = vals.get('type_id') vals_reset_kstate = dict(vals, kanban_state='normal') for t in self.browse(cr, uid, ids, context=context): - write_vals = vals_reset_kstate if t.type_id != new_stage else vals + write_vals = vals_reset_kstate if t.type_id != new_stage else vals super(task,self).write(cr, uid, [t.id], write_vals, context=context) result = True else: result = super(task,self).write(cr, uid, ids, vals, context=context) if ('type_id' in vals) or ('remaining_hours' in vals) or ('user_id' in vals) or ('state' in vals) or ('kanban_state' in vals): self._store_history(cr, uid, ids, context=context) + self.state_change_send_note(cr, uid, ids, context) return result def unlink(self, cr, uid, ids, context=None): @@ -1127,6 +1126,62 @@ class task(osv.osv): result += "\n" return result + + # --------------------------------------------------- + # OpenChatter methods and notifications + # --------------------------------------------------- + + def get_needaction_user_ids(self, cr, uid, ids, context=None): + result = dict.fromkeys(ids, []) + for obj in self.browse(cr, uid, ids, context=context): + if obj.state == 'draft' and obj.user_id: + result[obj.id] = [obj.user_id.id] + return result + + def message_get_subscribers(self, cr, uid, ids, context=None): + sub_ids = self.message_get_subscribers_ids(cr, uid, ids, context=context); + for obj in self.browse(cr, uid, ids, context=context): + if obj.user_id: + sub_ids.append(obj.user_id.id) + if obj.manager_id: + sub_ids.append(obj.manager_id.id) + return self.pool.get('res.users').read(cr, uid, sub_ids, context=context) + + def create_send_note(self, cr, uid, ids, context=None): + return self.message_append_note(cr, uid, ids, body=_("Task has been created."), context=context) + + def do_pending_send_note(self, cr, uid, ids, context=None): + if not isinstance(ids,list): ids = [ids] + msg = _('Task is now pending.') + return self.message_append_note(cr, uid, ids, body=msg, context=context) + + def do_open_send_note(self, cr, uid, ids, context=None): + msg = _('Task has been opened.') + return self.message_append_note(cr, uid, ids, body=msg, context=context) + + def do_cancel_send_note(self, cr, uid, ids, context=None): + msg = _('Task has been canceled.') + return self.message_append_note(cr, uid, ids, body=msg, context=context) + + def do_close_send_note(self, cr, uid, ids, context=None): + msg = _('Task has been closed.') + return self.message_append_note(cr, uid, ids, body=msg, context=context) + + def do_draft_send_note(self, cr, uid, ids, context=None): + msg = _('Task has been renewed.') + return self.message_append_note(cr, uid, ids, body=msg, context=context) + + def do_delegation_send_note(self, cr, uid, ids, context=None): + for task in self.browse(cr, uid, ids, context=context): + msg = _('Task has been delegated to %s.') % (task.user_id.name) + self.message_append_note(cr, uid, [task.id], body=msg, context=context) + return True + + def state_change_send_note(self, cr, uid, ids, context=None): + for task in self.browse(cr, uid, ids, context=context): + msg = _('Stage changed to %s') % (task.type_id.name) + self.message_append_note(cr, uid, [task.id], body=msg, context=context) + return True task() @@ -1222,7 +1277,7 @@ class project_task_history(osv.osv): for history in self.browse(cr, uid, ids, context=context): cr.execute('''select id - from + from project_task_history where task_id=%s and diff --git a/addons/project/project_view.xml b/addons/project/project_view.xml index f35a53576c2..7010031001c 100644 --- a/addons/project/project_view.xml +++ b/addons/project/project_view.xml @@ -291,12 +291,7 @@ - - - - - - + @@ -309,6 +304,7 @@ +