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

Unselect multiselect on product edit view #8578

Closed
wants to merge 1 commit into from

Conversation

syno-dlr
Copy link

@syno-dlr syno-dlr commented Feb 16, 2017

Fix for issue #8577 : #8577

Preconditions
Install Magento 2
Create a product with a multiselect attribute AND selected values.
Save It

Steps to reproduce
Unselect ALL values
Save product again

Expected result
All values should be unselected.

Actual result
Previous values are still selected.

Fix for issue magento#8577 : magento#8577

Preconditions
Install Magento 2
Create a product with a multiselect attribute AND selected values.
Save It

Steps to reproduce
Unselect ALL values
Save product again

Expected result
All values should be unselected.

Actual result
Previous values are still selected.
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Feb 16, 2017

CLA assistant check
All committers have signed the CLA.

@vrann vrann added this to the February 2017 milestone Feb 16, 2017
@vrann vrann self-assigned this Feb 16, 2017
@vrann
Copy link
Contributor

vrann commented Feb 23, 2017

@damien-synolia cannot reproduce, it works as expected on the latest mainline. Which version your fix is for?

@vrann vrann added the bugfix label Feb 23, 2017
@syno-dlr
Copy link
Author

Hey vrann !

I tested on the community edition 2.1.4 with sample data (from Magento Website).

Damien.

@maksek maksek modified the milestones: February 2017, March 2017 Mar 1, 2017
@okorshenko okorshenko added this to the March 2017 milestone Mar 1, 2017
@vrann
Copy link
Contributor

vrann commented Mar 2, 2017

@damien-synolia can you please verify do you observe the issue on the latest develop branch? Please attach screenshots if you do.

Just checked with the sample data installed and the following result is consistent with the expected.
screen shot 2017-03-02 at 1 54 10 pm

@vrann
Copy link
Contributor

vrann commented Mar 13, 2017

@damien-synolia closing this PR, please feel free to open a new one if issue is reproducible on the mainline

@vrann
Copy link
Contributor

vrann commented Mar 13, 2017

@damien-synolia there is related issue #7687, it was fixed in the mainline and being backported to the 2.1

@redelschaap
Copy link
Contributor

This issue is still present in 2.1.5 and not fixed by the solution given in the commit referenced in #7687.

magento-devops-reposync-svc pushed a commit that referenced this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants