Merge lp:~openerp-dev/openobject-addons/6.1-opw-574989-xal into lp:openobject-addons/6.1

Proposed by Xavier ALT
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
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_validation".

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

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

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

Unmerged revisions

6808. By Xavier ALT

[FIX] hr_holidays: leave request 'double_validation' related field should be readonly

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hr_holidays/hr_holidays.py'
--- hr_holidays/hr_holidays.py 2012-01-03 13:52:24 +0000
+++ hr_holidays/hr_holidays.py 2012-05-23 11:23:23 +0000
@@ -136,7 +136,7 @@
136 'category_id': fields.many2one('hr.employee.category', "Category", help='Category of Employee'),136 'category_id': fields.many2one('hr.employee.category', "Category", help='Category of Employee'),
137 'holiday_type': fields.selection([('employee','By Employee'),('category','By Employee Category')], 'Allocation Type', help='By Employee: Allocation/Request for individual Employee, By Employee Category: Allocation/Request for group of employees in category', required=True),137 'holiday_type': fields.selection([('employee','By Employee'),('category','By Employee Category')], 'Allocation Type', help='By Employee: Allocation/Request for individual Employee, By Employee Category: Allocation/Request for group of employees in category', required=True),
138 'manager_id2': fields.many2one('hr.employee', 'Second Approval', readonly=True, help='This area is automaticly filled by the user who validate the leave with second level (If Leave type need second validation)'),138 'manager_id2': fields.many2one('hr.employee', 'Second Approval', readonly=True, help='This area is automaticly filled by the user who validate the leave with second level (If Leave type need second validation)'),
139 'double_validation': fields.related('holiday_status_id', 'double_validation', type='boolean', relation='hr.holidays.status', string='Apply Double Validation'),139 'double_validation': fields.related('holiday_status_id', 'double_validation', type='boolean', relation='hr.holidays.status', string='Apply Double Validation', readonly=True),
140 }140 }
141 _defaults = {141 _defaults = {
142 'employee_id': _employee_get,142 'employee_id': _employee_get,