Skip to content
Snippets Groups Projects
Commit 0c6af1dc authored by Georg Krause's avatar Georg Krause
Browse files

Merge branch 'fix-18' into 'master'

Don't abort whole download, when one file doesn't exist

Closes #18

See merge request !12
parents 5078d71b 09855474
No related branches found
No related tags found
1 merge request!12Don't abort whole download, when one file doesn't exist
Pipeline #14683 passed
......@@ -36,10 +36,10 @@ async def get_track_download_url(id, remote, format=None):
download_url = payload["uploads"][0]["listen_url"]
except IndexError:
if remote.token:
raise click.ClickException("This file is not available for download")
raise click.ClickException("This file is not available for download: " + id)
else:
raise click.ClickException(
"This file is not available for download, try to login first"
"This file is not available for download, try to login first: " + id
)
if download_url.startswith("/"):
......@@ -73,9 +73,13 @@ async def track_download(
async with ctx.obj["remote"]:
progressbar = tqdm.tqdm(id, unit="Files")
for i in progressbar:
try:
download_url, format, track_data = await get_track_download_url(
i, ctx.obj["remote"], format=format
)
except click.ClickException as e:
logs.logger.error(e.message)
continue
logs.logger.info("Downloading from {}".format(download_url))
filename_params = utils.flatten(track_data)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment