Odoo Addons (MOVED TO GITHUB)

Merge lp:~openerp-dev/openobject-addons/5.0-opw-16818-ado into lp:openobject-addons/5.0

  1. 5.0-opw-16818-ado
  2. Merge into 5.0
Proposed by Amit Dodiya (OpenERP)
Status: Merged
Approved by: Jay Vora (Serpent Consulting Services)
Approved revision: no longer in the source branch.
Merged at revision: 2914
Proposed branch: lp:~openerp-dev/openobject-addons/5.0-opw-16818-ado
Merge into: lp:openobject-addons/5.0
Diff against target: 12 lines (+1/-1)
1 file modified
point_of_sale/pos.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/5.0-opw-16818-ado
Reviewer Review Type Date Requested Status
Priyesh (OpenERP) Pending
Anup(SerpentCS) Pending
Jay Vora (Serpent Consulting Services) Pending
Review via email: mp+71996@code.launchpad.net

Description of the change

Hello Sir,

[FIX]: [5.0 - POS] stock move lines coming from point of sale do not have correct content case:(16818)

Current behaviour: The "return packing" wizard show the description : stock move (POS id): It is impossible to choose the product that is returned.

I have changed the code in "pos.py" file now it shows name of pos instead of stock move (POS id).

Thanks.
Amit

To post a comment you must log in.
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Amit,

I recommend to keep the self.pool.get() out of the for loop.

Thanks.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'point_of_sale/pos.py'
2--- point_of_sale/pos.py 2010-09-14 15:17:04 +0000
3+++ point_of_sale/pos.py 2011-08-18 07:35:36 +0000
4@@ -349,7 +349,7 @@
5 (location_id, stock_dest_id)= (stock_dest_id, location_id)
6
7 self.pool.get('stock.move').create(cr, uid, {
8- 'name': 'Stock move (POS %d)' % (order.id, ),
9+ 'name': 'Stock move (%s) ' % (order.name, ) + line.product_id.partner_ref,
10 'product_uom': line.product_id.uom_id.id,
11 'product_uos': line.product_id.uom_id.id,
12 'picking_id': picking_id,