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

Recursion error, same as in #122 #153

Closed
bphermansson opened this Issue Mar 25, 2015 · 2 comments

Comments

2 participants
@bphermansson

bphermansson commented Mar 25, 2015

Hi
I get the recursion error described in #122 with 2.0.0b4. The fix in that thread seems to help.

@jodal jodal added this to the v2.0.0b5 milestone Mar 25, 2015

@jodal jodal added the bug label Mar 25, 2015

@jodal

This comment has been minimized.

Member

jodal commented Mar 25, 2015

This has recently surfaced as mopidy/mopidy-spotify#49 too.

@jodal

This comment has been minimized.

Member

jodal commented May 24, 2015

Currently I can't reproduce this. I could blindly disable the playlist_update_in_progress event in the hope of it making sure this never happens again. That doesn't feel like a very good solution, and we'll be left without that event forever.

Thus, I opt to close this issue until it resurfaces again, in the hope of ending up with a better solution then.

@jodal jodal closed this May 24, 2015

@jodal jodal removed this from the v2.0.0 milestone May 24, 2015

@jodal jodal removed the 1 - Ready label May 24, 2015

jodal added a commit that referenced this issue Jul 17, 2015

playlist: Register callbacks iff there is an event listener
Hopefully, this will remove the precondition for bug #122, #153, and #165.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment