Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Call to a member function getId() on null #1805

Closed
yuriichayka opened this issue Sep 5, 2023 · 0 comments · Fixed by #1806
Closed

Call to a member function getId() on null #1805

yuriichayka opened this issue Sep 5, 2023 · 0 comments · Fixed by #1806
Assignees
@gonzaloebiz
Labels
bug Magento 2.4
Milestone
10x.x.56

Comments

@yuriichayka
Copy link

yuriichayka commented Sep 5, 2023 •
edited
Loading 

1. Adobe Commerce 2.4.5-p4, Cloud
2. mc-magento2 103.4.55.
3. Configuration->Customers->Newsletter->Need to Confirm set to Yes
4. Configuration->Mailchimp->Mailchimp General Configuration->Magento always manage emails  set to No

Steps to reproduce

1. Register new user
2. Go to my account/newsletter and try to sign up for newsletter

Actual and Expected result

Expected result: The page works fine
Actual result: 500 response error appears("There has been an error processing your request" )

Additional information

Seems this is result of fixing 1311 bug, https://github.com/mailchimp/mc-magento2/pull/1756/files#diff-a4e2d144b3750bad4a40c519a18b504c83aaa8550210eb69670e6cfe04bdf3bc in particular
Stack Trace: /mailchimp/mc-magento2/Block/Newsletter.php75
gonzaloebiz added a commit that referenced this issue Sep 5, 2023
@gonzaloebiz
closes #1805 for magento 2.4
d257dee
@gonzaloebiz gonzaloebiz self-assigned this Sep 5, 2023
@gonzaloebiz gonzaloebiz added this to the 10x.x.56 milestone Sep 5, 2023
@gonzaloebiz gonzaloebiz linked a pull request Sep 5, 2023 that will close this issue
closes #1805 for magento 2.4 #1806
Merged
gonzaloebiz added a commit that referenced this issue Sep 5, 2023
@gonzaloebiz
closes #1805 for magento 2.4
f16642d
gonzaloebiz added a commit that referenced this issue Sep 5, 2023
@gonzaloebiz
Merge pull request #1806 from mailchimp/Issue1805-2.4
9fbc163 
closes #1805 for magento 2.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gonzaloebiz gonzaloebiz

Labels
bug Magento 2.4
Projects
None yet
Milestone
10x.x.56
Development

Successfully merging a pull request may close this issue.

closes #1805 for magento 2.4 mailchimp/mc-magento2
2 participants
@gonzaloebiz @yuriichayka