how to get a "record doesn't exist" error on UPDATE ?

steamPunk

I've written this bit of code that takes a list of order numbers from a form field and updates all the orders with those numbers in the table

		$tmp =  explode("\n",$_POST['COMM_NUMBERS']);
		$comms = array();

	foreach($tmp as $v){	
		$er = '';		 
		if($v!=""){
			$sql = "UPDATE comm_en SET PAYE='1',comm_temp='' WHERE ID_Commande_Site='".$v."' AND PAYE='0' LIMIT 1";
			if($result = mysql_query($sql)){
				$er =  "Succès";
			}else{
				$er = mysql_error();
			}
			op($sql);
			$comms[] = array("num"=>$v,"err"=>$er);
		}			
	}

what i'd like it to do is if it doesn't find an order with the order number in question it would return an error
but i've tried it with non-existant order numbers and it always returns "Success"

does anyone know where i'm going wrong ?

thanks

__berfuzz

I'm not entirely sure of what the code is suppose to do...

$result = mysql_query($sql)

sets $result to whatever mysql_query returns. Have a look in the manual to see what it returns, [man]mysql_query[/man]. Try using [man]mysql_num_rows[/man] to see if theres a hit.

NogDog

[man]mysql_affected_rows/man for the number of rows affected by an update.

__berfuzz

NogDog;10910336 wrote:
[man]mysql_affected_rows/man for the number of rows affected by an update.

Quite right, mysql_num_rows won't work! My bad... :o

steamPunk

Thanks for your replies NogDog and überfuzz

i'll give that mysql_affected_rows() a try

thanks again 🙂

Weedpacket

Just to clarify; an update statement that doesn't match any records does not count as an error - in the same way that a select statement returning an empty result set when there are no matching records doesn't count as an error.