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.

Sign up for GitHub

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

Jump to bottom

nonexistent playlist item IDs can be 'moved' in a playlist #341

Open
goto-bus-stop opened this issue Aug 10, 2018 · 0 comments
Open

nonexistent playlist item IDs can be 'moved' in a playlist #341

goto-bus-stop opened this issue Aug 10, 2018 · 0 comments
Labels
bug

Comments

@goto-bus-stop
Copy link
Member

One of two things happened:

  • A client bug (most likely) caused the same song to show twice in a filtered view; I deleted one and then moved the other to the top. The second action added the playlist item ID to the top of the playlist even though it was deleted before; or
  • I had the same playlist item ID in the playlist twice, and deleting one caused the playlist item model to be deleted too.

In the first case, we should make sure that playlist items exist before using them in move operations.

In the second case, we should check that a playlist item ID is not in any playlists before deleting the model.
@goto-bus-stop goto-bus-stop transferred this issue from u-wave/http-api Aug 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@goto-bus-stop