Question

My email will only send if I type in the actual address rather than using $usr_email, although it says ‘message sent’. The Email address comes from the user_email field in users table.
This is defined with
$id = intval($_SESSION[‘user_id’]);

if (isset($_POST['doSend'])) {


function getTwo($query){
    $res = mysql_query($query);
    if (!$res) {
        trigger_error("db: ".mysql_error()." in ".$query);
        return FALSE;
    }
    if ($row = mysql_fetch_row($res)) {
        return $row[0];
    }
}

$getuserinfo_q = "SELECT user_email AND user_name FROM users WHERE 
id='$_SESSION[user_id]'";

$getuserinfo_e=mysql_query($getuserinfo_q);

if(mysql_num_rows($getuserinfo_e) < 1){

    echo "User details not found - User_id is not in DB";

    exit();

}

$user_info_val=mysql_fetch_assoc($getuserinfo_e);

if(empty($user_info_val['user_email'])){

    echo "there is no such column name as 'user_email'"; //tell the user about column

    exit(); //shut off the script

}


$usr_email=$user_info_val['user_email'];
  $user_name=$user_info_val['user_name'];



$sqltest = "SELECT completed_status From users where id =
'$_SESSION[user_id]'";
$isSending = getTwo($sqltest);
$isSending === false;
if($isSending >= 6){
    require_once "Mail.php";
    require_once "Mail.php";



    $from = "<xxx>";
    $to = "$usr_email";
    $subject = "hi";
$body ="Chi ";

$host = "ssl://smtp.gmail.com";
$port = "465";
    $username = "xxx";
    $password = "xxxx";

 $headers = array ('From' => $from,
'To' => $to,
'Subject' => $subject);
$smtp = Mail::factory('smtp',
array ('host' => $host,
    'port' => $port,
    'auth' => true,
    'username' => $username,
    'password' => $password));

$mail = $smtp->send($to, $headers, $body);

    if (PEAR::isError($mail)) {
        echo("<p>" . $mail->getMessage() . "</p>");
    } else {
        echo("<p>Message successfully sent!</p>");
    }
}
else
    header ("Location: error.php");



}

Answer 1

BIG EDIT:

OKAY, So lets hope this works -_-

$getuserinfo_q = "SELECT user_email AND user_name FROM users WHERE id ='
".intval($_SESSION['user_id'])."'";

$getuserinfo_e=mysql_query($getuserinfo_q);

if(mysql_numb_rows($getuserinfo_e) < 1){

echo "User details not found - User_id is not in DB";

exit();

}

$user_info_val=mysql_fetch_assoc($getuserinfo_e);

if(empty($user_info_val['user_email'])){

echo "there is no such column name as 'user_email'"; //tell the user about column

exit(); //shut off the script

}


$usr_email=$user_info_val['user_email'];


$sqltest = "SELECT completed_status FROM users WHERE id ='
".intval($_SESSION['user_id'])."'";
$isSending = getTwo($sqltest);
$isSending === false;
if($isSending >= 6){
    require_once "Mail.php";

// Start NEW CLASS (for your weird Mail function)

$m_class=new Mail;


    $from = "<xxx>";
    $to = "$usr_email";
    $subject = "hi";
$body ="Chi ";

$host = "ssl://smtp.gmail.com";
$port = "465";
    $username = "xxx";
    $password = "xxxx";

 $headers = array ('From' => $from,
'To' => $to,
'Subject' => $subject);


$smtp = $m_class->factory('smtp',
array ('host' => $host,
    'port' => $port,
    'auth' => true,
    'username' => $username,
    'password' => $password));

$mail = $m_class->send($to, $headers, $body);

}
else
    header ("Location: error.php");

As you may have noticed, I removed the error checking ( the PEER::isError() thing) simply because I could not get my head around how that function should be called (not statically). But, other than that, the code above should do the trick (hopefully).

Give it a go, and let me know how it works.