JoHoop / JoHoop/studip-sync-files

Empty Course throws error

Open
#5 0 comments 0 reactions 0 assignees View on GitHub
bug help wanted
Dominant language
Python
Stars
12
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Downloading a course which has no files in it throws the error `raise BadZipFile("File is not a zip file")`.
This should be catched and just display the information, rather than displaying an error. This might get confusing when downloading multiple courses at a time to determine which course is throwing the error.

This is **not** breaking the script, but is confusing and annoying.

Steps to reproduce:
1) Add an Course with no files to the course list

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.