Error in service signUpForContactList
------------------------------------- Key: OFBIZ-4001 URL: https://issues.apache.org/jira/browse/OFBIZ-4001 Project: OFBiz Issue Type: Bug Components: marketing Affects Versions: SVN trunk Reporter: Willem Janssen Fix For: SVN trunk Attachments: signupforcontactlist.docx The service signUpForContactList does not store the emailaddress of the new entry to the list, in stead it uses the emailaddress of the first partyId _NA_ to store to the list. See the attached document for this issue in the ofbiz demo environment. The service signUpForContactList calls service createContactListParty that has a check on a missing contactMechId. However this parameter is not a parameter that is used by the signUpForContactList service. So the check on the contactMechId has always the same result, it is missing! signUpForContactList uses parametes prefferedContactMechId so I made a patch that adds the check on this parameter to createContactListParty. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
[ https://issues.apache.org/jira/browse/OFBIZ-4001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Willem Janssen updated OFBIZ-4001: ---------------------------------- Attachment: signupforcontactlist.docx > Error in service signUpForContactList > ------------------------------------- > > Key: OFBIZ-4001 > URL: https://issues.apache.org/jira/browse/OFBIZ-4001 > Project: OFBiz > Issue Type: Bug > Components: marketing > Affects Versions: SVN trunk > Reporter: Willem Janssen > Fix For: SVN trunk > > Attachments: signupforcontactlist.docx > > > The service signUpForContactList does not store the emailaddress of the new entry to the list, in stead it uses the emailaddress of the first partyId _NA_ to store to the list. > See the attached document for this issue in the ofbiz demo environment. > The service signUpForContactList calls service createContactListParty that has a check on a missing contactMechId. However this parameter is not a parameter that is used by the signUpForContactList service. So the check on the contactMechId has always the same result, it is missing! > signUpForContactList uses parametes prefferedContactMechId so I made a patch that adds the check on this parameter to createContactListParty. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-4001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Willem Janssen updated OFBIZ-4001: ---------------------------------- Attachment: ContactListServices.diff > Error in service signUpForContactList > ------------------------------------- > > Key: OFBIZ-4001 > URL: https://issues.apache.org/jira/browse/OFBIZ-4001 > Project: OFBiz > Issue Type: Bug > Components: marketing > Affects Versions: SVN trunk > Reporter: Willem Janssen > Fix For: SVN trunk > > Attachments: ContactListServices.diff, signupforcontactlist.docx > > > The service signUpForContactList does not store the emailaddress of the new entry to the list, in stead it uses the emailaddress of the first partyId _NA_ to store to the list. > See the attached document for this issue in the ofbiz demo environment. > The service signUpForContactList calls service createContactListParty that has a check on a missing contactMechId. However this parameter is not a parameter that is used by the signUpForContactList service. So the check on the contactMechId has always the same result, it is missing! > signUpForContactList uses parametes prefferedContactMechId so I made a patch that adds the check on this parameter to createContactListParty. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-4001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12933019#action_12933019 ] Eric DE MAULDE commented on OFBIZ-4001: --------------------------------------- When I send a communation event to all my contact list, it works fine. It's true, I prefer to see related email adress for each party contact mech. > Error in service signUpForContactList > ------------------------------------- > > Key: OFBIZ-4001 > URL: https://issues.apache.org/jira/browse/OFBIZ-4001 > Project: OFBiz > Issue Type: Bug > Components: marketing > Affects Versions: SVN trunk > Reporter: Willem Janssen > Fix For: SVN trunk > > Attachments: ContactListServices.diff, signupforcontactlist.docx > > > The service signUpForContactList does not store the emailaddress of the new entry to the list, in stead it uses the emailaddress of the first partyId _NA_ to store to the list. > See the attached document for this issue in the ofbiz demo environment. > The service signUpForContactList calls service createContactListParty that has a check on a missing contactMechId. However this parameter is not a parameter that is used by the signUpForContactList service. So the check on the contactMechId has always the same result, it is missing! > signUpForContactList uses parametes prefferedContactMechId so I made a patch that adds the check on this parameter to createContactListParty. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-4001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacques Le Roux closed OFBIZ-4001. ---------------------------------- Resolution: Fixed Fix Version/s: Release Branch 10.04 Release Branch 09.04 Assignee: Jacques Le Roux Thanks Willen, Your patch is in trunk at r1037567, R10.04 at r1037568, R9.04 at r1037570 Please don't use tabs but spaces in your future patches (see why at [Contributors Best Practices|https://cwiki.apache.org/confluence/display/OFBADMIN/OFBiz+Contributors+Best+Practices]) > Error in service signUpForContactList > ------------------------------------- > > Key: OFBIZ-4001 > URL: https://issues.apache.org/jira/browse/OFBIZ-4001 > Project: OFBiz > Issue Type: Bug > Components: marketing > Affects Versions: SVN trunk > Reporter: Willem Janssen > Assignee: Jacques Le Roux > Fix For: Release Branch 09.04, Release Branch 10.04, SVN trunk > > Attachments: ContactListServices.diff, signupforcontactlist.docx > > > The service signUpForContactList does not store the emailaddress of the new entry to the list, in stead it uses the emailaddress of the first partyId _NA_ to store to the list. > See the attached document for this issue in the ofbiz demo environment. > The service signUpForContactList calls service createContactListParty that has a check on a missing contactMechId. However this parameter is not a parameter that is used by the signUpForContactList service. So the check on the contactMechId has always the same result, it is missing! > signUpForContactList uses parametes prefferedContactMechId so I made a patch that adds the check on this parameter to createContactListParty. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
Free forum by Nabble | Edit this page |