Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENHANCEMENT] Изменение заполнителей сообщений #30

Open
Snoop1CattZ69 opened this issue May 17, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@Snoop1CattZ69
Copy link

Изменить заполнители сообщений.

Сейчас заполнители имеют вид {0}, это не удобно при переводе, т.к приходится проверять каждое сообщение при переводе на другой язык.

В идеале всё же сделать плейсхолдеры, скорость прогрузки прокси это конечно хорошо, но это мизерные миллисекунды потерь, если всё же поменять заполнители на более понятные)

Было бы в разы удобнее тем, кто переводит сообщения на свой язык.

@Snoop1CattZ69 Snoop1CattZ69 added the enhancement New feature or request label May 17, 2022
@Snoop1CattZ69 Snoop1CattZ69 changed the title [ENHANCEMENT] [ENHANCEMENT] Изменение заполнителей сообщений May 17, 2022
@mdxd44
Copy link
Member

mdxd44 commented May 19, 2022

На скорость запуска прокси плейсхолдеры не влияют. Сейчас мы используем MessageFormat.format(), иначе, если плейсхолдеры будут не просто цифрами, нам придется вручную делать .replace("{placeholder}", value) на каждом сообщении. Возможно позже мы сделаем свою имплементацию для этого.

Гораздо лучше когда можно заменить все плейсхолдеры вызовом одного метода, нежели чем каждый раз вызывать .replace():

    ...
    method(MessageFormat.format("{0}: {1} - {2}", args[0], args[1], args[2]));
    ...

или же

    ...
    method("{arg1}: {arg2} - {arg3}".replace("{arg1}", args[0]).replace("{arg2}", args[1]).replace("{arg3}", args[2]));
    ...

@mdxd44
Copy link
Member

mdxd44 commented May 4, 2024

мы сделали😀(но пока только в https://github.com/Elytrium/LimboAuth/tree/db-and-serializer-rewrite)

Placeholders.replace(Settings.MESSAGES.message, args[0])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants