vuestorefront / vue-storefront-1

The open-source frontend for any eCommerce. Built with a PWA and headless approach, using a modern JS stack. We have custom integrations with Magento, commercetools, Shopware and Shopify and total coverage is just a matter of time. The API approach also allows you to merge VSF with any third-party tool like CMS, payment gateways or analytics. Newest updates: https://blog.vuestorefront.io. Always Open Source, MIT license.
https://www.vuestorefront.io
MIT License
19 stars 13 forks source link

Displays broken/placeholder in image gallery #393

Open simonmaass opened 4 years ago

simonmaass commented 4 years ago

Current behavior

When selecting gallery.mergeConfigurableChildren = false the images are created by attributeImages() in core\modules\catalog\helpers\index.ts

This doesnt filter out "no_selection" images but just adds all images definied in gallery.imageAttributes

Expected behavior

before pushing new image in gallery we shoudl check if image exists: if (product[attribute] && product[attribute] !== 'no_selection')

Steps to reproduce the issue

Version of Vue Storefront

Can you handle fixing this bug by yourself?

Which Release Cycle state this refers to? Info for developer. (doesn't apply to Next)

Pick one option.

Environment details

Additional information

i will create PR

simonmaass commented 4 years ago

PR created