Merge lp:~openerp-dev/openobject-addons/7.0-opw-597447-fka into lp:openobject-addons/7.0

Proposed by Foram Katharotiya (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/7.0-opw-597447-fka
Merge into: lp:openobject-addons/7.0
Diff against target: 12 lines (+2/-1)
1 file modified
project_timesheet/project_timesheet.py (+2/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-597447-fka
Reviewer Review Type Date Requested Status
Vinay Rana (OpenERP) Pending
Review via email: mp+185967@code.launchpad.net

Description of the change

Hello,

I have fixed the bug of worklog entry in task.

Steps to reproduce:

- Go to Project -> Tasks
- Register any hours in the worklog in a open task, example in 2013-04-13.
- Go to Human Resources -> My timesheet
- Register a timesheet between 2013-04-15 to 2013-04-21.
- Send to approval.Approval it.
- Go to Project -> Tasks
- Register any new hours in the working log in the same open task with other user with no timesheet, in 2013-04-30.
- Error, You cannot modify hours in a confirmed timesheet

Thanks,
FKA

To post a comment you must log in.

Unmerged revisions

9417. By Foram Katharotiya (OpenERP)

[MERGE] with 7.0

9416. By Foram Katharotiya (OpenERP)

[FIX] can not register worklog with other user with no timesheet when the any one timesheet is confirmed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'project_timesheet/project_timesheet.py'
2--- project_timesheet/project_timesheet.py 2013-06-07 11:38:29 +0000
3+++ project_timesheet/project_timesheet.py 2013-09-17 05:05:57 +0000
4@@ -200,7 +200,8 @@
5 if amount_unit and 'amount' in amount_unit.get('value',{}):
6 vals_line['amount'] = amount_unit['value']['amount']
7
8- self.pool.get('hr.analytic.timesheet').write(cr, uid, [line_id.id], vals_line, context=context)
9+ if vals_line:
10+ self.pool.get('hr.analytic.timesheet').write(cr, uid, [line_id.id], vals_line, context=context)
11
12 return super(project_work,self).write(cr, uid, ids, vals, context)
13