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

Allow to add all directories with addDirectoryToBackup: [ "*" ] #37

Merged
merged 10 commits into from
Feb 20, 2025

Conversation

HydrolienF
Copy link
Contributor

Hi thanks for this nice plugins. I was trying to save all my server files (excepts the backup folder), but I haven't found a way to add all of them without naming them 1 by 1.
I have triied to set addDirectoryToBackup: [ "." ], it does include all my server files, but inside the parent directory. So I had the world/ duplicate in the .zip (container/world/ and world/)

I would appreciate if "*" for all the root directories and files was working, so I'm trying to do it. (I will switch from draft request to pull request once I will have end my test)

Allow "*" as a path & remove excluded directories from worlds to.

  • "" in the additional directory to backup add all the server root directory. Some directory can still be excluded. This allow to save every directory except some. (The "" need to be between "" in the .yml file for Bukkit)
  • People might want to save only some world, so I moved the world saves tasks in the 2nd for loop so that excluded directory are excluded from world to.
  • getDirectoryToBackup remove duplicate directory names if there is any. It might not matter at this point, but we should avoid duplicate directories if there is some before starting zip tasks.

Let me know if it's clear enough for you or if I need to change somethings.

- "*" in the additional directory to backup add all the server root directory. Some directory can still be excluded. This allow to save every directory except some.
- People might want to save only some world, so I moved the world saves tasks in the 2nd for loop so that excluded directory are excluded from world to.
- getDirectoryToBackup remove duplicate directory names if there is any. It might not matter at this point, but we should avoid duplicate directories if there is some before starting zip tasks.
@HydrolienF
Copy link
Contributor Author

I have found few minor bugs which should be fixed now.
With that codes changes, your plugins is doing what I wanted it to do : save all server files excepts backups/

@HydrolienF HydrolienF marked this pull request as ready for review November 14, 2024 11:40
@HydrolienF
Copy link
Contributor Author

I have triied to do as few changes as possible so that every changes can be understoud. Let me know if you need more explanation about something or want any changes.

@DVDishka
Copy link
Owner

Thank you for your additions! I am very happy to see new contributors and will review and implement your changes in near future

@HydrolienF
Copy link
Contributor Author

Hi, let me know if there is thing that you would like me to change in this pr.

@DVDishka
Copy link
Owner

I apologize for keeping you waiting so long. I've been very busy last few months and couldn't find time to do updates for the Backuper, but don't worry, I haven't abandoned it, I think I'll be back to work next week. Your PR is good, you don't need to change anything.

@HydrolienF
Copy link
Contributor Author

Great, no problem. Take your time.

@DVDishka
Copy link
Owner

Can I ask you to also add changes to the README to reflect these changes?

@HydrolienF
Copy link
Contributor Author

Yes, I will add the info in the README in this pr.

@DVDishka
Copy link
Owner

Thank you for your changes, I will merge it before the next release. I hope it is very soon)

@DVDishka DVDishka merged commit e719316 into DVDishka:master Feb 20, 2025
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.

2 participants