VirtueMart Forum

I found a bug in classes/ps_product.php

Line 412 through 414 contains a mysql query for copying product attributes.

Faulty code:

$q = "INSERT INTO #__{vm}_product_attribute
                  SELECT '$new_product_id', attribute_name, attribute_value
                  FROM #__{vm}_product_attribute WHERE product_id='$child_id'";
$db->query( $q );


Correct code:

$q = "INSERT INTO #__{vm}_product_attribute
                  SELECT NULL,'$new_product_id', attribute_name, attribute_value
                  FROM #__{vm}_product_attribute WHERE product_id='$child_id'";
$db->query( $q );

(The change is SELECT NULL,......)

Or you could specify which attributes to add. You got a illegal column count with the original code since it doesn't specify the attribute_id column (which is auto_incrementing)

Thank you,
will be fixed in VM 1.1.2.

ciao, Sören

I didn't see this in the changelog? Any news?

Sorry must have forgot to add it to the changelog, but correction has been made to the file.