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

ResqueScheduler::removeDelayed does not work on version 1.1 #35

Open
firoxer opened this issue Dec 13, 2016 · 0 comments
Open

ResqueScheduler::removeDelayed does not work on version 1.1 #35

firoxer opened this issue Dec 13, 2016 · 0 comments

Comments

@firoxer
Copy link

firoxer commented Dec 13, 2016

ResqueScheduler::removeDelayed calls removePrefix on line 115. However, in php-resque-scheduler version 1.1 the php-resque dependency is at version 1.2, where removePrefix does not exist yet, and the removePrefix call fails silently, returning just false. I assume the silent failure is caused by dynamic calls to Redis wrappers (or Redis itself).

Upgrading to dev-master fixed the problem.

I guess this bug isn't really worth a new release, though it'd be nice if the issue was left open for other people to notice until a new tagged version of php-resque-scheduler appeared. I'm not too keen on depending on dev-master. :)

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

No branches or pull requests

1 participant