-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
REST API multiselect attribute BC break between 2.1.8 and 2.3.0 #21364
Comments
Hi @mhujer. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @mhujer do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Now array of ids values is changed by comma seperated ids |
@ankit-sf So this change was intentional and is expected? |
no, i don't think it is intentional, it's just that it is not corrected in 2.3 but is corrected in previous versions 2.2 and 2.1 |
Hi @engcom-backlog-nazar. Thank you for working on this issue.
|
✅ Confirmed by @engcom-backlog-nazar Issue Available: @engcom-backlog-nazar, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @Valant13. Thank you for working on this issue.
|
Any progress on this? The last update was in May. It's October now. |
See post by @unicoder88 |
Is this tested with just only admin values applied? |
Hi @mhujer , Thanks for your reporting and collaboration. Steps to reproduce
Thanks. |
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/AC-773 |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Workaround
Implode the ids and send them as a string. But it makes your code incompatible with Magento 2.1.8 as it expects an array of ids.
I guess that it is some kind of BC break, because Magento 2.1.8 works in a completely opposite way - it requires an array of IDs and rejects string with a message:
{"message":"Error occurred during \"custom_attributes\" processing. Attribute \"equipment\" has invalid value. Invalid type for value: \"string\". Expected Type: \"string[]\"."}
Related issue: #6120
The text was updated successfully, but these errors were encountered: