Proper way?

[3raser]

Should this be the proper way of adding the getUsername() text in the query below:

 

mysql_query("UPDATE posts SET content = '$content', lastedit = ". getUsername() .":NOW() WHERE id = '$pid'") or die(mysql_error());

[MasterACE14]

wouldn't really say there's a 'proper' way. More a matter of preference. What you have there is fine.

[PFMaBiSmAd]

Assuming you WANT to combine the string that your php function getUsername() returns with a ':' and the string that the mysql function NOW() returns, you would need to use -

 

$query = "UPDATE posts SET content = '$content', lastedit = CONCAT('". getUsername() .":',NOW()) WHERE id = '$pid'";

 

You should also form your query statements in a php string variable (makes troubleshooting and error reporting easier.) Echo the $query variable so that you can see what portion of it php contributed and what the resulting sql syntax is.

[3raser]

Assuming you WANT to combine the string that your php function getUsername() returns with a ':' and the string that the mysql function NOW() returns, you would need to use -

 

$query = "UPDATE posts SET content = '$content', lastedit = CONCAT('". getUsername() .":',NOW()) WHERE id = '$pid'";

 

Thank you. I really appreciate the fast response time.