Comment 7 for bug 1212860

Revision history for this message
David Lefever @ Taktik (dl-taktik) wrote :

Hello,

the patch seems fine, but IMHO we shouldn't copy the tasks when we copy the phase:

map_phase_id[phase.id] = phase_obj.copy(cr, uid, phase.id, {'project_id': new_project_id}, context=context)
should be
map_phase_id[phase.id] = phase_obj.copy(cr, uid, phase.id, {'project_id': new_project_id, 'task_ids': []}, context=context)

Because just after, we copy all the tasks to the new phase.
I attached the new patch if someone wants to give it a try.