Weird function problem

[stevenrhiggins]

I have a page with several functions. Within two of these functions data is updated or deleted from mysql db.

To confirm that the data was updated or deleted, I use a

if ($result===TRUE) {
print"data was updated (deleted)";
}
else {
print"data was not updated (deleted";
}

When the php page is first called the

if ($result===True......

comments always appear on the page, even when the function has not been called.

What gives?

[vsego]

Please post whole script.

Btw, why triple "="?

[stevenrhiggins]

The whole script is rather long but here is the relevant part

function doUpdate($id,$storename) {
$connection = mysql_connect("","","")
or die ("Unable to connect to MySQL server.");
$db = mysql_select_db("mpa", $connection) or die ("Unable to select
database.");
$update=mysql_query("UPDATE storelocations SET storename='$storename', address1='$address1', address2='$address2', city='$city', state='$state', country='$country', phone='$phone' WHERE retailerid='$retailerid'");

if ($update==TRUE) {
print"<table width=\"740\"><tr><td class=\"text\" align=\"center\">The store " .$storename. "was updated.";
print"&nbsp;&nbsp; Update another? <a href=\"storelist.php?cmd=update_stores&id=$retailerid\">Yes</a>";
print"&nbsp;&nbsp;<a href=\"storelist.php\">No</a></td></tr>";
}
////////////////////////

The part that keeps showing up on the page is

"The store " .$storename. "was updated."

[vsego]

I guess this if() block is not part of the function. You must be setting $update somewhere. Send me a script to [e-mail] if you want me to look through it.

Btw, instead of "if($x==TRUE)..." you can use just "if ($x)..."

[stevenrhiggins]

I checked all that. The if () block is part of the function and the function is not called anywhere else in the script. I'll send it to you tomorrow...Saturday. Thanks for your interest.

[vsego]

Your if()s in the end look like this:

if ($cmd=verify_delete) {
verifyDeleteStore($id);
}

"=" is ASSIGNMENT of value! Comparisons are "==". :-)

if ($cmd == "verify_delete") { // quots are not necessary, but they make the code cleaner
verifyDeleteStore($id);
}

Btw, it would be enough to put connection code ($connection=...; $db=...) to the begining of script instead of holding a copy in each function.

Good luck!