oddlama / vane

Immersive and lore friendly enhancements for vanilla Minecraft
MIT License
276 stars 34 forks source link

lang-fr-fr.yml: expected version 3, but got 4 #114

Closed karthanistyr closed 2 years ago

karthanistyr commented 2 years ago

On a completely fresh install of paper 1.18.1 and vane 1.6.7

[14:20:05 INFO]: [vane-bedtime] Enabling vane-bedtime v1.6.7
[14:20:05 ERROR]: [vane-bedtime] lang-fr-fr.yml: expected version 3, but got 4
[14:20:05 ERROR]: [vane-bedtime] This language file is for a future version of vane-bedtime.
[14:20:05 ERROR]: [vane-bedtime] Please use the correct file for this version, or use an officially
[14:20:05 ERROR]: [vane-bedtime] supported language file.
[14:20:05 ERROR]: [vane-bedtime] Invalid localization file. Shutting down.

Vane core is set for lang: "fr-fr" and vane bedtime to lang: "inherit" Maybe I made a mistake updating the version on that file ?

oddlama commented 2 years ago

Would probably be my mistake though, as I have to make sure all version numbers are in sync. Can you work around the issue by just decreasing the version to 3 for that file?

I'll definitely leave this issue open till next release so I remember to re-check the language files. Thanks for reporting!

karthanistyr commented 2 years ago

I guess it's me yes: https://github.com/oddlama/vane/commit/3d6b412316f6674c192ee6078c2a5b93e325c041

Is there a workaround for vane overwriting the file everytime I restart the server ? I edit the file down to version: 3 but it gets overwritten back to 4 when the server restarts

oddlama commented 2 years ago

Ah damn. Unfortunately there isn't. The language files are not intended to be modified by the users so they will be overwritten if a new version is shipped in the jar file. If you manage to compile the plugin you could simply fix it for your version until I have the time to make a fix. Otherwise, users are encouraged to copy files to e.g. fr-fr-custom.yml to make changes to them, so you could also just do that. Copy the file, change the version, edit the language field in the config.yml. Sorry for the inconvenience.

I notice that this whole behavior isn't ideal at the moment. I guess I should maybe overhaul the language file system a little.

Notes to self:

karthanistyr commented 2 years ago

Your proposed workaround is the better solution. My own laziness had me to switch core to en; some messages are in english but most of the mod's contents are localised client-side so it's not so bad while this gets fixed eventually.

Thanks!

ryantheleach commented 2 years ago

Why are static language files even written to the plugins directory if they are not to be changed? (As a reference maybe?)

Yep, as a reference. I'd recommend naming them with .example or something.

oddlama commented 2 years ago

Yeah good idea. This is now tracked by #118

TBG-FR commented 1 month ago

Got the same issue today with the latest plugin version...

IMO that issue should not make the server stop, that's quite annoying...

In fact I think that something is broken, because the language doesn't change even after putting correct values in each plugin config...