Queue is stripping out data

Once you send the mail to the queue you don't have access to the session variables: Ie input::get. You passed in the $data array? You have to reference that variable $data['name'] Etc

@jekinney sorry I have a noob question. Do I put $data['name'] in the enquire.blade.php instead of Input::get ('name')? I am a little lost.

You can in line it.

If it's a blade template you don't need the <?PHP or set the variables. I suggest setting all your data and pass it in like you did except the date and user ip. Add those in your controller to your data array that's already being passed in to your mail code. I'm on my mobile so no ticks to show a code block.

Use the blade syntax curly braces and the $data['array key'] no need to set a different variable.

In your view (email template) use the variable to out put the data from the array

@jekinney Would it be something like this?

<?php

Input::get($data['name', 'email', 'subject', 'msg']);
?> 

<h1>We been contacted by.... </h1>

<p>
Name: $data('name'); <br>
Email address: $data('email');<br>
Subject: $data('subject'); <br>
Message: $data('msg');<br>
</p>

No. You need brackets not parenthesis.

Second drop the input in your view.

Is your contact view blade.php?

In your controller method getContactUs

$data = $request->all(); $data[] = [$request->getClientIp, 'date'=>date()];

This should give you all the variables you need to pass into your mail.

If someone doesn't beat me to it in my morning I'll show you with a code block.

@jekinney please bare with me, is this closer to what you mean? Controller:

        public function getContactUsForm(){

            //Get all the data and store it inside Store Variable
            $data = $request->all(); 
        $data[] = [$request->getClientIp, 'date'=>date()];

            //Validation rules
            $rules = array (
            'name' => 'required',
            'email' => 'required|email',
            'subject' => 'required',
            'msg' => 'required',
            );

            //Validate data
            $validator  = Validator::make ($data, $rules);

            //If everything is correct then run passes.
            if ($validator -> passes()){

                //Send email using Laravel send function
                Mail::Queue('enquiry', $data, function($msg) use ($data)
                {
            //email 'From' field: Get users email add and name
                 
            //email 'To' field: cahnge this to emails that you want to be notified.                    
            $msg->to('support@thehautie.com', 'Enquiry')->subject('contact request');

                });

                return Redirect::to('/')->with(array('note' => 'Thank you! Your message has been recieved.', 'note_type' => 'success') );  
            }else{
//return contact form with errors
                return Redirect::to('/contact')->withErrors($validator);
            }
        }
}

email view:

<?php
$data['name', 'email', 'subject', 'msg'];

?> 

<h1>We been contacted by.... </h1>

<p>
Name: $data['name']; <br>
Email address: $data['email'];<br>
Subject: $data['subject']; <br>
Message: $data['msg'];<br>
</p>

</p>

The contact page is a blade view.

the $request variable in the controller is throwing an undefined error

Okay if you do

$data = $request->input();

then use that as the data object you pass through you'll be able to access Input::get('name') by literally typing the variable name

<p>
Name:$name <br>
Email address: $email<br>
Subject:$subject <br>
Message: $msg<br>
</p>

with blade syntax around the variables of course

Or you could be more explicit and write:

$data = [
            'name' => $request->input('name'),
            'phone' => $request->input('phone'),
            'email' => $request->input('email'),
            'message_form' => $request->input('msg'),
        ];

@Man

@karlos545 the $request still throws an undefined variable error, and is $data = $request->input(); for the controller or the email being sent?

You need to pass in the Request object

 public function getContactUsForm(Request $request){

// Don't forget to import it at the top

@karlos545 now i get this error:

Call to undefined method Illuminate\Support\Facades\Request::input()

its not a Facade

use

$data = $request->input();

// Not 

$data = $request::input();

@Man :)

Yes, that is what i am using that is throwing the error

This is just a guess since we can't see your code, but I'd be willing to bet that it's throwing that error because you imported the Request facade at the top since you probably use the Request facade elsewhere in the controller.

@thomaskim Thats a great shout.

its the Request objet under the illuminate path that you want @Man

Put this in your controller.

$data['name'] = Input::get ('name');
$data['email'] = Input::get ('email');
$data['subject'] = Input::get ('subject');
$data['msg'] = Input::get ('msg');
$data['date_time'] = date("F j, Y, g:i a");
$data['userIpAddress'] = Request::getClientIp();

This is the entire controller as it stands:

<?php 

use \Input as Input;
use \Session as Session;
use \Redirect as Redirect;
use \Validator as Validator;

class ContactController extends Controller {



     //Server Contact view:: we will create view in next step
        public function getContact(){

            return View::make('contact');
        }

        //Contact Form
        public function getContactUsForm(Request $request){

            //Get all the data and store it inside Store Variable
            $data = $request->all(); 
            $data[] = [$request->getClientIp, 'date'=>date()];
            

            //Validation rules
            $rules = array (
            'name' => 'required',
            'email' => 'required|email',
            'subject' => 'required',
            'msg' => 'required',
            );

            //Validate data
            $validator  = Validator::make ($data, $rules);

            //If everything is correct then run passes.
            if ($validator -> passes()){

                //Send email using Laravel send function
                Mail::Queue('enquiry', $data, function($msg) use ($data)
                {
            //email 'From' field: Get users email add and name
                 
            //email 'To' field: cahnge this to emails that you want to be notified.                    
            $msg->to('my@email.com', 'Enquiry')->subject('contact request');

                });

                return Redirect::to('/')->with(array('note' => 'Thank you! Your message has been recieved.', 'note_type' => 'success') );  
            }else{
//return contact form with errors
                return Redirect::to('/contact')->withErrors($validator);
            }
        }
}

I don't see where else the request was coming from.

@jekinney did you get a chance to take a look at the code so far?

Could you show us the email view - 'enquiry' also.

Another thing - how is this working? The command should be:

Mail::queue() // lower case q

This is becoming one giant mess ^^

Add Request to your use block.

@dbrmr this is the current enquire.blade.php file. We have been fine tuning it through this forum:

<?php
$data = $request->input();

?> 

<h1>We been contacted by.... </h1>

<p>
Name:$name <br>
Email address: $email<br>
Subject:$subject <br>
Message: $msg<br>
</p>

The Queue alias is upper case in my app.php

@PLB-RR yes it is becoming a bit of a mess. adding request to the use did not solve the error.

The initial code in the question works, I just want it to work when it is passed through the queue.

@Man

<?php 

use Illuminate\Http\Request; // preferred Request Facade instead of Input;
use Carbon\Carbon; // Set the date in a carbon instance for flexibility.
use \Session as Session;
use \Redirect as Redirect;
use \Validator as Validator;

class ContactController extends Controller {



     //Server Contact view:: we will create view in next step
        public function getContact(){

            return View::make('contact');
        }

        //Contact Form
        public function getContactUsForm(Request $request){         
    
        $data = [
            'name' => $request->get'(name'), 
            'email' => $request->get('emai'l),
            'subject' => $request->get('subject'),
            'msg' => $request->get('msg'),
            'date' => Carbon::now(), 
            'client_ip' => $request->getClientIp(),
        ];
            //Validation rules Validate First less work if an error
            $rules = array (
            'name' => 'required',
            'email' => 'required|email',
            'subject' => 'required',
            'msg' => 'required',
            );

            //Validate data
            $validator  = Validator::make ($data, $rules);

            //If everything is correct then run passes.
            if ($validator -> passes()){
                //Send email using Laravel send function
                Mail::queue('enquiry', $data, function($msg) use ($data)
                {
            //email 'From' field: Get users email add and name
                 
            //email 'To' field: cahnge this to emails that you want to be notified.                    
            $msg->to('my@email.com', 'Enquiry')->subject('contact request');

                });

                return Redirect::to('/')->with(array('note' => 'Thank you! Your message has been recieved.', 'note_type' => 'success') );  
            } // Don't need else statements as a return stops the script from continuing.
            //return contact form with errors
            return Redirect::to('/contact')->withErrors($validator);
        }
}

View: Assuming once again it is a blade view just like any other Laravel view (file name has .blade.php)

<h1>We been contacted by.... </h1>

<p>
Name: {{ $data{'name'] }}<br>
Email address: {{ $data['email'] }} <br>
Subject: {{ $data['subject'] }}<br>
Message: {{ $data['msg'] }}<br>
Date: {{ $data['date'] }}<br>
User IP address: {{ $data['client_ip'] }}<br>

</p>

Setting the date in the view will set when the email was sent not when the email was fired to send (Before queued and after queued recursively). The client IP through the requst as in your original post I am guessing is the email servers IP? Not sure but I don't think it was the user's IP. Setting it in the array at the time of request (Before queuing) you can be assured it is the proper IP.

Test it and let me know!

@jekinney I get an Undefined variable: data error. I edited the data string to :

$data = ['name' => $request->get('name'), 'email' => $request->get('email'), 'date' => date(), 'client_ip' => $request->getClientIp(),];

because i think your original had some minor formatting errors, but I still got the undefined variable: data error

@Man

Yeah, your quick. I edited the above post twice right after each other!!!!! I forgot a ' after email on the first pass.

Where do you get the error?

Dont reference the $data array, the keys within the array get converted to variables so...

$data['name']
// can actually be referenced by entering 
$name

@Man

@Man

@karlos545 maybe right. I don't remember off the top of my head.

@jekinney the code gets sent to the queue now, only issue i have is when i run queue:work. I get the error:

 [ErrorException]      
  Undefined index: job