Laravel 8 Task scheduler

Question

I'm using laravel Scheduler, i want to set a subscription status to expired if a certain condition is met, and also need this to run every mins in the background..Figured task scheduler is the best option but couldn't figure out why the code is'nt executing after running php artisan schedule:work ..Below is my code Located in App/Console/Kernel.php (schedule function)

  $schedule->call(function () {
        $checkSubStatus = UserSubscription::where('status', 'approved')->where('users_id', auth()->user()->id)->first();
        $currentTime = Carbon::now();
        if($currentTime > $checkSubStatus->expiry_date) {
            $checkSubStatus->update([
                'status' => 'expired'
            ]);
        }
    })->everyMinute();

But works when i just delete the table, like so DB::table('users_subscription')->delete(); Please any help?

Is the task scheduler not working or the code inside the closure (function)? I mean, are you sure this code is executed? — Anthony Aslangul
– Anthony Aslangul, Commented Aug 21, 2021 at 14:12
The task scheduler is working once i run php artisan schedule:work but the code does'nt get executed, unless i try something else like delete a table — Dipo Deen
– Dipo Deen, Commented Aug 21, 2021 at 14:18
You're trying to get user's id with session information, but it won't work in queued or scheduled jobs because session does not exist in that time. because of this probably you're getting error in this block "auth()->user()->id" — Emre Kaya
– Emre Kaya, Commented Aug 21, 2021 at 14:30
You are super right , it works when i hard code an id in there, Is there a way to combat this? — Dipo Deen
– Dipo Deen, Commented Aug 21, 2021 at 14:38

Anthony Aslangul · Accepted Answer · 2021-08-21 14:59:02Z

1

You have an exception caused by auth()->user()->id, since no user is logged when your closure is executed.

You can check the logs in storage/logs/laravel.log and if it is the case, the solution is just to avoid using any authentification mechanisms inside the scheduler.

As an alternative, you have to rethink what UserSubscription should be expired:

UserSubscription::where('status', 'approved')
->where('expiry_date', '<', now())
->update(['status' => 'updated');

No user anymore, you just expire every subscriptions where the date is before now.

Credit to Emre Kaya for finding the error.

edited Aug 21, 2021 at 14:59

answered Aug 21, 2021 at 14:35

Anthony Aslangul

3,8672 gold badges24 silver badges33 bronze badges

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

5 Comments

Anthony Aslangul Over a year ago

In deed, i updated my answer. Emre Kaya, if you write an answer i'll be glad to delete mine.

Dipo Deen Over a year ago

You're right tho,i can see the error logs...Somethin about call to update function on null which indicates that the id is'nt working. is there any solution for this ?

Anthony Aslangul Over a year ago

I updated my answer, maybe it can help you, if you have any question tell me

Dipo Deen Over a year ago

Damnn..Best answer ever, Thanks so much, it works perfectly..how come i didnt think like this..lol.Most grateful

Anthony Aslangul Over a year ago

No problem :) I see you are new on SO, don't forget to validate the answer (here and in your other questions) so other people with the same kind of error can quickly understand this is what they are looking for. Have a nice day.

Collectives™ on Stack Overflow

Laravel 8 Task scheduler

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related