-
-
Notifications
You must be signed in to change notification settings - Fork 795
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
[Bug]: Recently Added shouldn't rely on Update but Create timestamp #2650
Comments
In my case, the workaround was relatively simple: I added a lot (350+) of covers all at once, in a short timeframe, so I could rewrite the
But of course, if I fix an old cover again, that poor old album is going to come back up the list and try to play with the young kids and everyone is going to look at him sideways and think "ok boomer" and release a barely hidden sigh that will just make everyone really uncomfortable... |
You have set ND_RECENTLYADDEDBYMODTIME to true which is not what you want from what your description reads. https://www.navidrome.org/docs/usage/configuration-options/#advanced-configuration The functionality you describe is implemented with #2553 as long as you dont Set ND_RECENTLYADDEDBYMODTIME (default is False) |
Hmm... well that setting doesn't do what I thought it did though: I thought that would set the the |
That's correct. It is used for controlling how to display the Recent Added Albums view. |
okay, so couldn't this be made in reverse, like at import time, if that setting is on, use the file mtime as the |
I think the fields should be used for what they mean: Also, making this distinction at scan time, means that if you want to change it, Navidrome needs to rescan everything again - with a full scan - which can take days sometimes for some large libraries. |
i see. so what are my options here? i guess i could remap the |
That should be the behaviour if you remove |
okay well maybe that's where my confusion comes from: I thought |
No it is not!
#1350 (fixed by #2553) was released in 0.50.0, which replaces import date with birth date, on systems that support it, as that's a sensible thing to do (use the actual file creation date instead of the import date) |
Ookay... I completely misread this! thanks for the clarification... is there a way to rescan files so that their creation date is retrofitted back into the |
Try a full rescan. If that doesn't work, there's no other option except for re-adding all your files, which currently will make you lose your playlists and favourites/ratings/play counts :/ |
i noticed to that if the song is already in the database, a full rescan will not refresh it with the file creation date. |
This issue has been automatically marked as stale because it has not had recent activity. The resources of the Navidrome team are limited, and so we are asking for your help. |
I confirm that:
Version
0.50.1
Current Behavior
Modifications to (say) an album cover brings said album back into the top of the "Recently Added" queue, even though it might be an old album.
Expected Behavior
Instead, "Recently Added" should only rely on the creation (or import?) date of the file that's being processed.
Steps To Reproduce
Expected result: the newer album before step 2 should still show up as the newest.
Actual result: modified album bumps the newer album.
Environment
How Navidrome is installed?
Docker
Configuration
Anything else?
At first I thought this was because Navidrome only held a single timestamp for albums, but after inspecting the sqlite database (after fear of losing my ratings and play counts, which I think i did?), I noticed there was a
created_at
ANDupdated_at
column, so that data is actually there, just not used correctly, I believe.But here's two albums from the database that are sorted, IMHO, backwards:
There, "Johnny Cash" gets sorted first, as more "Recently Added" than "zouz".
Now of course, one fix to this is to run an SQL query updating my entire database to wipe off the differnet
update_at
timestamps, or at least tweak the ones that are incorrect, but that seems like shoving a problem under the rug...Besides, maybe we could have a Recently Updated view as well? That would be interesting, listing the albums that have a different
created_at
andupdated_at
timestamp! :) Unfortunately, it looks like all those timestamps differ here, sometimes by only a few seconds...Code of Conduct
The text was updated successfully, but these errors were encountered: