mirror of
https://github.com/RandomNinjaAtk/arr-scripts.git
synced 2026-03-02 22:57:35 -05:00
[BUG] - Lidarr - Deezer truncates long album titles #89
Labels
No labels
Needs Triage
Not Reproducible
Upstream Issue
User Error
bug
documentation
enhancement
good first issue
help wanted
invalid
lidarr
lidarr
question
radarr
readarr
sabnzbd
sonarr
synology (host)
unraid (host)
waiting for logs
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/arr-scripts#89
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Bassetts on GitHub (Jan 9, 2024).
Application
Lidarr
Host platform
Ubuntu 22.04.1
Script
Audio.service.bash
Script Version
2.24
Describe the bug
It seems that Deezer truncates album titles to a maximum of 130 characters. I can't find any documentation of this and have only found it through trial and error. This means that if you have an album with a title longer than this then both the artist album search and fuzzy search will fail to find a result.
I have managed to work around this with a fairly quick hack which truncates the album title to 130 characters before cleaning it. This means a result is found when doing the artist album search.
Note: the album downloaded after my quick and dirty hack but I got a
System.IO.PathTooLongException: The specified file name or path is too long, or a component of the specified path is too long.exception when trying to import the tracks. I am not sure if this is another bug.MusicBrainz entry for the album
Deezer link for the album
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The album is found and downloaded
Logs/Screenshots
@RandomNinjaAtk commented on GitHub (Jan 9, 2024):
I welcome a PR for the change. Sounds like we may just need to truncate the album folder name. If I had to guess, because the folder name is the entire album name, its too long for lidarr to handle, hence the system io error.
If you end up writing a PR, please also apply it to Tidal, because I'm sure it's also affected by this issue.
@Bassetts commented on GitHub (Jan 9, 2024):
I think there are probably two PRs here. One to fix the issue with Deezer truncating the album title so the search fails, and another to fix the filename path being too long.
I am happy to work on both of those when I get some time. My bash isn't great so it would be good to get some practice in 😄
@RandomNinjaAtk commented on GitHub (Jan 11, 2024):
@Bassetts
The changes I just pushed may solve it... I based the changes on what I saw in your fork... However, I have not fully tested these changes... It should work....
Edit:
Let me know if this resolves the issue...