Data not being passed to queued job

You should declare these variables public.

Thanks, I was just coming back to post that. I figured it out after examining the SerializesModels trait.

I am having the same issue, could you give an example of what needs to be set public? According to https://laracasts.com/series/intermediate-laravel/episodes/11 the variables passed into the constructor need to be protected? Any help would be appreciated.

Here is what I am doing:

<?php

namespace App\Jobs;

use App\Jobs\Job;
use Illuminate\Queue\SerializesModels;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Bus\SelfHandling;
use Illuminate\Contracts\Queue\ShouldQueue;

use App\ArtworkImages;
use Storage;
use Uuid;
use Image;

class ImageResizeS3 extends Job implements SelfHandling, ShouldQueue
{
    use InteractsWithQueue, SerializesModels;
    
    protected $data;
    
    /**
     * Create a new job instance.
     *
     * @return void
     */
    public function __construct($data)
    {
        $this->id = $data['id'];
        $this->s3key = $data['s3key'];
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        Storage::makeDirectory('tmp/');
        $tmp_filename = 'tmp/'.Uuid::generate(4)->string.'-id-'.$this->id;
    
...

    }
}

Dispatching the job

$this->dispatch(new ImageResizeS3(array(
            'id' => $ID,
            's3key' => $targetKeyname
        )));

@tromtv the $data variable in your constructor method is only available and is unique to the local scope (that is, within the constructor itself). It has nothing to do with class property $data.

You should declare class properties $id and $s3key and make them public, likewise

class ImageResizeS3 extends Job implements SelfHandling, ShouldQueue
{
    public $id;
    public $s3key;

    public function __construct($data)
    {
        $this->id = $data['id'];
        $this->s3key = $data['s3key'];
    }
}

@toniperic Thanks for your quick reply. I don't know how I missed this, thanks for pointing it out. I guess I do not really understand how in https://laracasts.com/series/intermediate-laravel/episodes/11 at 7:05 min:sec he is not setting the assigned variables public in the class and still can access them from within the handle function. But anyways, its working now. Thanks so much again for pointing this out!

Best

You don't have to set it public. You can also set it protected (and probably private too). Point is you have to declare it independently of what goes on in the __config() function.

Unfortunately this is not mentioned explicitly in the documentation:

https://laravel.com/docs/5.5/queues#connections-vs-queues

Variables are indeed declared in the code, but the significance is left for the reader to work out himself.

I suppose this is just standard functionality for how PHP works with properties.

Just declare those variable in which you want to recieve data and you can see the dump in terminal.

//Controller
public function func_name(){
    $job = (new SendWelcomeEmail($data))
    dispatch($job);
}


//In Jobs/SendWelcomeEmail
class SendWelcomeEmail implements ShouldQueue
{
     protected $data;

     public function __construct($data)
        {
            $this->data= $data;    
        }

    public function handle(){
        //check your console
        dd($this->data);
    }
}