URGENT HELP NEEDED PHP MYSQL CODE

simpson1991

if (isset($_POST['field1'])) { 

$Pid = mysql_real_escape_string($_POST['thisID']); 
$field1 = mysql_real_escape_string($_POST['one']); 
$field2 = mysql_real_escape_string($_POST['two']); 
$field3 = mysql_real_escape_string($_POST['three']); 
$field4 = mysql_real_escape_string($_POST['four']); 

[B]$newvariable = mysql_query("SELECT id from table2 WHERE columb2 = '" . $field1 . "'"); [/B]

$sql = mysql_query("UPDATE table1 SET column1='$field2', column2='$field3', column3='$field4', column4='$field1', [B]column5='$newvariable'[/B] WHERE id='$Pid'");

All of the code worked until i needed the id from table 2 to be inserted into table 1 aslong as the variable matched column 2 in table 2. I hope this makes sense any help would be much appreciated

simpson1991

please ignore the tags in my code i was trying to make them bold to show the code that is not working but any help would be great with this.

bradgrafelman

$newvariable is either going to contain a MySQL resource or boolean FALSE (if an error occurred), so trying to use it inside a string doesn't make any sense. You'll want to check if you got the former and, if so, retrieve the value of the `id' column (e.g. by using [man]mysql_result/man).

Also note that the entire [man]mysql[/man] extension is outdated and deprecated; [man]MySQLi[/man] or [man]PDO[/man] should be used instead.

simpson1991

i have tryed laying it out like this:

$sql = "SELECT id from table2 WHERE column2 = '$field1'
$pcategoryid = mysql_query($sql)

if that is what you mean but i still do not receive the value needed. I do not receive any errors from the code but the field in the database just changes to "0".

basically $field1 variable is from the post form,
$field1 needs comparing with where clause with the column2 in table2,
and then the id found in table 2 needs placing into table 1.

simpson1991

can no one help 🙁 ???

bradgrafelman

Sure, I can. In fact, I already did. You still have the exact same issue as the one I addressed in my post above.

$pategoryid in your code above is still going to be either a MySQL resource or just boolean FALSE (if an error occurs). You still haven't added code to a) check which of those values you did get back and, b) assuming it was a MySQL resource, use that resource to actually retrieve the value from the first row (if there was one - something else you should be checking for), such as by using [man]mysql_result/man or one of the mysql_fetch_*() functions.

simpson1991

bradgrafelman;11015173 wrote:
Sure, I can. In fact, I already did. You still have the exact same issue as the one I addressed in my post above.

$pategoryid in your code above is still going to be either a MySQL resource or just boolean FALSE (if an error occurs). You still haven't added code to a) check which of those values you did get back and, b) assuming it was a MySQL resource, use that resource to actually retrieve the value from the first row (if there was one - something else you should be checking for), such as by using [man]mysql_result/man or one of the mysql_fetch_*() functions.

Sorry did not fully realize what you meant at first there :/ but thank you i got it working now appreciate the help 🙂