OpenERP Community Backports (Addons)

[trunk/7.0][stock] Valuation move lines do not include UoM

  1. Series 6.1
  2. Bug #1179705
Bug #1179705 reported by Mohammad Alhashash
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Confirmed
Medium
OpenERP R&D Addons Team 2
OpenERP Community Backports (Addons)
Status tracked in 7.0
6.1
New
Undecided
Unassigned
7.0
Fix Committed
Undecided
Unassigned

Bug Description

Create a reception or delivery of a stockable product and process it. Either check the account lines of the valuation entry in the database or open Journal Items > Stock Journal and modify the tree view through the developer mode to show the product_id, product_uom_id, and quantity fields.

While the quantity field has the stock move quantity in move UoM, the product_uom_id is always NULL! Without unit of measure, the quantity value is meaningless. Having these fields in journal items is very useful for reporting.

The problem cause is that method _create_account_move_line() of stock.move does not add the product_uom_id to the created lines.

Regards,

Mohammad Alhashash

See original description

Related branches

lp://staging/~alhashash/openobject-addons/7.0-alhashash-bug1179705
OpenERP Core Team: Pending requested
Diff: 19 lines (+2/-0)
1 file modified
stock/stock.py (+2/-0)
lp://staging/~alhashash/ocb-addons/7.0-alhashash-bug1179705
Holger Brunn (Therp): Approve (code review)
Pedro Manuel Baeza: Approve (code review)
Yann Papouin: Approve (code review)
Diff: 19 lines (+2/-0)
1 file modified
stock/stock.py (+2/-0)
Mohammad Alhashash (alhashash)
description: updated
Revision history for this message
Amit Parik (amit-parik) wrote : Re: [trunk/7.0][stock] Valuation move lines does include UoM

Hello Mohammad,

Nice caught..!!! Faced the same problem.

Thank you!

summary: - [7.0][stock] Valuation move lines does include UoM
+ [trunk/7.0][stock] Valuation move lines does include UoM
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Amit Parik (amit-parik) wrote :

Want to add one more point, Invoice from stock valuation correctly write the uom on journal item.

Thank you!

Mohammad Alhashash (alhashash)
summary: - [trunk/7.0][stock] Valuation move lines does include UoM
+ [trunk/7.0][stock] Valuation move lines does not include UoM
Amit Parik (amit-parik)
tags: added: warehosue
tags: added: warehouse
removed: warehosue
Mohammad Alhashash (alhashash)
summary: - [trunk/7.0][stock] Valuation move lines does not include UoM
+ [trunk/7.0][stock] Valuation move lines do not include UoM
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Mohammed,

I'm removing OCB-addons as an affected project for this bug as per project policy. Of course the bug affects ocb-addons/6.1, but we only want to track bugs that we fix in OCB-specific changes. Therefore, you should only add bugs to the project that you are going to propose a solution for.

Thanks,
Stefan.

no longer affects: ocb-addons
no longer affects: ocb-addons/7.0
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I have added again series in OCB project, because a patch has been proposed for 7.0.

Regards.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.