yandex-music-api icon indicating copy to clipboard operation
yandex-music-api copied to clipboard

Feature request : Сделать экранирование названия трека

Open json1c opened this issue 2 years ago • 1 comments

Есть метод track.download, который скачивает трек в определенную папку Если в названии трека есть символ /, то создается еще одна папка. Выглядит это примерно так:

downloads/ym_100524465:21106180uLZRc/IC3PEAK, Kim Dracula - Червь / Worm.mp3

Предлагаю сделать экранирование названия трека для таких случаев. Если нужно могу разобраться и кинуть PR.

json1c avatar Aug 07 '22 06:08 json1c

Но ведь у track.download передается путь включая имя файла. Это нормально что если вы передали в путь /, то будет подпапка. То что у трека может оказаться / в названии это забота того кто вызывает метод. Потому что библиотека не подставляет имена треков. Это сделал разработчик использующий библиотеку. На нём и ответственность за экранирование.

MarshalX avatar Aug 07 '22 09:08 MarshalX