Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined index: command #47

Closed
mralston opened this issue Jan 6, 2020 · 4 comments
Closed

Undefined index: command #47

mralston opened this issue Jan 6, 2020 · 4 comments

Comments

@mralston
Copy link

mralston commented Jan 6, 2020

Laravel version 5.3 & 6.9.
laravel-job-status version: 0.2.2

I see the following occurring in the Laravel log file:

[2020-01-06 13:46:02] production.ERROR: Undefined index: command  
[2020-01-06 13:46:03] production.ERROR: Undefined index: command  
[2020-01-06 13:46:15] production.ERROR: Undefined index: command  
[2020-01-06 13:46:15] production.ERROR: Undefined index: command  

This happens when the queue processes an event with a ShouldQueue listener.

I'm not certain why laravel-job-status is being invoked in this situation (neither the event nor the listener have the Trackable trait.

Nevertheless, I tracked it down to the updateJobStatus() method in LaravelJobStatusServiceProvider.php.

Here's a screenshot of the error occurring whilst debugging in PHPStorm.
Screenshot_2019-12-17_at_11 16 05

I gather that you're working on a rewrite at the moment and can see that the LaravelJobStatusServiceProvider.php code has changed significantly in GitHub from what I'm pulling down with Composer.

@imTigger
Copy link
Owner

Please test if it still happens in latest 1.1.0 version.

@mralston
Copy link
Author

I've just looked through the logs on our production server, that message hasn't appeared in there for a while now.

We're still running v0.2.2, but we updated to Laravel 6 a couple of months ago. Presumably some part of that combination fixed it.

@imTigger
Copy link
Owner

I have some systems running on Laravel 5.8 + JobStatus 0.2., and Laravel 6 + JobStatus 1.
Havn't tried Laravel 6 + JobStatus 0.2.*
Please try the new version if problem persists.

@mralston
Copy link
Author

I think it's fixed.
Off to look at what goodies v1 has in store... :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants