Timesheet validation Sign in/Sign out count check
Bug #616258 reported by
Vincent Renaville@camptocamp
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Odoo Addons (MOVED TO GITHUB) |
Fix Released
|
Undecided
|
Ravindra Mekhiya(OpenERP) | ||
Bug Description
Hello,
I have a problem and a patch for a use case concerning timesheet :
- an employee make his timesheet for the week
- He forget to make his last Signout on Friday
- He confirm the timesheet
- The next week, he start a new timesheet, but he cannot made a new signin because signin must follow a signout
- He decide to change the timesheet from last week, but he can't because in his company only de HR manager can unconfirm timesheet
So i have made a patch on button_confirm, that check that count signin an signout and raise a error if he found a difference when you confirm your timesheet
Thanks for your Help
Vincent
Related branches
Revision history for this message
| Vincent Renaville@camptocamp (vrenaville-c2c) wrote | #1 |
| Changed in openobject-addons: | |
| assignee: | nobody → RME(OpenERP) (rme-openerp) |
| Changed in openobject-addons: | |
| status: | New → Confirmed |
Revision history for this message
| Ravindra Mekhiya(OpenERP) (rme-openerp) wrote | #2 |
Revision history for this message
| Vincent Renaville@camptocamp (vrenaville-c2c) wrote | #3 |
Revision history for this message
| Ravindra Mekhiya(OpenERP) (rme-openerp) wrote | #4 |
Revision history for this message
| Jay Vora (Serpent Consulting Services) (jayvora) wrote | #5 |
| Changed in openobject-addons: | |
| milestone: | none → 5.0.13 |
| status: | Confirmed → Fix Released |
To post a comment you must log in.
Hello Vincent,
Should not the check be made before the allowing workflow signal?
I meant, your method should be called just after the for loop starts.
Thanks.