Comment 7 for bug 1082009

Revision history for this message
Anto (abourguignon) wrote :

Hello,

Thanks for your intervention Amit ! After a review with al, fp and qdp, the behavior of the hr_holidays' calendar view seems ok to us:
- any administrator, hr officier or hr manager can move a confirmed leave request anywhere, which is normal.
- any other user (hr employee) won't be able to do so: drag'n'dropping the request somewhere else will result in an error. This behavior has been introduced by fp's merge, revno 7969, which overrides the write() method of hr_holidays.

That beeing, as you said, read only values should prevent a leave request from beeing moved in such cases (gantt and calendar views). Moreover, it's not really efficient to override the write() method every time we need that kind of feature.
But this is a more important problem that will be handled later. Two tasks have just been created regarding this issue:
- Calendar's drag'n'drop usability
- Model state field inducing readonly mechanism

Hope that helped ;)

Regards,
Anto