Hi,
As the createLead service is creating PartyRelationship record on the basis
of some parameters conditions so it should return the both partyId only, and
the code committed is improved code to make it more sensible so that the
partyId is now in whole service identified by the leadContactPartyId,
finally it just change way of outing the partyId. I think the code is fine
and creating a lead means tie up a party in the LEAD role with the
PartyGroup. I do not seen any problem in returning the leadContactPartyId as
partyId.
Rishi Solanki
Enterprise Software Developer
HotWax Media Pvt. Ltd.
On Wed, Jun 24, 2009 at 11:51 AM, Anil Patel <
[hidden email]>wrote:
> Adam,
> The service is returning id of two parties created in in the service.
> I don't see anything bad about it. Can you please explain why is it
> not good?
>
> Also in this modification I have changed variable name used in method,
> partyId out parameter was there before my this commit.
>
> Regards
> Anil Patel
>
>
>
> On Jun 23, 2009, at 11:46 PM, Adam Heath wrote:
>
>
[hidden email] wrote:
>>
>>> Author: apatel
>>> Date: Wed Jun 24 02:48:28 2009
>>> New Revision: 787890
>>>
>>> URL:
http://svn.apache.org/viewvc?rev=787890&view=rev>>> Log:
>>> Minor code clean up in createLead service. Convert lead now expires
>>> relationship instead of deleting it.
>>>
>>> Modified:
>>> ofbiz/trunk/applications/marketing/config/MarketingUiLabels.xml
>>>
>>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>>
>>> Modified:
>>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>> URL:
>>>
http://svn.apache.org/viewvc/ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml?rev=787890&r1=787889&r2=787890&view=diff>>>
>>> ==============================================================================
>>> ---
>>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>> (original)
>>> +++
>>> ofbiz/trunk/applications/marketing/script/org/ofbiz/sfa/lead/LeadServices.xml
>>> Wed Jun 24 02:48:28 2009
>>> @@ -52,15 +52,18 @@
>>> <set field="parameters.roleTypeId" value="LEAD"/>
>>> <call-simple-method
>>> method-name="createPersonRoleAndContactMechs"
>>> xml-resource="component://party/script/org/ofbiz/party/party/PartySimpleMethods.xml"/>
>>> <check-errors/>
>>> +
>>> + <set field="leadContactPartyId" from-field="partyId"/>
>>> + <clear-field field="partyId"/>
>>> <set field="partyRelationshipCtx.partyIdFrom"
>>> from-field="userLogin.partyId"/>
>>> - <set field="partyRelationshipCtx.partyIdTo"
>>> from-field="partyId"/>
>>> + <set field="partyRelationshipCtx.partyIdTo"
>>> from-field="leadContactPartyId"/>
>>>
>>
>> Service is createLead, but returns leadContactPartyId? That doesn't
>> seem sensible.
>>
>
>
Locked
