[RESOLVED] foreach and if statement trouble

mpar612

Hi everyone,

I am having an issue that has been bugging me all day. I might be making some minor error that I just have not been able point out.

In the code below, the first part of the if statement is what always prints. It's as if everything after the else is ignored.

Does anyone see any mistakes in my code? Thanks in advance!!!

$sql_links = "SELECT link_text, url, link_category, link_image FROM links WHERE link_category = '$row_cat[category]'";
$rows_links = $db->getAll($sql_links);
foreach ($rows_links as $row_links) {
if ($row_links[link_image] == 0) {										print"<tr><td>&nbsp;</td><td><a href=\"$row_links[url]\">$row_links[link_text]</a></td><td></tr>";
} else {														print"<tr><td><img src=\"$row_links[link_image]\" height=\"60\" width=\"60\"></td><td><a href=\"$row_links[url]\">$row_links[link_text]</a></td><td></tr>";
}
}

Installer

Presumably you have "link_image" stored as text. Text that doesn't start with a digit is "==" 0. Try:

if ($row_links[link_image] === 0)

mpar612

Thanks! That didn't do anything. It still functions the same way.

Does anyone have any other thoughts?

Thanks very much!

Installer

If you're storing "0" when there's no link image for the row then try this:

if ($row_links['link_image'] == '0')

(You should quote your associative array keys.)

mpar612

Thanks for your response! That didn't do anything and it still functions the same way.

Here is all of my PHP code except my db connect info. Maybe I am making some error in another part of the code? Does anyone have any other suggestions? I'm kind of dumbfounded by this.

I appreciate any help. Thanks!

<?php
print "<tr><td colspan=\"2\">";
$rows_cat = $db->getAll('SELECT category FROM link_category');
foreach ($rows_cat as $row_cat) {
print "<tr><td><b><font face=\"Arial\" color=\"#000000\" size=\"2\">
$row_cat[category]:</b></font>
</td></tr>";

$sql_links = "SELECT link_text, url, link_category, link_image FROM links WHERE link_category = '$row_cat[category]'";
$rows_links = $db->getAll($sql_links);
foreach ($rows_links as $row_links) {
if ($row_links['link_image'] == 0) {

print"<tr><td><img src=\"$row_links[link_image]\" height=\"60\" width=\"60\"></td><td><a href=\"$row_links[url]\">$row_links[link_text]</a></td><td></tr>";
} else {
print"<tr><td>&nbsp;</td><td><a href=\"$row_links[url]\">$row_links[link_text]</a></td><td></tr>";
}
}
}
?>

Installer

Still assuming that you're storing an "empty" link_image field as text "0" (since you haven't said otherwise), this works for me (identical to my last post and tested):

if ($row_links['link_image'] == '0')

mpar612

Sorry, that I forgot to mention that.

I pasted my code from an old file into a new one and it worked. I'm not sure what's up with that.

Thanks for your help!