Merge lp:~openerp-dev/openobject-addons/6.1-opw-574989-xal into lp:openobject-addons/6.1
Status: | Approved |
---|---|
Approved by: | Naresh(OpenERP) |
Approved revision: | 6808 |
Proposed branch: | lp:~openerp-dev/openobject-addons/6.1-opw-574989-xal |
Merge into: | lp:openobject-addons/6.1 |
Diff against target: |
12 lines (+1/-1) 1 file modified
hr_holidays/hr_holidays.py (+1/-1) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-574989-xal |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Naresh(OpenERP) (community) | Approve | ||
Review via email: mp+106986@code.launchpad.net |
Description of the change
Hi,
This MP fix an AccessError when changing "Leave Type" field on an existing "Leave Request" when new "Leave Type" have a different value of "double_
As an admin user:
- Install module "hr_holidays"
- Modfify "Compenatory Days", check the "Double validation" field, then Save
- Make sure "demo" user have a employee configured
As demo user:
- Create a new leave request
* description = Test
* leave type = Legal Leaves (so no double validation)
* start date = TODAY
* end date = TODAY
- Save
- Edit and change leave type to "Compensatory Days" (so with double validation)
- Save
Current: raise access on "hr.holiday.status" (i.e. Leave Type)
Expected: no AccessError
Cheers,
Xavier
Unmerged revisions
- 6808. By Xavier ALT
-
[FIX] hr_holidays: leave request 'double_validation' related field should be readonly
Hello,
This bug is not reproducible on Trunk , also on runbot too . If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.
Thanks,
Naresh Soni