Sponge can't find/create file

[Simon_Flash]

I wrote a bit a day or two ago about how the AssetManager works, so I’m going to redirect the general idea to that post rather than reiterate it here.

Can somebody please explain AssetManager for config files?

I don’t use many of those configuration options for precisely that reason; I find I’m able to do what I need to easier on my own and it ends up clearer as well. Here’s how you’d go about setting up an asset: PluginContainer container; //Should be stored in your @Plugin class Path directory; //Easiest to inject this with @ConfigDir Path path = directory.resolve("config.conf"); if (Files.notExists(path)) { Asset asset = container.getAsset("default-config.conf").get(); asset.copyToFile(pa…

And if you’d like to see my ramble about how it works internally, that one is here.

Can somebody please explain AssetManager for config files?

Okay, so there are a couple questions here. First, you can inject your PluginContainer as you’ve shown just fine (if that’s within your @Plugin class), but you don’t have to. You can get the container for any plugin using Sponge.getPluginManager.getPlugin(id) (which returns an Optional<PluginContainer>, which is why it’s recommended to have in injected for you. What I personally do is inject it into the constructor of my @Plugin class (like this) - so you have a couple of options to work with. …

Relating to your specific issue, the logs point to Files#createFile() as the issue. There are a couple of reasons why this might fail, but the one that should stand out is IOException - if an I/O error occurs or the parent directory does not exist.

This is indeed the case - the parent directory does not exist because the file is pointing to /experienceforclaims/config.conf. This is because you did not create your Path correctly in line 72 (25 in the paste) as Path#getFileName only returns the last element of the path, not all of them. You should be using Path#resolve instead.

Couple other notes on issues I saw or general recommendations:

• You don’t have to store the Optional<Asset> (just get it when you need it, which should be once anyways). I’d also store the loader as a HoconConfigurationLoader directly, but that’s a personal choice.
• In line 72 (30 in the paste), you make a call to Asset#copyToFile which internally uses Files#copy. This will result in a FileAlreadyExistsException because it’s already been created - see the first post I linked for two possibilities on how to this instead.
• Many of your exceptions are marked as TODO (which is fine), but note that they will also stack up quickly - if one fails, then the ones after it will fail and result in fields not be initialized, which then cause a NullPointerException. You should make sure that your exceptions are grouped so these subsequent methods are only being called if the config has been loaded successfully.
• Instead of using System.out.print() (which should be println() anyways), you should use the Logger for your plugin.
• Your loading message shouldn’t be determined by the config; it should be a standard message that provides meaningful information. The same holds true for any log message in your plugin - as nice as translations are, they should be standardized so you know exactly where they’re coming from. I’d expect few people to touch this anyways.

If you have any followup questions, feel free to let me know. Welcome to Sponge!