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.

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

Update AbstractForm.php to resolve error in magento 2.4.5 #25

Closed

Conversation

matteorossi-thespacesm
Copy link

Added override of \Magento\Backend\Block\Widget\Form method getForm to resolve Error: "RemovePasswordAndUserConfirmationFormFieldsPlugin::afterGetForm(): Argument #2 ($result) must be of type Magento\Framework\Data\Form, null" in magento 2.4.5, as addressed in magento/magento2#36145

Added override of \Magento\Backend\Block\Widget\Form method getForm to resolve Error: "RemovePasswordAndUserConfirmationFormFieldsPlugin::afterGetForm(): Argument m2epro#2 ($result) must be of type Magento\Framework\Data\Form, null" in magento 2.4.5, as addressed in magento/magento2#36145
@HannaKap
Copy link
Collaborator

@matteorossi-thespacesm Thank you for the suggested improvement, we are about to implement this kind of change in one of our future releases.

@HannaKap HannaKap closed this Apr 13, 2023
@matteorossi-thespacesm
Copy link
Author

I'd want to observe that mine is not a suggestion but a bug fix.
We are to deploy a magento 2.4.5 -p2 site and we can't use your module because it is bugged!

@chrisastley
Copy link

I'd want to observe that mine is not a suggestion but a bug fix. We are to deploy a magento 2.4.5 -p2 site and we can't use your module because it is bugged!

What issue does it cause you being the way it is now? Just curious as I'm already using M2ePro on the latest version of Magento with no apparent bugs.

@matteorossi-thespacesm
Copy link
Author

I can't open the wizard to migrate from magento 1.9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants