-
Notifications
You must be signed in to change notification settings - Fork 9.4k
code refactor, php8.x adjustmentts, param types, etc #39868
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
base: 2.4-develop
Are you sure you want to change the base?
code refactor, php8.x adjustmentts, param types, etc #39868
Conversation
Hi @jakwinkler. Thank you for your contribution!
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@magento run all tests |
1 similar comment
@magento run all tests |
@magento run all tests |
@magento create issue |
@jakwinkler, I do agree with these changes in general, but they most likely will break anyone who extended from these interfaces or classes. |
Description (*)
Just some type fixes + upgrade to php8.x
Needs to be tested and validated properly, as some method properties still do not match in core.
I've made this fix due to this code:
as it suppose to be an integer, but string has to be passed as the
$1
parameterContribution checklist (*)
Resolved issues: