[iamcenz]

Hello all. I got an while() loop that is being a pain in the butt. I dont know why it isnt working. I am using the exact same code in another spot on the site and it works perfectly.

here is the code that isnt working.

   <?php
while($row_a = mysql_fetch_array($res))
{ 
?>
        <div id="rec_title">- <?php echo $row_a['title']; ?></div> <div id="status"><img src="/images/usercp/<?php echo $row_a['approved'];?>.png" /></div><br/>
		<?php } ?>

here is the exact same loop on another page doing the same thing and working

<?php
while($row_a = mysql_fetch_array($res))
{ 
?>
<div id="inbox_content">
<div id="from"><?php  echo $row_a['from_user']; ?></div>
<div id="title"><a href="view_pm.php?id=<?php echo $row_a['pm_id']; ?>">
  <?php  echo $row_a['title']; ?>
</a></div>
<div id="delete"><a href="delete_pm.php?id=<?php echo $row_a['pm_id']; ?>" onmouseout="MM_swapImgRestore()" onmouseover="MM_swapImage('delete','','images/usercp/delete_button_over.png',1)"><img src="images/usercp/delete_button.png" alt="Delete" name="delete" width="55" height="18" border="0" id="delete2" /></a></div>
</div>
<?php } ?>

[RudiVisser]

What do you mean by

Quote

isn't working

?

What is the value of $res, have you used var_dump to check that it is indeed a MySQL Result Resource?

[Null::Void()]

Why did you do <?php } ?>, Your code is all messy, i am not sure why it works on the other one. Try to contain all your code in one <?php ?> and never do what you did with the braces, Also post the Query but i am pretty sure its the structure of your code.

[RudiVisser]

Null::Void(), on 04 October 2012 - 02:12 PM, said:

Why did you do <?php } ?>, Your code is all messy, i am not sure why it works on the other one. Try to contain all your code in one <?php ?> and never do what you did with the braces

Because he broke out of PHP to output HTML. It's not the right way to achieve the result he wants, but it's certainly valid.

This post has been edited by RudiVisser: 04 October 2012 - 06:27 AM

[Duckington]

The issue is not with the loop, it is with the $res variable you are using in the loop. Make sure it contains what you expect it to.

[Null::Void()]

RudiVisser, on 04 October 2012 - 06:25 AM, said:

Null::Void(), on 04 October 2012 - 02:12 PM, said:

Why did you do <?php } ?>, Your code is all messy, i am not sure why it works on the other one. Try to contain all your code in one <?php ?> and never do what you did with the braces

Because he broke out of PHP to output HTML. It's not the right way to achieve the result he wants, but it's certainly valid.

I know its valid but it seems like an easy way to get lost and have things not work as expected.