Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error raised when ext is None, since python cannot evaulate len(None) #208

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

jonas-t-s
Copy link
Contributor

Hello there

I found out, that in the code ext max be None, which is foreseen by the next if statement. But the statement len(ext) is evaluated before that if statement. This raises an error which aborts Moodle-DL.

I fix that in this pull request with checking if ext is not None and then evaluete the lenght of it. Otherwise I set the length to 0(which is not used but for completness I added it) .

@aaannndddyyy
Copy link

aaannndddyyy commented Feb 25, 2024

Thanks. It worked for me and fixed the issue (issue #207 )where I could not download anything with moodle-dl

@C0D3D3V C0D3D3V merged commit 7daef2a into C0D3D3V:main Mar 28, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants