|
|
|
Adds just like sales and purchase a new parties link
Please review
Patch Set 1 #
Total comments: 4
Total messages: 6
|
Paul J Stevens
Looks like this is a patch against the 1.6 branch, correct?
|
15 years, 2 months ago (2010年10月26日 13:59:57 UTC) #1 | ||||||||||||||||||||||
Looks like this is a patch against the 1.6 branch, correct?
Yes of course, on approval of this I could propose the 1.8 branch patch too. Tryton maintains last 3 releases and sale_opportunities is avalilable since 1.6 On 2010年10月26日 13:59:57, Paul J Stevens wrote: > Looks like this is a patch against the 1.6 branch, correct?
On 2010年10月27日 02:13:05, sharoonthomas wrote: > Yes of course, on approval of this I could propose the 1.8 branch patch too. > > Tryton maintains last 3 releases and sale_opportunities is avalilable since 1.6 But any change of this kind will be applied on series.
Please update it to trunk http://codereview.appspot.com/2736043/diff/1/opportunity.py File opportunity.py (right): http://codereview.appspot.com/2736043/diff/1/opportunity.py#newcode870 opportunity.py:870: Select distinct parties from the missing ending comment http://codereview.appspot.com/2736043/diff/1/opportunity.py#newcode880 opportunity.py:880: 'SELECT DISTINCT("party") FROM "%s" WHERE "state" IN ' + No need of "+" Could you reindent this way: SELECT DISCTINCT(party) FROM "%" WHERE state IN ('lead', 'opportunity') It is more readable to break on SQL statement like SELECT, FROM etc. http://codereview.appspot.com/2736043/diff/1/opportunity.xml File opportunity.xml (right): http://codereview.appspot.com/2736043/diff/1/opportunity.xml#newcode765 opportunity.xml:765: <record model="ir.action.wizard" id="act_open_lead_opportunity"> act_opened_lead_opportunity_party http://codereview.appspot.com/2736043/diff/1/opportunity.xml#newcode766 opportunity.xml:766: <field name="name">Parties associated to open Leads and Opportunities</field> "Opened Leads" to keep the same wording.