Hello Can someone look at my code and tell me why I am getting this message?

haggertfly

Warning: Cannot modify header information - headers already sent by (output started at /home/content/f/t/p/ftpcarrie/html/mail/class.SimpleMail.php5:64) in /home/content/f/t/p/ftpcarrie/html/mail/admin_transact.php5 on line 81

class.SimpleMail.php5

<?php

class SimpleMail {
public $to = NULL;
public $cc = NULL;
public $bcc = NULL;
public $from = NULL;
public $subject = '';
public $body = '';
public $htmlbody = '';
public $send_text = TRUE;
public $send_html = FALSE;
private $message = '';
private $headers = '';

public function send($to = NULL,
$subject = NULL,
$message = NULL,
$headers = NULL) {
if (func_num_args() >= 3) {
$this->to = $to;
$this->subject = $subject;
$this->message = $message;
if ($headers) {
$this->headers = $headers;
}
} else {

            if ($this->from) {
                    $this->headers .= "From: " . $this->from . "\r\n";
                    }
            if ($this->cc) {
                    $this->headers .= "Cc: " . $this->cc . "\r\n";
                    }              
            if ($this->bcc) {
                    $this->headers .= "Bcc: " . $this->bcc . "\r\n";
                    }                     
            if ($this->send_text and !$this->send_html) {
                    $this->message = $this->body;

            } elseif ($this->send_html and $this->send_text) {
            $this->message = $this->htmlbody;
            $this->headers = "MIME-Version: 1.0\r\n";
            $this->headers = "Content-type: text/html; " .
                                             "charset=iso-8859-1\r\n";

            } else {

            $_boundary = "==MP_Bound_xyccr948x==";

            $this->headers = "MIME-Version 1.0\r\n";
            $this->headers .= "Content-type: multipart/alternative; " .
                                             "boundary=\"$_boundary\"\r\n";

            $this->message = "This this a Multipart Message in " .
                                          "MIME format\n";
            $this->message .= "--$_boundary\n";
            $this->message .= "Content-Type: text/plain; " .
                                              "charset=\"iso-8859-1\"\n";
            $this->message .= "Content-Transfer-Encoding: 7bit\n\n";
            $this->message .= $this->body . "\n";
            $this->message .= "--$_boundary\n";
            $this->message .= "Content-Type: text/html; " .
                                              "charset=\"iso-8859-1\"\n";
            $this->message .= "Content-Transfer-Encoding: 7bit\n\n";
            $this->message .= $this->htmlbody . "\n";
            $this->message .= "--$_boundary--";     
            }
    }
       if (!mail ($this->to,$this->subject,$this->message,$this->headers)) {
            throw new Exception('Sending mail failed');
            return FALSE;
    } else {
            return TRUE;
            }
    }

}
?>

admin_transact.php5

<?php
require('config.php');
require('class.SimpleMail.php5');

$conn = mysql_connect(SQL_HOST, SQL_USER, SQL_PASS)
or die('Could not connect to Mysql Database. ' . mysql_error());

mysql_select_db(SQL_DB, $conn);

if (isset($POST['action'])) {
switch ($POST['action']) {
case 'Add New Mailing List':
$sql = "INSERT INTO ml_lists (listname) " .
"VALUES ('" . $POST['listname'] . "')";
mysql_query($sql)
or die('Could not add mailing list. ' . mysql_error());
$sql = "DELETE FROM ml_subscriptions " .
"WHERE ml_id=" . $POST["ml_id"];
mysql_query($sql)
or die('Could not delete mailing list subscriptions. ' .
mysql_error());
break;

case 'Send Message':
	if (isset($_POST['msg'], $_POST['ml_id']))  {
		if (is_numeric($_POST['ml_id'])) {
			$sql = "SELECT listname FROM ml_lists " . 
				   "WHERE ml_id='" . $_POST["ml_id"] . "'";
			$result = mysql_query($sql, $conn)
			or die (mysql_error());
			$row = mysql_fetch_array($result);
			$listname = $row['listname'];
		}   else   {
		    $listname = "Master";
		}

		$sql = "SELECT DISTINCT usr.email, usr.firstname, usr.user_id " . 
			   "FROM ml_users usr " . 
			   "INNER JOIN ml_subscriptions mls " . 
			   "ON usr.user_id = mls.user_id " . 
			   "WHERE mls.pending=0";
	    if ($_POST['ml_id'] !='all') {
		   $sql .= " AND mls.ml_id=" . $_POST['ml_id'];
		}

		$result = mysql_query($sql)
		  or die('Could not get list of email addresses. ' . mysql_error());

		$headers = "From: " . ADMIN_EMAIL . "\r\n";

		while ($row = mysql_fetch_array($result)) {
			if (is_numeric($_POST['ml_id'])) {
				$ft = "You are recieving this message as a member of the ";
				$ft .= $listname . "\n mailing list. If you have recieved ";
				$ft .= "this email in error, or would like to\n remove your";
				$ft .= "name from this mailing list, please visit the ";
				$ft .= "following URL:\n";
				$ft .= "http://" . $_SERVER['HTTP_HOST'] .
					   dirname($_SERVER['HTTP_HOST']) . "/remove.php?u=" .
					   $row['user_id'] . "&ml=" . $_POST['ml_id'];
			}	else	{

				$ft = "You are recieving this email because you subscribed ";
				$ft = "to one or more\n mailing lists. Visit the following ";
				$ft = "URL to change your subscriptions:\n";
				$ft = "http://" . $_SERVER['HTTP_HOST'] . 
					  dirname($_SERVER['PHP_SELF']) . "/user.php?u=" . 
					  $row['user_id'];
			}

			$msg = stripslashes($_POST['msg']) . "\n\n";
			$msg .= "_______________\n";
			$msg .= $ft;

			$email = new SimpleMail();

			$email->send($row['email'],
						stripslashes($_POST['subject']),
						$msg,
						$headers)
			 or die('Could not send email.');
		}
	}
	break;
}

}

header('Location: admin.php5');

sneakyimp

i'm not certain, but it would seem that this code:

if (!mail ($this->to,$this->subject,$this->message,$this->headers)) {
throw new Exception('Sending mail failed');
return FALSE;

is outputting text. if you output any text to the browser, then you cannot send additional headers as you do on this line:

header('Location: admin.php5');

ALSO...you should use the code formatting tags. put a [ PHP ]
before
and
[/ PHP] after.

Weedpacket

sneakyimp's got it in one: if you throw an exception, you have to either catch it at some point or the script will halt and the exception text will be output (see more about try/catch/throw exception handling in the manual).

In this case, just return false. If the function returns false, then there was an error. That tells you everything the attempted exception was telling you.