shaked6540 / YoutubePlaylistDownloader

A tool to download whole playlists, channels or single videos from youtube and also optionally convert them to almost any format you would like
Apache License 2.0
1.64k stars 258 forks source link

When the download path is no longer available, the program just crashes instead of showing an error. #395

Open isonerinan opened 1 week ago

isonerinan commented 1 week ago

I realized that the external hard drive that I download my videos to changed from D: to E:, and when the program tried to install my video to the old location, it just crashed instead of showing an error that the location I'm trying to access is not available.

cristianst85 commented 1 week ago

Were you using just D:\ or something like D:\Videos\?

isonerinan commented 1 week ago

Something like D:\Videos\. Changing D:\ to E:\ solves my problem but if it were to happen again, just a pop-up error would help so much.

cristianst85 commented 1 week ago

Yeah, it's unfortunate. The application tries to create the output folder if it does not exist, but in this case I guess it fails (silently?) since it's on a drive that does not exist anymore. In the near future I am gonna submit a fix that it's also going to address this issue indirectly. It will show an error message if the output folder does not exist or the user does not have write permissions. This should happen before any download is started.