Skip to content

fix(messages): synchronize bundled language files - #25

Merged
WieszczY85 merged 1 commit into
mainfrom
syntaxdevteam-improvements/dodaj-brakujace-wiadomosci-w-pluginie
Oct 1, 2026
Merged

WieszczY85 merged 1 commit into
mainfrom
syntaxdevteam-improvements/dodaj-brakujace-wiadomosci-w-pluginie

Conversation

@WieszczY85

Copy link
Copy Markdown
Member

Motivation

  • Brakujące wiadomości wynikały z tego, że podczas aktualizacji pluginu synchronizowany był tylko aktywny plik językowy, więc inne dostarczone pliki (np. pl lub en) nie otrzymywały dopisanych nowych kluczy i powodowały zwracanie Message not found!.

Description

  • Dodano verifyAndUpdateLanguageFiles() do ConfigHandler, które iteruje po dostarczonych plikach językowych (pl, en) i uzupełnia tylko brakujące klucze, zachowując zmiany administratora.
  • Wydzielono rekurencyjną funkcję synchronizeSections(...) zwracającą Boolean i używaną zarówno do synchronizacji config.yml, jak i plików językowych.
  • Wywoływanie verifyAndUpdateLanguageFiles() dodano do PluginInitializer.setupConfig() przed inicjalizacją MessageHandler, dzięki czemu nowe komunikaty są dostępne od razu po starcie.
  • Poprawiono bezpieczne wczytywanie domyślnych zasobów przy pomocy reader.use { ... } oraz dodano listę bundledLanguages = listOf("pl", "en").

Testing

  • Uruchomiono jednostkowe i integracyjne testy Gradle poleceniem ./gradlew test --console=plain i wszystkie zakończyły się powodzeniem (BUILD SUCCESSFUL).
  • Wykonano git diff --check oraz sprawdzono status repozytorium po zmianach; nie wykryto problemów syntaktycznych ani nierozwiązanych konfliktów.
  • Zmiany zakomitowano (fix(messages): synchronize bundled language files).

Codex Task

@WieszczY85
WieszczY85 merged commit 83d6a6c into main Oct 1, 2026
2 checks passed
@WieszczY85
WieszczY85 deleted the syntaxdevteam-improvements/dodaj-brakujace-wiadomosci-w-pluginie branch October 1, 2026 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant