what is wrong with this line of code

[hungryfrank]

hi

why this line don't work

tep_db_query("update " . TABLE_PRODUCTS . " set products_ordered = products_ordered + " .  $QuantityPurchased . " and 	products_quantity = products_quantity- " .  $QuantityPurchased . " where products_id = '" . tep_get_prid($SKU) . "'");	
	}

in the next lines it enters the correct info into a different table

[BrockleyJohn]

Try separating the minus from products_quantity with a space

 

edit to add:

 

It would be safer to cast the variable to integers too, eg.

. (int) $QuantityPurchased .

[tgely]

Tip: Stock Keeping Unit mainly associated with SKU number or products_model and rarely with products_id.

[hungryfrank]

thanks guys .

@@action Hero. i am trying to enter my ebay orders into osc orders so i use the sku to know what attributes were used in the ebay listing.everything seems to work but i have two problems this is one and the other i will post shortly.

 

but i still have a problem 

if i use 

tep_db_query("update " . TABLE_PRODUCTS . " set products_ordered = products_ordered + " . sprintf('%d', $QuantityPurchased) . " where products_id = '" . $e_id . "'");
tep_db_query("update " . TABLE_PRODUCTS . " set products_quantity = products_quantity - " . sprintf('%d', $QuantityPurchased) . " where products_id = '" . $e_id . "'");

it works  but if i use one call  then products_ordered turns to zero and product s_quantity dosen't change

tep_db_query("update " . TABLE_PRODUCTS . " set products_ordered = products_ordered + " . sprintf('%d', $QuantityPurchased). " and products_quantity = products_quantity - " . sprintf('%d', $QuantityPurchased). " where products_id = '" . $e_id . "'"); 

[Jack_mcs]

The format of the update is set a=value, b=value, not set a=value and b=value.  Also, there's no need for the spintf calls. just cast the values to int as mentioned previously: (int)$QuantityPurchased

[hungryfrank]

thx Jack