Errors with Laravel when adding an IF statement

Question

I've recently inherited some websites from the developer that was here before me, and I'm having trouble adding a simple IF statement to his code.

The page simply says that "an error has occurred" and there's nothing being written to the logs folder.

Here is the code:

protected function sendContactEmail($data)
{
    $enquiry = $data['enquiry'];
    $email = $data['email'];
    $name = $data['name'];

    if($enquiry=='General Enquiry'){
        $to = '********';
        $cc = '********';
    }
    else if($enquiry=='Product Feedback'){
        $to = '********';
        $cc = '********';
    }
    else { //Trade Enquiry
        $to = '********';
    } 

    //email to admin
    Mail::send('contact.email.contact-admin', $data, function ($message) use ($enquiry) {
        $message->from('no-reply@********.com.au', '********');
        $message->replyTo('no-reply@********.com.au', '********');
        $message->subject($enquiry);
        $message->to($to);
        $message->cc($cc);
    });

    //email to user
    Mail::send('contact.email.contact-user', $data, function ($message2) use ($email) {
        $message2->from('no-reply@********.com.au', '********');
        $message2->replyTo('no-reply@********.com.au', '********');
        $message2->subject('********');
        $message2->to($email);
    });
}

Anyone got any ideas on where to start?

What version of Laravel are you using? Is your environment set to DEV? — fubar
– fubar, Commented Feb 5, 2019 at 2:26
$cc isn't given a default value. When $enquiry is matches none of the conditions, $cc is never initialized. — Oluwafemi Sule
– Oluwafemi Sule, Commented Feb 5, 2019 at 2:28
@OluwafemiSule - I didn't know they need to be initialised. When the script tries to use the variable, it will always have been defined by that point. However, I added defaults anyway and it hasn't fixed the issue. — Craigy Craigo
– Craigy Craigo, Commented Feb 5, 2019 at 2:58

Oluwafemi Sule · Accepted Answer · 2019-02-05 03:04:43Z

5

Both $to and $cc variables are not inherited from the parent scope of the Closure passed in Mail::send.

Both variables have to be specified in the use language construct. ¹

Mail::send('contact.email.contact-admin', $data, function ($message) use ($enquiry, $to, $cc) {
    $message->from('no-reply@********.com.au', '********');
    $message->replyTo('no-reply@********.com.au', '********');
    $message->subject($enquiry);
    $message->to($to);
    $message->cc($cc);
});

answered Feb 5, 2019 at 3:04

Oluwafemi Sule

39.3k1 gold badge63 silver badges88 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Errors with Laravel when adding an IF statement

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related