Merge lp:~openerp-dev/openobject-addons/6.1-opw-580642-cbi into lp:openobject-addons/6.1
Status: | Needs review |
---|---|
Proposed branch: | lp:~openerp-dev/openobject-addons/6.1-opw-580642-cbi |
Merge into: | lp:openobject-addons/6.1 |
Diff against target: |
47 lines (+15/-17) 2 files modified
hr_timesheet/hr_timesheet.py (+1/-3) hr_timesheet/security/ir.model.access.csv (+14/-14) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-580642-cbi |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
OpenERP Core Team | Pending | ||
Review via email: mp+135355@code.launchpad.net |
Description of the change
The issue that was encountered: With hr_timesheet_sheet and project installed, employees were not able to delete their own timesheet lines due to an access rule problem.
The reason: A super call in an unlink method tried to delete the record when it was already gone because of a cascading delete.
The fix: I removed the super call, since it is useless. I also included the changes to the access rights from trunk, which would otherwise still prevent employees from deleting their own timesheet lines.
Unmerged revisions
- 7075. By Chris Biersbach (OpenERP)
-
[FIX] Employees can now delete their own timesheet lines. The access rights are a backport from trunk, the rest removes an unnecessary super call that caused an access rule to incorrectly not pass