what could be wrong?

[sirTemplar]

what could be wrong with this code?

$result = mysql_query ("SELECT EVENTYEAR, EVENTTEXT,
YEAR(EVENTDATE) AS EVENTDATE FROM thisday
WHERE substring(EVENTDATE,6) = substring(curdate(),6)
ORDER BY EVENTYEAR ASC,
",$conn);

if ($row = mysql_fetch_array($result)) {

do {

echo "<table bgcolor=#FFFEEF border=0 cellpadding=1 cellspacing=0 style=border-collapse: collapse bordercolor=#111111 width=100%>";
echo "<font face=Verdana size=1 color=#800000><li>fra {$row['EVENTYEAR']} </b> {$row['EVENTTEXT']}
</font></li></table>";
}
while($row = mysql_fetch_array($result));

} else {print "no event";}

?>
</td>
</tr>

it is giving me this error:

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /var/www/html/websites/folder/folder/myfile.php on line 42
no event

line 42 is

if ($row = mysql_fetch_array($result)) {

but when i run the sql query via phpMyAdmin, i do get the correct results! any idea, please

[bad76]

Hi,
try with mysql_error() and if you need help too post also the error

see you

ps: for more info look this...

http://www.phpbuilder.com/board/showthread.php?s=&threadid=10242657

[Norman_Graham]

Your mysql_fetch_array() doesn't produce a valid result (like the error message states) because your query doesn't work. One possible error could be the trailing comma after ASC, which will certainly cause an error.

Beware of relying on PHPMyAdmin as a test of your queries. PHPMyAdmin has lots of tidying procedures in place which means that it accepts all sorts of garbage and tidies it up into a valid query. This is very useful for quick database maintentance. MySQL server, however, is not so forgiving.

Secondly, the citation of the connection procedure ($conn) is placed wrongly. It should be placed in the mysql_fetch_array() statement, thus:

$row=mysql_fetch_array($result, $conn)

Best o luck

Norm 😃

[sdjensen]

Why are you trying this inside the loop "while($row = mysql_fetch_array($result));"?

Remove the last while-line and try again. You don't need the do { } so remove that as well

$result = mysql_query ("SELECT EVENTYEAR, EVENTTEXT, 
YEAR(EVENTDATE) AS EVENTDATE FROM thisday 
WHERE substring(EVENTDATE,6) = substring(curdate(),6) 
ORDER BY EVENTYEAR ASC, 
",$conn); 

if ($row = mysql_fetch_array($result)) { 
echo "<table bgcolor=#FFFEEF border=0 cellpadding=1 cellspacing=0 style=border-collapse: collapse bordercolor=#111111 width=100%>"; 
echo "<font face=Verdana size=1 color=#800000><li>fra {$row['EVENTYEAR']} </b> {$row['EVENTTEXT']} 
</font></li></table>"; 

} else {print "no event";} 
?> 
</td> 
</tr> 

[sirTemplar]

the problem was the trailing comma after asc as Norman Graham
said. thank you pals!

[Norman_Graham]

Might I make another suggestion: don't use upper case for field names and table names in MySQL. Upper case should really be for command words (SELECT, ORDER BY, FROM, WHERE etc.) while lower case is for names and values. You'll thank yourself for sticking to this in 6 months time when you want to re-use a piece of code and can't remember what the hell you were thinking of when you wrote it.

Best o luck

Norm

[sirTemplar]

thanks for the suggestion Norman, sure to do that next time 🙂