Skip to content

Fix #7333 Unable to set negative custom option fixed price in admin view. #15179

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

Closed
wants to merge 1 commit into from
Closed

Fix #7333 Unable to set negative custom option fixed price in admin view. #15179

wants to merge 1 commit into from

Conversation

dverkade
Copy link
Member

@dverkade dverkade commented May 12, 2018

Fixes the ability to set a negative custom option price in the backend.

Description

In Magento 1 it was possible to set a negative price change to a custom option. This PR fixes issue #7333 and makes it possible to set a negative price in the backend of Magento. This PR fixes the backend so that the negative price change can actually be entered in the Magento admin. There is a separate PR for the frontend changes which is PR #14975

There also seem to be two other PR's for this issue which are stale and are not moving along:
#13393
#14342

Fixed Issues

  1. Unable to set negative custom option fixed price in admin view. #7333: Unable to set negative custom option fixed price in admin view.

Manual testing scenarios

  1. Create a new simple product in the backend and try to create new custom options where the price change has a negative value (for instance, "-10").

@dverkade
Copy link
Member Author

Let me check the failing tests.

@VladimirZaets
Copy link
Contributor

Hi, @dverkade, thank you for collaboration. If you need some help I will glad to help you.

@dverkade dverkade closed this May 14, 2018
@dverkade dverkade deleted the 2.2-Fix-negative-value-for-custom-option branch May 14, 2018 15:37
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.

3 participants