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

fixed directory separator character #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

morfie78
Copy link

To fix issue #5 , I changed the hard-coded slash in the file path for the config.yml to use java.io.File.separator instead.

@JRoy
Copy link

JRoy commented Jul 26, 2019

Why not just replace File file = new File(getDataFolder() + File.separator + "config.yml"); with File file = new File(getDataFolder(), "config.yml");

@morfie78
Copy link
Author

I can only imagine that the original code of '\' inside the string with the file name was there because getDataFolder() wasn't returning the ending slash, hence the File.separator. Try it out locally and test. If this is the case, great. If not, I can update my pull request.

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