DJDoubleD / QobuzDownloaderX-MOD

Downloads streams directly from Qobuz. Experimental refactoring of QobuzDownloaderX by AiiR
GNU General Public License v3.0
478 stars 16 forks source link

[BUG] Like the previous issue with a special character, but this time, the album has a single quote in it, and it confuses the program #12

Closed k-quiroz closed 1 year ago

k-quiroz commented 1 year ago

Tried to download an album with a single quote ( ' ) in its name. In this case, I tried to download hololive meets DJ'TEKINA//SOMETHING remix, and it shows an error. I think it can't find the path because of the single quote. image

DJDoubleD commented 1 year ago

The single quote isn't the problem, that's a valid character to use in a folder name.
The problem is an unlucky and unforeseen circumstance:

The default "max file name length" is set to 36, which I'm guessing was left on the default value. Trimming the album name to remove trailing spaces was done before truncating the album name to fit the selected max length. It just so happens that when you trim hololive meets DJ'TEKINA//SOMETHING remix down to 36 characters, you get hololive meets DJ'TEKINA//SOMETHING => a new path string with trailing space...

I will fix this in a next release by making sure that a trimmed down path string can never return a value with a trailing space. But a simple workaround while you wait is increasing the max file name length setting. This is probably what you want anyway to get the full album title in the path.