hr_evaluation does not repect sequence when more than one phase is in wait

[Salton Massally]

Creating lets say 3 hr_evaluation_phases in an evaluation plan and setting the button two to wait will cause any evaluation created from that plan to jump to the phase right at the buttom once the first phase is completed rather than going to the next one.

 I investigated and in openerp/addons/hr_evaluation/hr_evaluation.py line 324, waiting_id is simply being reassigned and ends up being the value of the last id which passes through the loop. Since the phase ids are sorted in asc order by the sequence, it ends up being the phase in a waiting state which the greaters sequences.