Keep in mind the code I sent is untested ... and I realized some errors
when I pasted it into a PHP page. Corrections are below.
As for the line breaks, I've used SQL formatted that way before and it
hasn't cause me any problems.
Regards,
Matt.
Corrections: $sql .= "mformat='$mformat'," should have a trailing ;
$result = mysql_query($sql) or die('mysql error
#'.mysql_errno.': 'mysql_error()); should have a . before mysql_error());
-----Original Message-----
From: Hutchins, Richard [mailto:Richard.Hutchins@Getingeusa.com]
Sent: Tuesday, June 10, 2003 11:42 AM
To: 'php-db@lists.php.net'
Subject: RE: Is this query even possible?
Yup, Matthew's right. I definitely forgot the other $sql. = that needs to go
in front of the WHERE clause.
Question for Matthew though:
Is this:
> $sql .= "UPDATE newdacs";
> $sql .= "SET emailfwd='$emailfwd',";
> $sql .= "mformat='$mformat',"
> $sql .= "filter_code='$filter_code'";
Pretty much the same as this:
> > $sql = "UPDATE newdacs
> > SET emailfwd='$emailfwd',
> > mformat='$mformat',
> > filter_code='$filter_code'";
I mean, the line breaks in the second example won't cause the query to bomb,
will they? I agree that using the $sql .= is cleaner, but the other way
isn't wrong, is it?
Rich
> -----Original Message-----
> From: Matthew Moldvan [mailto:mmoldvan@trilogyintl.com]
> Sent: Tuesday, June 10, 2003 11:33 AM
> To: 'Susan Ator'; 'php-db@lists.php.net'
> Subject: RE: Is this query even possible?
>
>
> 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
>
--
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
