MySQL PHP Problem

Liongold

I have a problem with this code:

$get = "SELECT * FROM ".TBL." WHERE test = '1' && (test = '".$fieldx['ShortVersion']."' || test2 = '".$fieldx['ShortVersion']."' || test3 = '".$fieldx['ShortVersion']."' || test4 = '".$fieldx['ShortVersion']."')"; 
                $getprocess = mysql_query($get) or die(mysql_error());
                echo "<br>Awarding ...";
                //Debug ready till here
                while($getup = mysql_fetch_array($getprocess)) {
                    // This does not run
                };

When I run this code (as part of a function), the while lopp does not run and when I add

or die("error".mysql_error())

to the mysql_fetch_array, I get no error. What can the problem be? Please help because I don't know what's wrong.

Liongold

Sorry when it runs I get only "error" with no mysql related errors.

Weedpacket

Have you verified that you have a working database connection in the first place? (Note, too, that the mysql_* functions are deprecated and should not be used.)

bradgrafelman

Also note that a query that executes successfully but does not return any rows in the result set is not considered an error; perhaps this is the case you're running into? Have you tried printing the SQL query string out so that you can manually execute it to see if it works?

cretaceous

also your query logic appears to be borked:

$get = "SELECT * FROM ".TBL." WHERE test = '1' && (test = '".$fieldx['ShortVersion']."' || test2 = '".$fieldx['ShortVersion']."' || test3 = '".$fieldx['ShortVersion']."' || test4 = '".$fieldx['ShortVersion']."')";

Specifically :: "WHERE test = '1' && (test = '".$fieldx['ShortVersion']."' || ... "

so test must be '1' and there's no point having the second entry .. or you have a typing mistake: 'test1' perhaps?
(Tip: use column names that are unambiguous)

Derokorian

Also, for simplicity, youcould change you're query to roughly $fieldx['shortversion'] IN (test1, test2, test3, test4)

Also note, you are checking for test when I think you want test1.

Liongold

Tried it and it works. Thanks.

Liongold

The test double was a mistake. I meant test1 in the second case.