[RESOLVED] quick one

rbblue8

hey guys...

i'm trying to figure out if this will work.

SELECT * from 3876_match where status NOT LIKE Closed and user1='admin' OR user2='admin'

thats a mysql query.

basically i'm searching to see if the status is closed... and to see if the username is in user1 or user 2 field.

do i have to make two different querys for that or will this do?

thanks!

laserlight

Looks like it might work... but what is the table schema like, and what exactly do you want to retrieve?

rbblue8

Hey,

it's basically like a if statment.. that if a row is found with the status of NOT closed and the user name is found in either user1 or user2 field it will return a mysql_num_row of 1 or more. and if that happanes it prevents the rest of the script from going through.

but it does not see to be working and the mysql_error() does not return anything....

here is the code.

/lets get the datbase information!
require('/homepages/13/d153872232/htdocs/nightlance.com/match/main_mysql.php'); // main databse
$sql="SELECT db_name, status from Events where name='$event'";
$q=mysql_query($sql, $db) or $error=("Error when trying to get the databse name from the main datbase!<br>".mysql_error());
$r=mysql_fetch_assoc($q);

if(isset($error))
	{
	echo '<center id="error">'.$error.'</center>';
	}

$database=$r['db_name'];

$event_status=$r['status'];


//lets see if they are in any matches
$search_string=("SELECT * from ".$database."_match where status NOT LIKE Closed and user1='$username' OR user2='$username'"); //here is where the error is

$search_query=mysql_query($search_string, $events) or $error=('Error when trying to see if you are currently in a match <br />'.mysql_error());
$search_result=mysql_num_rows($search_query);
if($search_result !== '0')$stop='Yes';
echo $search_string;
if(isset($error))
	{
	echo '<center id="error">'.$error.'</center>';
	}

ahundiak

http://dev.mysql.com/doc/refman/5.0/en/operator-precedence.html

Should probably use parens just to make sure or the preecedence
SELECT * FROM name WHERE (user1 = 'admin' OR user2 = 'admin') AND status != 'Closed';

Some more unsolicited advice

I would avoid using table names starting with numbers

http://dev.mysql.com/doc/refman/5.0/en/legal-names.html

It is recommended that you do not use names of the form Me or MeN, where M and N are integers. For example, avoid using 1e or 2e2 as identifiers, because an expression such as 1e+3 is ambiguous. Depending on context, it might be interpreted as the expression 1e + 3 or as the number 1e+3.

Consider using numerical codes for status fields. 1 = open, 2 = closed etc.
Use of repeatative fields (user 1, user2) probably indicates that the schema needs to be redefined.

laserlight

Why do you have two columns for username?

My guess is that you need to write "'Closed'", not "Closed".

ahundiak

And why are you using
$search_query=mysql_query($search_string, $events)
instead of
$search_query=mysql_query($search_string, $db)

rbblue8

why does my .mysql_error not work??

rbblue8

ok..

i belive i found the problem.

it's a page generation issue.

basically the first time you visit the page it works and gets the correct information.

however when you hit the submit buttion it fails to get the correct information! It's completly skips whats posted above....

rbblue8

fixed it.. 🙂 was an issue when my form that was posting to that script 🙂