Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Add option for download by URL to download music to music directory, not temporary directory. #356

Closed
gittigy opened this issue Mar 12, 2023 · 4 comments
Labels

Comments

@gittigy
Copy link

gittigy commented Mar 12, 2023

Is your feature request related to a problem? Please describe.
I noticed songs that I had added by URL were often re-downloaded because they go to the temp directory and there is a temp directory size threshold to be maintained.

Describe the solution you'd like
I would like the option to download a song by URL and it be added to the music folder that uploaded music goes to. This would allow songs to be replayed without having to constantly redownload it as songs are added by URL.

Describe alternatives you've considered
I believe the size threshold can be increased but this doesn't really solve the problem.

Additional context
Botamusique is a fantastic music bot for mumble. Thank you very much for sharing such a great addition to mumble!

@wyldeTurkey
Copy link

YES. Where are the files actually being saved to? I want the files placed on the file system permanently so i can store and manage them and possibly restore them in the case of a failure. I do not want a youtube streamer. I want the youtube video's audio downloaded, labeled and stored permanently like mumble-ruby-plugin-bot. At the moment youtube streaming plugin. It should remove the word "download" from it's title.

@TerryGeng
Copy link
Collaborator

There is a quick solution: change the tmp_folder to a permanent location (I point it to my Download folder) and set tmp_folder_max_size = -1.

@gittigy
Copy link
Author

gittigy commented Apr 11, 2023

There is a quick solution: change the tmp_folder to a permanent location (I point it to my Download folder) and set tmp_folder_max_size = -1.

I tried changing the tmp_folder to my Download folder but doing so seemed to cause an error on the rescan function that is called when a file is uploaded or if the "refresh_cache_on_startup = True". I've attached a screenshot of the error.
Error_on_tmp_dir_change

I think setting "tmp_folder_max_size = -1" accomplishes the goal I'm seeking in that the temporary size threshold will never be reached, thus songs added by URL will keep being stored in the tmp directory and no songs will be deleted. Am I correct on this? If this is correct, I should be good I think.

@TerryGeng, when you changed the tmp directory, how did you avoid the errors on the refresh/rescan function? did you update something else to make this work? Thanks for your input!

@TerryGeng
Copy link
Collaborator

@gittigy Sorry for the late response. I think I have never encountered this in the past... I would guess just by changing the download path in the config, the database got confused. Could you completely delete the database and try starting fresh?

@azlux azlux closed this as completed Jun 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants