[jira] [Commented] (OFBIZ-6791) Various fixes related with sales opportunity

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-6791) Various fixes related with sales opportunity

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-6791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15110443#comment-15110443 ]

Jacques Le Roux commented on OFBIZ-6791:
----------------------------------------

Come on Hans, you really wants to upset me :-o ?

No I did not miss it, but it seems you missed my anwser.
I see your point, I though thought that, with my explanation, you would debug/fix that yourself since you committed in 1st place.

I already answered you (*see thread link in description*) that to create an opportunity you need a leadPartyId. Because OOTB in UI (same with services) to create an opportunity you must
# use the EditSalesOpportunity request which loads the EditSalesOpportunity screen itself using the OpportunityForms.xml#EditSalesOpportunity form.
# This form calls the createSalesOpportunity request when it's not an update.
# The createSalesOpportunity request calls the createSalesOpportunity service.
# The createSalesOpportunity service triggers the SECAs which launch the createSalesOpportunityLeadRole and/or createSalesOpportunityAccountRole services if leadPartyId and/or accountPartyId (ONLY IF) is/are not empty.

Since leadPartyId and/or accountPartyId are no mandatory in EditSalesOpportunity form, nor in createSalesOpportunity/updateSalesOpportunity services, that's why you have this SECA condition, else you would not be able to create/update an opportunity.

I hope I gave you enough details to fix the issue. Hint: it's the list which should be fixed, not the SECA (and even less only one of them but the 4 - create/update for lead and account - if you would have been consistent but still wrong)

You really have a skill to have others working for you, don't you? :D

I'm an OFBiz volunteer, but sometimes I wonder... And today I better understand Adrian's harsh reaction on dev ML one day, please refer to it...

> Various fixes related with sales opportunity
> --------------------------------------------
>
>                 Key: OFBIZ-6791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6791
>             Project: OFBiz
>          Issue Type: Bug
>          Components: marketing, order
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: Upcoming Branch
>
>
> Hans directly committed
> * r1723007 "sales opportunity creation: required-field removed, disturbed the form submission, entityone no value field, error in the log"
> * and r1723248 "various fixes to the salesopportunity list and create functions"
> I wanted to backport them and create this Jira for releases notes. But before I checked the commits and found an issue, see http://markmail.org/message/mspie3qdjcac2tci



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)