Not returning an index of the values

[travisco87]

Nothing is being returned. Why? 

<?php
error_reporting(E_ALL);
ini_set("display_errors", 1);
require("../PHPMailer/class.phpmailer.php");
include 'includes.php';
        $mail = new PHPMailer;
        $subject = $_POST['subject'];
        $text = $_POST['newsletterBody'];
        $mail->IsSMTP();                                      // Set mailer to use SMTP
        $mail->Host = 'localhost';  // Specify main and backup server
        $mail->Port = '465';
        $mail->SMTPAuth = true;                               // Enable SMTP authentication
        $mail->Username = '**EMAIL_USERNAME**';                            // SMTP username
        $mail->Password = '**EMAIL_PASSWORD**';
        $mail->SMTPAuth = true;
        $mail->SMTPSecure = 'ssl';                            // Enable encryption, 'ssl' also accepted

        $mail->From = '**EMAIL_ADDRESS**';
        $mail->FromName = '**EMAIL_NAME';
        
    
        $email = getEmail("62", $DBH);
        
        
        foreach ($email as $newEmail)
                {
                    $addEmail = $newEmail->email;
                    $mail->AddAddress($addEmail);
                    $DBH = null;
                    $addEmail = "";
                }
        

        $mail->AddReplyTo('**EMAIL_REPLY_TO**', '**EMAIL_REPLY_TO_NAME**');
    
        $mail->WordWrap = 50;                                 // Set word wrap to 50 characters
        $mail->IsHTML(true);                                  // Set email format to HTML

        $mail->Subject = $subject;
        $mail->Body    = $text;
        $mail->AltBody = $text;

        if(!$mail->Send()) {
           echo 'Message could not be sent.';
           echo 'Mailer Error: ' . $mail->ErrorInfo;
           exit;
        }
        
            $insertSql = "INSERT INTO newsletter_log (date, title, body) VALUES (?,?,?)";
            $insertParams = array(time(), $subject, $text);
            $newsletterAdd = $DBH->prepare($insertSql);
            $newsletterAdd->execute($insertParams)
        
        echo 'Message has been sent <a href='newsletter.php'>Return back</a>";

?>

This is the include section that you need

function getEmail($inId, $DBH)
    {
        if (!empty($inId))
        {
            $blogstmt = $DBH->prepare("SELECT email_addr FROM newsletter_emails WHERE id = :userId");
            $blogstmt->bindParam(":userId", $inId);
            $blogstmt->execute();
        }
        else
        {
            $blogstmt = $DBH->prepare("SELECT * FROM newsletter_emails");
            $blogstmt->execute();
        }
    
        $postArray = array();
        
        $results = $blogstmt->fetchAll(PDO::FETCH_ASSOC);
        foreach($results as $row){
            $myPost = new nlEmails($row["id"], $row['email'], $DBH);
            array_push($postArray, $myPost);
        }
        
        return $postArray;
        $DBH = null;
    }
 
class nlEmails {
    public $id;
    public $email;
    
    function __construct($inId=null, $inEmail=null, $DBH)
    {
        if(!empty($inId))
        {
            $this->id = $inId;
        }
        if(!empty($inEmail))
        {
        $this->email = $inEmail;
        }
    }
}

The database is created with the newsletter_emails "id", "first_name", "last_name", "email_addr". Thanks! 

[maxxd]

If you run the query directly, does it return the expected results? In other words, are you sure you have a record with the id 62 in the database?

[Ch0cu3r]

Also what is the the n1Emails class used for? You seem to return the results as an associative array and then convert the results into a object manually.

 

PDO can return the results as an object, use PDO::FETCH_OBJ instead of PDO::FETCH_ASSOC.

[jcbones]

So which is it?

 

The database is created with the newsletter_emails "id", "first_name", "last_name", "email_addr". Thanks!

 

$myPost = new nlEmails($row["id"], $row['email'], $DBH);

[maxxd]

One more thing I just thought of - when you say nothing is being returned, do you mean you get no data, or you get no output at all? I'm pretty sure that jcbones' observation should have thrown an error because the referenced array index doesn't exist, but if you're not getting that, then you need to turn on error reporting.