Is there alternative for mysql_affected_rows?

TomatoLover

Hi,

I want to check if the query ran correctly, so I used
mysql_affected_rows right after running query.
It does actually update data in the database when I
check it. However, mysql_affected_rows doesn't
return 1 even though it is definitely updating only
one row.

Here is my code.

if($p){ //if everything is OK
$query = "UPDATE table_login SET Password = PASSWORD('$p') WHERE Email = '" . $email."'";
$result = @($query);
if(mysql_affected_rows() == 1){
$message .= 'Your password has been changed.';
include('footer2.inc');
exit();
}else{
$message .= 'Your password could not be changed due to a system error.<br> We apologize for any inconvenience.';
}
mysql_close(); //close connection

Like I said, update query does work.
however,

if(mysql_affected_rows() ==1)

doesn't return 1, it skips inside of braces and
goes to else braces.

Do you know if there is something to use instead of
mysql_affected_rows() ?

LordShryku

too much...

if(mysql_query($query)) {
    $message .= 'Your password has been changed.';
    include('footer2.inc');
    exit();
}
else{
    $message .= 'Your password could not be changed due to a system error.<br> We apologize for any inconvenience.';
}
mysql_close();

TomatoLover

Thanks for your reply, LordShryku.
I tried what you said, but it seems not working.
It skips whole if-else block...

Is there any other idea?

laserlight

You didnt use mysql_affected_rows() correctly.

if ($p) { //if everything is OK
	$query = "UPDATE table_login SET Password = PASSWORD('$p') WHERE Email = '" . $email . "'";
	$result = @mysql_query($query);
	if (mysql_affected_rows($result) == 1) {
		$message .= 'Your password has been changed.';
		include('footer2.inc');
		exit();
	} else {
		$message .= 'Your password could not be changed due to a system error.<br> We apologize for any inconvenience.';
	}

TomatoLover

laserlight, I tried what you said, but didn't work.
I guess MySQL doesn't like me...

I've been using "mysql_affected_rows()" many times,
but never put $result in the parentheses, and it works fine.

In this code, I am using "mysql_affected_rows" twice.
The first one wasn't working yesterday, but today it is
working for some reason. But not the second one.
Not confuse php, I even used $query1 and $query2,
$result1 and $result2 for each query.
However, the second one is not working.
I get error message, which I am supposed to get
when the number of affected rows is not 1.
But the reality is only one row is affected.

What should I do?

ahundiak

You don't need to use $result as an argument to mysql_affected_rows().

I'd suggest echoing out the value of mysql_affected_rows(). You may have duplicate records. Or you may get a value of -1 indicating an error in your update statement.

TomatoLover

I had tried that before.
But for some reason, it was empty.
It didn't even return 0.
However, the table was updated when I checked it.
I have no idea what is going on because,
like I said, the first "mysql_affected_rows()" works, but
not the second.

Thanks for your reply, anyway.