I would say something like the following would work better: if($FUNC=="USERPOST" || $FUNC=="MODU") { $sql .= "UPDATE newdacs"; $sql .= "SET emailfwd='$emailfwd',"; $sql .= "mformat='$mformat'," $sql .= "filter_code='$filter_code'"; if($FUNC=='USERPOST') $sql .= ", unote='$unote'"; $sql .= "WHERE user_id='$user_id'"; $result = mysql_query($sql) or die('mysql error #'.mysql_errno.': 'mysql_error()); } Try that ... also look into www.php.net under string manipulation for other options. Regards, Matthew Moldvan System Administrator Trilogy International, Inc. -----Original Message----- From: Susan Ator [mailto:SAtor@npr.org] Sent: Tuesday, June 10, 2003 11:18 AM To: 'php-db@lists.php.net' Subject: Is this query even possible? > This is what I am trying to do: > > if ($FUNC==("USERPOST") || $FUNC==("MODU")) { > $sql = "UPDATE newdacs > SET emailfwd='$emailfwd', > mformat='$mformat', > filter_code='$filter_code' > if ($FUNC=='USERPOST') { > ,unote='$unote' > } > WHERE user_id='$user_id'"; > $set_newdacs_result = mysql_query($sql) or print > mysql_error(); > } > > There are several more queries like this which is why I want to avoid > duplicating the code since there are only 1 or 2 differences between them > depending on function call. > > This is the error I am getting: > > You have an error in your SQL syntax near 'if (USERPOST=='USERPOST') > { ,unote='' } WHERE user_id='2'' at line 5 > > Is what I am attempting possible or am I spinning my wheels? > > susan -- PHP Database Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php -- PHP Database Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php