-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Newsletter (un)subscription sent after changing email address #36440
Comments
Hi @Axel29. Thank you for your report.
Make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
For more details, review the Magento Contributor Assistant documentation. Add a comment to assign the issue: To learn more about issue processing workflow, refer to the Code Contributions.
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Hi @engcom-Dash. Thank you for working on this issue.
|
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://6e6c553486348bbc4d18f26bd2e4872a.instances.magento-community.engineering |
Hi @Axel29 , We are Worked this issue in magento 2.4.4 instance...We are able to reproduce the issue with respective results. Pre condition: Steps to reproduce: While subscribe the current email id i received subscription mail as well as while changing the customer email i received the subscription email id. Let us know if we missing anything Thanks!!! |
Unfortunately, not enough information was provided to create a Jira ticket. Please make sure you added the following label(s): Once all required labels are present, please add |
Unfortunately, not enough information was provided to create a Jira ticket. Please make sure you added the following label(s): Once all required labels are present, please add |
❌ Something went wrong. Cannot create Jira issue. |
Hi @engcom-Dash , I edited the post to make it more clear. The bug is simply that there should NEVER be a subscription or unsubscription confirmation email sent when changing the customer email address. The action of changing my account's email address has absolutely nothing to do with the newsletter whatsoever, and it is more than confusing to recieve a subscription confirmation email when I simply change my email address... Hoping this is more clear this way. |
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, unfortunately there is no ability to deploy Magento instance at the moment. Please try again later. |
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, unfortunately there is no ability to deploy Magento instance at the moment. Please try again later. |
Hi @Axel29 , Thanks for your contribution and information..... We got email address which is received when customer change the customer email address as well as we didn't receive any subscription related email when customer changing the email address. Thanks!!!! |
✔️ Issue confirmed Issue got reproduced in 2.4-develop branch. Description: Newsletter (un)subscription sent after changing email address. Pre-requisite:
Steps to reproduce:
Expected result:
Actual result: |
Hi @engcom-Lima. Thank you for working on this issue.
|
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/AC-7091 |
❌ You don't have permission to export this issue. |
Preconditions and environment
Steps to reproduce
Expected result
Actual result
The bug is simply that there should not be a subscription or unsubscription confirmation email sent when changing the customer email address.
The action of changing a customer account's email address has absolutely nothing to do with the newsletter whatsoever, and it is more than confusing to recieve a subscription confirmation email when I simply change my email address...
Additional information
This bug seems to be introduced in commit 47e3ada on the class
\Magento\Newsletter\Model\Plugin\CustomerPlugin
with the following line:return $customer->getEmail() !== $subscriber->getEmail() || (int)$subscriber->getCustomerId() === 0;
This is because when changing the email, the customer gets saved, triggering this Plugin and then the method suggests that the email should be sent but this has no sense.
Also, the same bug occurs if the customer is unsubscribed from the newsletter.
In this case, the email sent when changing the customer's email is not the subscription confirmation but the unsubscribe confirmation.
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: