Merge lp:~openerp-dev/openobject-addons/6.1-opw-582705-skh into lp:openobject-addons/6.1

Proposed by Somesh Khare
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-582705-skh
Merge into: lp:openobject-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
sale_crm/wizard/crm_make_sale.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-582705-skh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+138651@code.launchpad.net

Description of the change

Hello Sir,

[FIX]sale_crm: wrong salesman info on Quotation when convert to Quote from Opportunity.

Install sale_crm module.
Steps:
1. Create an Opportunity and set the user as "Demo User"
2. Press button "Convert To Quote"

See the "Salesman" information on the newly created Quotation, It should be either Salesman on the Opportunity or Partner User or logged in user. Here in the code it missed the Opportunity user.

Kindly review the branch and please share your views on it.

Thanks,
Somesh Khare

To post a comment you must log in.

Unmerged revisions

7096. By Somesh Khare

[FIX]sale_crm: wrong salesman info on Quotation when convert to Quote from Opportunity (Case: ref 582705)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'sale_crm/wizard/crm_make_sale.py'
2--- sale_crm/wizard/crm_make_sale.py 2012-02-14 12:25:20 +0000
3+++ sale_crm/wizard/crm_make_sale.py 2012-12-07 07:38:21 +0000
4@@ -101,7 +101,7 @@
5 'fiscal_position': fpos,
6 }
7 if partner.id:
8- vals['user_id'] = partner.user_id and partner.user_id.id or uid
9+ vals['user_id'] = case.user_id and case.user_id.id or partner.user_id and partner.user_id.id or uid
10 new_id = sale_obj.create(cr, uid, vals, context=context)
11 sale_order = sale_obj.browse(cr, uid, new_id, context=context)
12 case_obj.write(cr, uid, [case.id], {'ref': 'sale.order,%s' % new_id})