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

Proposed by Somesh Khare
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-582705-opw-skh
Merge into: lp:openobject-addons
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/trunk-582705-opw-skh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+138654@code.launchpad.net

Description of the change

Hello Sir,

[FIX]sale_crm: wrong salesperson 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 "Salesperson" information on the newly created Quotation, It should be either Salesperson 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

8228. 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
=== modified file 'sale_crm/wizard/crm_make_sale.py'
--- sale_crm/wizard/crm_make_sale.py 2012-11-29 22:26:45 +0000
+++ sale_crm/wizard/crm_make_sale.py 2012-12-07 07:46:22 +0000
@@ -102,7 +102,7 @@
102 'fiscal_position': fpos,102 'fiscal_position': fpos,
103 }103 }
104 if partner.id:104 if partner.id:
105 vals['user_id'] = partner.user_id and partner.user_id.id or uid105 vals['user_id'] = case.user_id and case.user_id.id or partner.user_id and partner.user_id.id or uid
106 new_id = sale_obj.create(cr, uid, vals, context=context)106 new_id = sale_obj.create(cr, uid, vals, context=context)
107 sale_order = sale_obj.browse(cr, uid, new_id, context=context)107 sale_order = sale_obj.browse(cr, uid, new_id, context=context)
108 case_obj.write(cr, uid, [case.id], {'ref': 'sale.order,%s' % new_id})108 case_obj.write(cr, uid, [case.id], {'ref': 'sale.order,%s' % new_id})

Subscribers

People subscribed via source and target branches

to all changes: