Comment 2 for bug 1218036

Thanks Michael. I think based on my experience that the above functionality is very important. It is nice to have in your modules. I will explain why:
The manager of the department is the person who manage all the issues of his department. Suppose that he plan to do some tasks in the next days [seminars, submit proposals, finished tasks] and he needs all his employees to be with him. Then he will refuse to give any one a leave for example for the next days.
Instead of doing this manually, why do not give him the privilege to do this through the system.
In the past we asked the employee to assign the form of the leave from his manager. We can do the same by the computer through th HR system.
So I think the system should record the approve [or descion] of the manager together with a short explanation why he took this descion.

But what about if we have a list of a managers over that employee then what we should do: one solution is to give a default case for example just to have approve from the dirct manager and also give the flixibility to specify what level of managers the approve should be taken.
Any way, I think that functionality should be added.
Note that in the original hr modules from openerp this is bug even that in their documents they said that their system worked in the way I mentioned above.

As another thing that I did not check in your modules: If one of the HR officers ask for a leave then who will approve it for him [Can he appove it for him self]

Thanks again