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

[Bug] [Critical] Selecting existing Directory as Download Folder deletes existing data #891

Open
radry opened this issue Feb 25, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@radry
Copy link

radry commented Feb 25, 2024

Device information

  • Suwayomi-Server version: v1.0.0-r1498-windows-x64
  • Server Operating System: Win 10
  • Server JVM version: bundled with server
  • Client Operating System: Win 10

Steps to reproduce

  1. Launch Server .bat
  2. Set an existing Directory as Download Folder in Launcher
  3. Start Server with WebUI
  4. Download a Manga (to be sure it writes into the set directory, maybe this caused the deletion of all existing content)

Expected behavior

Existing Data is not deleted

Actual behavior

Everything in the existing Directory is deleted without warning and no way to recover it

Other details

This is a critical bug! Data was lost!

@radry radry added the bug Something isn't working label Feb 25, 2024
@TyrantRC
Copy link

TyrantRC commented Feb 25, 2024

I'm not a dev but I also noticed something funky when I was trying to set custom folders in the "Suwayomi-Server Launcher". If I manually input the name of a new folder in the "path box", every time I type a new character, the launcher will create a new folder with the name of the existing value at that moment.

eg: If I want to change my download folder
from "E:\Images\Manga\.Suwayomi"
to "E:\Images\Manga\.Suwayomi\abcd",
the launcher will create 4 folders like this:

and it leaves the original folder alone. This happens to the "Root Directory" as well.

So I might suggest checking thoroughly if your original folder is not hidden in an alternative path route, and also deleting cache for the suwayomi webclient (ctrl+f5 in your browser while in the suwayomi tab should suffice).

Even then, this is something that should be dealt with, because last time I also had to manually remove like 20 folders because I was messing with directories from that launcher. For next time, I suggest backing up your folders and using the server.conf file instead of the java launcher to set new routes for directories. Copying the exact path into the box also seems to work, if you deviate it brings out these problems it seems.

@radry
Copy link
Author

radry commented Feb 25, 2024

If I manually input the name of a new folder in the "path box", every time I type a new character, the launcher will create a new folder with the name of the existing value at that moment.

You are right. That happened to me too, but it's not critical, just annoying.

The issue however is that the existing content was delete for me. I already had a directory G:\Manga with existing folders and files inside. I set this directory in the Launcher and then all the existing content was gone. I don't know if it happened when it launched or when it downloaded a manga for the first time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants