[1.16.4] Config file will not update.

[metword]

Hello, I am having problems with my config file. Whenever I make changes in the file, they do not transfer over into my config variable.

 

Changes made in the config are not changing the actual config variable.

 

Config builder:

package com.metword.minecraftmod.config;

import com.electronwill.nightconfig.core.file.CommentedFileConfig;
import com.electronwill.nightconfig.core.io.WritingMode;
import com.metword.minecraftmod.ExampleMod;
import net.minecraftforge.common.ForgeConfigSpec;
import net.minecraftforge.fml.common.Mod;

import java.io.File;

@Mod.EventBusSubscriber

public class Config {
    private static final ForgeConfigSpec.Builder server_builder = new ForgeConfigSpec.Builder();
    public static final ForgeConfigSpec server_config;
    private static final ForgeConfigSpec.Builder client_builder = new ForgeConfigSpec.Builder();
    public static final ForgeConfigSpec client_config;

    static {
        TextConfig.init(server_builder, client_builder);

        server_config = server_builder.build();
        client_config = client_builder.build();
    }

    public static void loadConfig(ForgeConfigSpec config, String path) {

        ExampleMod.LOGGER.info("Loading Config: " + path);
        final CommentedFileConfig file = CommentedFileConfig.builder(new File(path)).sync().autosave().writingMode(WritingMode.REPLACE).build();
        ExampleMod.LOGGER.info("Built Config: " + path);
        file.load();
        ExampleMod.LOGGER.info("Loaded Config: " + path);
        config.setConfig(file);
    }
}

 

Config loader:

ModLoadingContext.get().registerConfig(ModConfig.Type.SERVER, Config.server_config);
ModLoadingContext.get().registerConfig(ModConfig.Type.CLIENT, Config.client_config);

Config.loadConfig(Config.client_config, FMLPaths.CONFIGDIR.get().resolve("minecraftmod.client.toml").toString());
Config.loadConfig(Config.server_config, FMLPaths.CONFIGDIR.get().resolve("minecraftmod.server.toml").toString());

 

Edited January 13, 2021 by metword

[ChampionAsh5357]

Do not force load the configuration file. The client and common config will be available after registry and before common setup while the server isn't present until the associated server loads for the first time (these are on a per-save instance).

[metword]

37 minutes ago, ChampionAsh5357 said:

Do not force load the configuration file

So I should remove the Config Loader code?

 

I'm a little confused what you mean by this.

 

Edited January 13, 2021 by metword

[ChampionAsh5357]

2 hours ago, metword said:

So I should remove the Config Loader code?

Yes, you should not even need to reference this at all.

[metword]

32 minutes ago, ChampionAsh5357 said:

Yes, you should not even need to reference this at all.

I am still having issues. The config still reverts to the default value.

[ChampionAsh5357]

3 hours ago, metword said:

I am still having issues. The config still reverts to the default value.

When and how are you trying to access the values?

[metword]

4 hours ago, ChampionAsh5357 said:

When and how are you trying to access the values?

I’m trying to use the value as a string in my main class.

[ChampionAsh5357]

And when are you trying to access the value?

[metword]

Here:

"Config variable that will not update" is the variable that I'm trying to access. It simply reverts to the default value.

 

// Do something when certain thing is said in chat

    public static final Minecraft mc = Minecraft.getInstance();

    boolean serverFound = false;

    @SubscribeEvent
    public void onChat (ClientChatReceivedEvent event)
    {
        // Logger
        LOGGER.info("serverFound is now " + serverFound);

        String message = event.getMessage().getString();
        if (message.contains("CONFIG VARIABLE THAT WILL NOT UPDATE"))
        {
            serverFound = true;

            // Logger
            LOGGER.info("serverFound has been set to true");
        }

        if ((message.contains("entered the world.") && serverFound) == true)
        {

            // Logger
            LOGGER.info("serverFound is now true");

            Minecraft.getInstance().player.sendChatMessage("/teleport 0 150 0");
            serverFound = false;

            // Logger
            LOGGER.info("serverFound has been set to false");
        }
    }

}

 

Edited January 16, 2021 by metword

[metword]

Does anyone know how to fix my problem?

Its some issue with the text edit file and the code not syncing. The text edit file for config not updating the code.

Edited January 17, 2021 by metword

[ChampionAsh5357]

2 hours ago, metword said:

Does anyone know how to fix my problem?

Patience, you can't expect everyone to get back so fast. As for the issue, it probably is that your singleton reference to the configuration is not actually linked to the spec for updating the variables. I would suggest looking at how forge sets up their configuration. However, these are all guesses as they are snippets of code and not the entire repository context.

[metword]

1 hour ago, ChampionAsh5357 said:

it probably is that your singleton reference to the configuration is not actually linked to the spec for updating the variables.

Sorry, after reading the Minecraft Forge code I am still confused at how to link these two...

[ChampionAsh5357]

Please link your repository then. I'm questioning a few things regarding that TextConfig class and context surrounding what you are using on the client and server configuration.

[metword]

https://github.com/metword/TextReaderMod

[ChampionAsh5357]

It seems your testing if the value changed using the logger, which will always be the default value when first initialized.

 

This is not to mention that the entirety of the code is reaching across sides and still trying to force load the config.

[metword]

 The logger is just to help me read more clearly what is going on in the code.

 

I am still confused how I should connect the two.

[metword]

Thank you all so much. Loading the config was my problem.

I think it is all in order now!