Mailing code giving an error

Question

I am new to php & have tried out some code in php for sending mail to user by some simple means, i am facing some issues as code giving a error..!! please help me.

php

$to      =  ' '". $_SESSION['email'] ."' ';
        $subject = 'Your vault number';
        $message = 'Your vault number is '". $_SESSION['vault_no'] ."' ';
        $headers = 'From: innovation@miisky.com' . "\r\n" .
            'Reply-To: innovation@miisky.com' . "\r\n" .
            'X-Mailer: PHP/' . phpversion();

mail($to, $subject, $message, $headers);

sorry sir..!! its throwing a Parse error: syntax error, unexpected '"' in your code $to = ' '". $_SESSION['email'] ."' '; — user, Apr 19 '16 at 06:26
yes sir!! i have specified session_start(); at beginning of my code! — user, Apr 19 '16 at 06:31
I only faced this Problem when I either forgot to start the session, the values where emty or they where an array instead of a string. — pguetschow, Apr 19 '16 at 06:32
Is there an error in $to = ' '". $_SESSION['email'] ."' '; part of code.! its pointing error here — user, Apr 19 '16 at 06:32
Oh yeah, I was blind. That's your problem :) $to = $_SESSION['email']; — pguetschow, Apr 19 '16 at 06:33

score 2 · Accepted Answer · answered Apr 19 '16 at 06:33

2

You have a Syntax error in your string assignment:

$to      =  ' '". $_SESSION['email'] ."' ';
              ^ here                   ^ and here

You don't even need delimiters as part of your final string,

$to      =  $_SESSION['email'];

is enough, as $_SESSION['email'] already is a string.

answered Apr 19 '16 at 06:33

T3 H40

2,326
8
32
43

Damn you beat me about a few seconds :) – pguetschow Apr 19 '16 at 06:37
@TechTreeDev Yes :) BTW, OP doesn't need delimiters there, in fact I think this would break his `mail()` function if you add `'` to the address – T3 H40 Apr 19 '16 at 06:38

score 1 · Answer 2 · answered Apr 19 '16 at 06:34

1

Your Problem ist that you have a syntax error on your $to variable.

$to      =  ' '". $_SESSION['email'] ."' ';

Change them to

$to      =  $_SESSION['email'];;

answered Apr 19 '16 at 06:34

pguetschow

5,176
6
32
45

Mailing code giving an error

2 Answers2