Not working in queue

[edikurniawan-dev]

I have a job to sent email to all users, after process sent email done, i update the data, but why column updated_by be null?

// app/Jobs/ProcessSendEmail.php

class ProcessSendEmail implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    /**
     * Create a new job instance.
     */
    public function __construct(protected Payroll $payroll)
    {
        //
    }

    /**
     * Execute the job.
     */
    public function handle(): void
    {
        try {
            Mail::to($payroll->email)->send(new PayrollEmail($payroll, true));

            Payroll::find($this->payroll->id)->update([
                'sent_at' => Carbon::now(),
            ]);
        } catch (\Exception $e) {
        }
    }
}

[alexmayo]

Does the Payroll model use the HasUserStamps trait? See the below example.

use Sqits\UserStamps\Concerns\HasUserStamps;

class Payroll extends Model {

    use HasUserStamps;

    //...

}

[edikurniawan-dev]

Yes, Payroll model use the HasUserStamps trait.

This code work fine if in controller, but if in jobs updated_by to be null

Mail::to($payroll->email)->send(new PayrollEmail($payroll, true));

Payroll::find($this->payroll->id)->update([
      'sent_at' => Carbon::now(),
]);

[alexmayo]

You should pass the userId to the job, and then utilise Auth::loginUsingId to authenticate inside the job. Otherwise, the job (and userstamps) has no idea who the current user is, which sets the updated_by to null.

// app/Jobs/ProcessSendEmail.php

class ProcessSendEmail implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    /**
     * Create a new job instance.
     */
    public function __construct(protected Payroll $payroll, protected  $userId)
    {
        //
    }

    /**
     * Execute the job.
     */
    public function handle(): void
    {
        if(Auth::loginUsingId($this->userId)) {
            try {
                Mail::to($this->payroll->email)->send(new PayrollEmail($this->payroll, true));

                Payroll::find($this->payroll->id)->update([
                    'sent_at' => Carbon::now(),
                ]);
            } catch (\Exception $e) {
            }
        }
    }
}

Then, when queueing the job, ensure you pass the current user ID to the new job.

new ProcessSendEmail($payroll, $user->id);

[edikurniawan-dev]

Ahh I see, thanks for that solution