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

Refresh of tag cache doesn't remove deleted files #500

Closed
fatg3erman opened this Issue Aug 19, 2013 · 1 comment

Comments

2 participants
@fatg3erman
Contributor

fatg3erman commented Aug 19, 2013

If I update my tag_cache file using mopidy-scan, then force a reload by calling the HTTP API at mopidy.library.refresh(), it will reload the tag cache and newly-added files will be found, but deleted files will still be there if I search for them. I have to restart mopidy for the deleted files to disappear from mopidy's search results.

The debug output in the console even says that the expected number of files have been loaded from the tag cache, but the search results contain more than that.

Mopidy 0.14.2 on OSX.

@ghost ghost assigned adamcik Oct 20, 2013

@adamcik

This comment has been minimized.

Member

adamcik commented Oct 20, 2013

Looking at the code I can see where this is happening, should be fairly easy to fix so hopefully we can squeeze it into 0.17. Thanks for the report :-)

@adamcik adamcik closed this in ecc0bae Oct 27, 2013

jodal added a commit that referenced this issue Oct 27, 2013

Merge pull request #549 from adamcik/fix/bug-500-library-refresh-does…
…-not-remove-uris

local: Delete uris in library refresh (fixes #500)

adamcik added a commit to adamcik/mopidy that referenced this issue Dec 24, 2013

local: Add local library provider back
- Re-add a local library provider that uses our new library interface
- Re-add our json library using the new interface
- Hardcode these to use each other for now
- Scanner bit is still missing, will re-add in one of the next commits
- Bypassed test for mopidy#500 for the time being

adamcik added a commit to adamcik/mopidy that referenced this issue Dec 31, 2013

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