isotope / core

Core repository of Isotope eCommerce, an eCommerce extension for Contao Open Source CMS
https://isotopeecommerce.org
135 stars 107 forks source link

Generate variants wizard thorws array_diff error #369

Closed ghost closed 11 years ago

ghost commented 11 years ago

Hi,

if I want to use the generate variants wizard i get the following error:

@Warning: array_diff() [function.array-diff]: Argumenthttp://www.contao-forge.org/issues/ is not an array in D:\xampp\htdocs\xxx\system\modules\isotope\dca\tl_iso_products.php on line 1116

http://www.contao-forge.org/issues/

http://www.contao-forge.org/issues/ D:\xampp\htdocs\xxx\system\modules\isotope\dca\tl_iso_products.php(1116): array_diff('a:4:{s:9:"publi...', Array) http://www.contao-forge.org/issues/ D:\xampp\htdocs\xxx\system\modules\backend\Backend.php(230): tl_iso_products->generateVariants(Object(DC_ProductData), 'tl_iso_products', Array) http://www.contao-forge.org/issues/ D:\xampp\htdocs\xxx\contao\main.php(101): Backend->getBackendModule('iso_products') http://www.contao-forge.org/issues/ D:\xampp\htdocs\xxx\contao\main.php(286): Main->run() http://www.contao-forge.org/issues/ {main} @

I think the error is caused by the fact that the $objProduct->variant_attributes is a serialized string and not an array that can be diffed. My solution suggest is to deserialize the $objProduct->variant_attributes string. I've tested this with the current RC1 Version from the extension repository.

--- Originally created by cboelter on 2012-03-01 17:11:59 (ID 2945)

ghost commented 11 years ago

I have the same issue

--- Originally created by freeyland on 2012-03-13 15:47:55

aschempp commented 11 years ago

Already fixed in https://github.com/isotope/core/commit/8d0f7ce44f8194e31726bc35136db44af01f36b6

--- Originally created by andreas.schempp on 2012-03-18 09:13:38