-
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
Remove sku from operators to validate condition #20412
Remove sku from operators to validate condition #20412
Conversation
Hi @igor-imaginemage. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @igor-imaginemage I see that |
Hi @sivaschenko, on Here is the same class on |
This is the code on 2.3 and 2.1 (working correctly): This is the code on 2.2 (not working properly with sku): It's the same method (getOperatorForValidate), but only on 2.2 it's using the |
Thanks for the detailed explanation @igor-imaginemage ! The only my concern is the actual reason why the These two commits: 8d62341 and d5b596a were introduced to fix the issue #11897 . In order to merge this pull request, it's required to ensure that the mentioned issue for multiple SKUs will not reappear. Would you be able to add proper handling of multiple SKUs condition to this pull request? |
@sivaschenko, I just tested with mutiple skus and it's working correctly. I tested a cart rule with multiple skus and also a product list widget, as reported in the issue #11897. Both are working correctly. This verifications on This previous inclusion of sku was forcing the |
@sivaschenko, this change that I've made in this PR is already on 2.1 and 2.3. By some reason is not available only on 2.2. Maybe who fixed that forgot to create the PR to 2.2. |
Hi @sivaschenko, thank you for the review. |
@igor-imaginemage thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
@magento-engcom-team give me test instance |
Hi @sivaschenko. Thank you for your request. I'm working on Magento instance for you |
Hi @sivaschenko, here is your new Magento instance. |
Hi @igor-imaginemage, thank you for your contribution! |
Description
@andersonalves faced an issue with our customers running Magento 2.2. When you create a product with a sku and you use the sku as a condition to a cart rule it will use the "{}"(contains) condition, and not equals "==". We debugged the code and we found an issue available only on Magento 2.2.
Fixed Issues
This PR will fix this issue on Magento 2.2. Magento 2.1 and 2.3 are working correctly, this fix was already applied to them.
Manual testing scenarios
conf123
.123
inside the skus, for exampleconf-red-123
andconf-blue-123
.123
.123
, it contains123
in the sku, but is not equal.