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

Can not update configurable products, price of simples suddenly become 0,00 #5973

Closed
tempo-ruud opened this issue Aug 3, 2016 · 72 comments
Closed
Labels
bug report Component: Catalog Fixed in 2.2.x The issue has been fixed in 2.2 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release

Comments

@tempo-ruud
Copy link

tempo-ruud commented Aug 3, 2016

Preconditions

  1. Magento 2.1
  2. Nginx
  3. PHP 7.0.9
  4. Memcached
  5. Redis Server

Steps to reproduce

  1. Add a new configurable product
  2. Create the simple products while creating the configurable product
  3. Save the product
  4. Open the configurable product again
  5. Upload images or change any other data
  6. Save the product

Expected result

  1. Configurable product get updated
  2. Simple products related with the configurable product should stay the same

Actual result

  1. The simple products suddenly have the price of 0
@tempo-ruud
Copy link
Author

I even disabled all installed modules and that did not help

@tempo-ruud tempo-ruud changed the title Magento 2.1 New installation - After editing configurable product the related simples get 0.00 as price Can not update configurable products, price of simples suddenly become 0,00 Aug 4, 2016
@heldchen
Copy link
Contributor

heldchen commented Aug 8, 2016

same here. we use an adminlogger extension, which logs all actions. when the configurable parent is edited, children are updated after saving. the update changes price (sets it to 0.00) and the configurable_attribute fields of all associated simple products. this is unbelievable devastating as now the products are orderable for free.

@duhon duhon self-assigned this Aug 8, 2016
@tempo-ruud
Copy link
Author

@duhon can you give me any update on how much time it will take before this gets patched. We have a huge issue with this as we want to create a lot of configurable products.

There are now 2 big issues on the configurables that cost us way more time than before. The second one is that we can not select existing simple products on a newly created (configurable) product.

In my opinion these bugs are to huge for a master branch.

@duhon
Copy link
Contributor

duhon commented Aug 8, 2016

@tempo-ruud I'm trying to reproduce the problem, but I have everything working. You can help me. Have you reproduced the problem without Memcached and Redis?

@tempo-ruud
Copy link
Author

@duhon Nope, both are enabled right now, but if you wish, I can give you access to my webserver and Magento admin, so we can figure it out together.

@heldchen
Copy link
Contributor

heldchen commented Aug 8, 2016

no memcached here, no redis either. magento 2.1.0 in developer mode on local dev machines. we're using 2 frontend store views (DE + FR) and seeing the error when editing the configurable product in the default storeview.

@tempo-ruud
Copy link
Author

I only have one storeview enabled

@duhon
Copy link
Contributor

duhon commented Aug 8, 2016

I still cannot reproduce this issue.
Please also describe all steps you have done in a more detailed way and if you can attach the screenshot or some logs it may help as well.
If you are using EE version, contact support.

@duhon
Copy link
Contributor

duhon commented Aug 8, 2016

@heldchen are you using a clean installation without any additional settings?

@heldchen
Copy link
Contributor

heldchen commented Aug 8, 2016

here's a screenshot of one of the examples logged by the adminloger extension (bottom to top):

image

the user was editing the product parent product 31604. upon saving, its stock item was saved (15832), then the product (31640), then the configurable attribute, and then all the child items stock item and product. the stock item kept their values, but the product update set the price to 0.0000. see for example child product 31601.

all products were imported/created in 2.0.2 or 2.0.3 and and thereafter upgraded in all the following releases up until 2.1.0.

as for the question regarding "clean installation": we started clean except for the products-related data which was imported quite early in the process through csv imports.

I can try to create a small db dump containing just some of the broken products if that helps?

@heldchen
Copy link
Contributor

heldchen commented Aug 8, 2016

I played around a bit more with it. it only seems to happen the first time a parent product is saved after upgrading to 2.1.0. later saves of the parent product do not affect the price of children anymore. @tempo-ruud is that the same on your system?

@tempo-ruud
Copy link
Author

@heldchen Nope, for me it is a clean setup of Magento 2.1. I only installed a theme. After the installation of the theme I imported a csv with only simple products. This did not seem to work as there where a lot of configurable products in the shop and they shop admin could not assign the simple products to a newly created configurable product.
So the admin is now creating the configurable products manually. But after he created them and edit them again he has a problem with the prices, images and seo values as they become empty.

@heldchen
Copy link
Contributor

heldchen commented Aug 9, 2016

yeah, that sounds like the exact workflow as we did after initial product import - the customer only had simple and grouped products in the old shop but wanted some of them as configurable products in the new shop. so the show owner manually created the configurable parent products, whereas the simple child products were imported through csv.

@tempo-ruud
Copy link
Author

@duhon did this help you reproduce the error.
By the way, if you wanna know which server config I am using. Here is a link to the repo:
https://github.com/magenx/Magento-Automated-Server-Configuration-from-MagenX

@tempo-ruud
Copy link
Author

@heldchen did you already fix this. I also posted the question on stack overflow as I get really crazy of this. What ever I try I cannot fix it.

@oserediuk oserediuk assigned oserediuk and unassigned duhon Aug 10, 2016
@paales
Copy link
Contributor

paales commented Aug 10, 2016

Experienced this as well after having products imported.

@tempo-ruud
Copy link
Author

Thanks @paales for confirming

@heldchen
Copy link
Contributor

@tempo-ruud for our system, the values are 'only' resetted once - after correcting the values, further saves of the parent product are not affecting the childrens anymore. are you making the same experience?

@tempo-ruud
Copy link
Author

tempo-ruud commented Aug 10, 2016

@heldchen no we have the problem each time we save the product.

@oserediuk
Copy link
Contributor

Hello @tempo-ruud ! Did your modify imported simple product after import? How you assign simple product to configurable? Which attribute do you use for this?

@oserediuk
Copy link
Contributor

Hello guys! Do you all use nginx?

@tempo-ruud
Copy link
Author

tempo-ruud commented Aug 11, 2016

@oserediuk Yes I use nginx, I use this server configuration: https://github.com/magenx/Magento-Automated-Server-Configuration-from-MagenX

@heldchen
Copy link
Contributor

@oserediuk we only imported simple products and manually associated them to the parents. apache2 / php-fpm / 7.0.8

@oserediuk
Copy link
Contributor

@heldchen which attribute type do you use for configurable options?

@tempo-ruud
Copy link
Author

@oserediuk We only imported simple products with the csv imported that is default in Magento 2.
After that we wanted to assign the simples to configurables, but this was not possible in Magento 2.1 so we created manually the configurable products with newly created simples.

@JamboMedia
Copy link

Excacly the same for us. We added a dutch language package.

On admin language en_US the problem isn't there.

@heldchen
Copy link
Contributor

heldchen commented Sep 8, 2016

yes, we use de_CH and fr_CH. when one of those is used for a backend user, the data is corrupted - therefore en_US for now in the backend.

@tempo-ruud
Copy link
Author

@heldchen Offtopic: I was wondering, for which company do you work in swiss?

@heldchen
Copy link
Contributor

heldchen commented Sep 8, 2016

@tempo-ruud sent you an email, I don't want to use github for "advertising" :)

@AVoskoboinikov
Copy link
Contributor

Hi guys!

I've managed to reproduce this issue, so I created internal ticket to fix this (#MAGETWO-58209). This issue will be closed after fix.

Thanks to everybody for reporting and investigations!

@heldchen
Copy link
Contributor

heldchen commented Sep 9, 2016

awesome, thanks for your time!

@AVoskoboinikov AVoskoboinikov added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Sep 9, 2016
@AVoskoboinikov AVoskoboinikov removed their assignment Sep 12, 2016
@Pagayo
Copy link

Pagayo commented Oct 6, 2016

This simple fix saved me today 👍
Hope it get fixed soon, not every Dutch merchant reads English that well.

@tempo-ruud
Copy link
Author

@Pagayo glad that I could help you with this.

@tdgroot
Copy link
Member

tdgroot commented Nov 2, 2016

Switching the Interface Locale back to en_US worked for me, this is a great temporary fix!

@ghost
Copy link

ghost commented Nov 2, 2016

Thanks for this, also a dutch user :-)

@jurgenbrouwer
Copy link

Can you provide any update on this issue @AZVO ?
We ran into this as well using another language in the backend. For now we can use the English language in the backend since we are still developing.

@SolsWebdesign
Copy link
Contributor

Yes, thanks (another Dutch user)...

@hansvanl
Copy link

hansvanl commented Jan 4, 2017

Would love an update also.

@ghost
Copy link

ghost commented Feb 4, 2017

I have the same problem with the Dutch language pack.

@mike-happy
Copy link

Is this issue fixed in Magento 2.1.4?

@veloraven
Copy link
Contributor

This issue was fixed in develop branch in scope of internal ticket MAGETWO-57279
Internal ticket for ver. 2.1 - MAGETWO-58518

@magento-engcom-team magento-engcom-team added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development bug report Component: Catalog Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed labels Sep 11, 2017
@magento-engcom-team
Copy link
Contributor

@tempo-ruud, thank you for your report.
The issue is already fixed in 2.2.0

@magento-engcom-team magento-engcom-team added 2.1.x Fixed in 2.2.x The issue has been fixed in 2.2 release line Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release labels Sep 20, 2017
mmansoor-magento pushed a commit that referenced this issue Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Component: Catalog Fixed in 2.2.x The issue has been fixed in 2.2 release line Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release
Projects
None yet
Development

No branches or pull requests