This PR simplifies the config code, no longer using any of the network or nbt methods that were unneeded previously. This has been tested on a solo client as well as a dedicated server, but could probably use more testing.
This solves a number of problems, including the exception that was always thrown during load-in caused by network desync, as well as any possibility of client hijacking of config values.
MessageConfigSync can probably be deleted, but I just commented it out.
This PR simplifies the config code, no longer using any of the network or nbt methods that were unneeded previously. This has been tested on a solo client as well as a dedicated server, but could probably use more testing.
This solves a number of problems, including the exception that was always thrown during load-in caused by network desync, as well as any possibility of client hijacking of config values.
MessageConfigSync can probably be deleted, but I just commented it out.