You only want to retrieve the news from your table once.
So, pull pull this code:
>  $container[] = $message;
>  while (list($news, $date) = mysql_fetch_row($newsfetch)) {
>      $container[] = '<a href='newslink.php'>' . $news . '</a>';
>      $container[] = '<br><hr><br>';
>   }
>   $container[] = '</body></html>';

outside of (before) this loop:

> while ($data = mysql_fetch_row($emailfetch)) {

        ~Chris                         /"\
                                       \ /     Microsoft Security Specialist:
                                        X      The moron in Oxymoron.
                                       / \     http://www.thebackrow.net

On Wed, 3 Apr 2002, Gerard Samuel wrote:

> Ok, this one is breaking my back all day.
> First some sudo-code ->
>
> fuction email_to_user() {
>      $sql = 'select distinct(email) from user';
>      $emailfetch = mysql_query($sql);
>
>      $sql = 'select news, date from news order by sid desc limit 10';
>      $newsfetch = mysql_query($sql);
>
>      while ($data = mysql_fetch_row($emailfetch)) {
>          $container = array();
>          $container[] = '<html><head><title></head><body>';
>
>          $message = 'Have a nice day';
>          $container[] = $message;
>         // PROBLEM IN THIS WHILE LOOP MAYBE //
>          while (list($news, $date) = mysql_fetch_row($newsfetch)) {
>              $container[] = '<a href='newslink.php'>' . $news . '</a>';
>              $container[] = '<br><hr><br>';
>          }
>          $container[] = '</body></html>';
>          $message = '';
>          foreach($container as $foo) {
>              $message .= $foo;
>          }
>          mail(Send mail to $data[0]);
>          unset($container);
>      }
> }
>
> Basically it grabs all the user's email addresses, then loop them.
> On each loop grab all news items.
> Then emails results to the user and moves on to the next user.
>
> Im running this on my test box, that only has two users, but the 2nd
> user never gets the expected results.
> The first user get the message and the news.
> The second only gets the message.
> The code structure is pretty much unchanged from a working example till
> I started using $container to hold array elements.
>
> Could anyone see bad logic in the above code??
> Thanks
>
>
> --
> PHP General Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>


-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to