[jira] Created: (OFBIZ-4001) Error in service signUpForContactList

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

[jira] Created: (OFBIZ-4001) Error in service signUpForContactList

Nicolas Malin (Jira)
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.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-4001) Error in service signUpForContactList

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: 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.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-4001) Error in service signUpForContactList

Nicolas Malin (Jira)
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.

Reply | Threaded
Open this post in threaded view
|

[jira] Commented: (OFBIZ-4001) Error in service signUpForContactList

Nicolas Malin (Jira)
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.

Reply | Threaded
Open this post in threaded view
|

[jira] Closed: (OFBIZ-4001) Error in service signUpForContactList

Nicolas Malin (Jira)
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.