-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Remove duplicate declaration #23794
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
Remove duplicate declaration #23794
Conversation
On line 33 and 37 there is a duplicate declaration of $this->formKey.
|
Hi @gfernandes410. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
lbajsarowicz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
|
Hi @lbajsarowicz, thank you for the review.
|
|
@gfernandes410 thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
|
✔️ QA Passed |
|
Hi @gfernandes410, thank you for your contribution! |
On line 33 and 37 there is a duplicate declaration of $this->formKey
Description (*)
Deleted dupliquete lines on __construct function.
Manual testing scenarios (*)
Questions or comments
With this change the code will be more optimized and clean.
Contribution checklist (*)