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

Multiple features with same value are causing not described errors #10757

Closed
kpodemski opened this Issue Sep 28, 2018 · 2 comments

Comments

@kpodemski
Contributor

kpodemski commented Sep 28, 2018

Describe the bug
Title says it all.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Catalog > Product -> Edit'
  2. Add feature, Color: White, and below again, Color: white
  3. Click the "Save" button
  4. See error, it says that settings cannot be saved but there's literally no information at all about what went wrong, of course it's obvious mistake to have 2 features with same value but if someone have 20+ features it's easy to overlook it

Screenshots
If applicable, add screenshots or screenrecords to help explain your problem.

Additionnal information
PrestaShop version: 1.7.5.0
PHP version: N/A

@khouloudbelguith

This comment has been minimized.

Contributor

khouloudbelguith commented Oct 1, 2018

Hi @kpodemski,

Thanks for your report.
I manage to reproduce the issue with the branch 1.7.5.x.
image
When we refresh the page, only one feature will be displayed.

In the previous version 1.7.4.2, when we have multiple features, settings will be saved successfully with green alert but when we refresh the page, only one feature will be displayed.
We will see how to fix it.

Best regards, Khouloud

@marionf marionf added Minor To Do and removed TBR labels Oct 1, 2018

@marionf marionf added this to To do in PrestaShop 1.7.5 via automation Oct 1, 2018

@marionf marionf added this to the 1.7.5.0 milestone Oct 1, 2018

@marionf marionf removed their assignment Oct 1, 2018

@PierreRambaud PierreRambaud self-assigned this Oct 9, 2018

@eternoendless eternoendless moved this from To do to To be reviewed in PrestaShop 1.7.5 Oct 18, 2018

@eternoendless eternoendless moved this from To be reviewed to To be tested in PrestaShop 1.7.5 Oct 18, 2018

@marionf marionf removed the To Do label Oct 18, 2018

@marionf marionf moved this from To be tested to In progress in PrestaShop 1.7.5 Oct 18, 2018

@marionf marionf moved this from In progress to To be reviewed in PrestaShop 1.7.5 Nov 7, 2018

@matks

This comment has been minimized.

Contributor

matks commented Nov 9, 2018

Fixed by #10956

@matks matks moved this from To be reviewed to To be tested in PrestaShop 1.7.5 Nov 9, 2018

@matks matks moved this from To be tested to To be merged in PrestaShop 1.7.5 Nov 9, 2018

@matks matks moved this from To be merged to Done in PrestaShop 1.7.5 Nov 9, 2018

@matks matks closed this Nov 9, 2018

@marionf marionf added the Fixed label Nov 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment