Merge lp:~openerp-dev/openobject-addons/trunk-bug-1021788-jir into lp:openobject-addons

Proposed by Jignesh Rathod(OpenERP)
Status: Work in progress
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1021788-jir
Merge into: lp:openobject-addons
Diff against target: 21 lines (+2/-2)
1 file modified
mrp/mrp.py (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1021788-jir
Reviewer Review Type Date Requested Status
Amit Parik (community) Needs Fixing
OpenERP Core Team Pending
Review via email: mp+114110@code.launchpad.net

Description of the change

Hello ,

Solve the problem of workorder costing.currently workorder costing
calculate base on number of hours instead of working hours and it should be based on
working hours.

Thank you.

To post a comment you must log in.
7009. By jir <jir@jir-desktop>

[FIX] Solve the problem of workordercosting.

Revision history for this message
Amit Parik (amit-parik) wrote :

Good work!

review: Approve
7010. By Jignesh Rathod(OpenERP)

[MERGE]: Merge with lp:openobject-addons

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Jignesh,

It should be fixed on mrp_opertation.py not on mrp.py.

Thanks!

Revision history for this message
Amit Parik (amit-parik) :
review: Needs Fixing

Unmerged revisions

7010. By Jignesh Rathod(OpenERP)

[MERGE]: Merge with lp:openobject-addons

7009. By jir <jir@jir-desktop>

[FIX] Solve the problem of workordercosting.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mrp/mrp.py'
2--- mrp/mrp.py 2012-08-15 20:51:43 +0000
3+++ mrp/mrp.py 2012-08-23 06:56:20 +0000
4@@ -818,7 +818,7 @@
5 for wc_line in production.workcenter_lines:
6 wc = wc_line.workcenter_id
7 if wc.costs_journal_id and wc.costs_general_account_id:
8- value = wc_line.hour * wc.costs_hour
9+ value = wc_line.delay * wc.costs_hour
10 account = wc.costs_hour_account_id.id
11 if value and account:
12 amount += value
13@@ -830,7 +830,7 @@
14 'journal_id': wc.costs_journal_id.id,
15 'ref': wc.code,
16 'product_id': wc.product_id.id,
17- 'unit_amount': wc_line.hour,
18+ 'unit_amount': wc_line.delay,
19 'product_uom_id': wc.product_id.uom_id.id
20 } )
21 if wc.costs_journal_id and wc.costs_general_account_id:

Subscribers

People subscribed via source and target branches

to all changes: