RE: Is this query even possible?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [PHP Home]     [PHP Users]     [Postgresql Discussion]     [Kernel Newbies]     [Postgresql]     [Yosemite News]

  Powered by Linux