fix typo and translate messages server-side before sending to irc(wrong formated due to translation expression included in message) #1

crabman77 · 2022-07-04T22:07:45+02:00

crabman77 commented

2022-07-04 22:07:45 +02:00

Pour l'issue nalc/nalc-server-mods#116
Traduire les messages irc mal formatés à cause de l'expression de traduction de l'engine.

typo dans 2 traductions.

Il faut ajouter "language=fr" dans le minetest.conf, même si de base ça sera en fr si ça n'y est pas, pour ceux qui ont un serveur dans un autre language ils pourront juste modifier dans le fichier de conf(un peu useless car c'est traduit que en fr en fait mais ça enlevera quand même le mauvais format).

Pour l'issue https://sys4.fr/gitea/nalc/nalc-server-mods/issues/116 Traduire les messages irc mal formatés à cause de l'expression de traduction de l'engine. + typo dans 2 traductions. Il faut ajouter "language=fr" dans le minetest.conf, même si de base ça sera en fr si ça n'y est pas, pour ceux qui ont un serveur dans un autre language ils pourront juste modifier dans le fichier de conf(un peu useless car c'est traduit que en fr en fait mais ça enlevera quand même le mauvais format).

crabman77 added 1 commit 2022-07-04 22:07:46 +02:00

8abee494e3 fix typo and translate messages server-side before sending to irc(wrong formated due to translation expression included in message)

sys4 commented

2022-07-09 14:47:38 +02:00

Je me demandais si ce ne serait pas mieux d'aller chercher dans minetest.conf la variable irc_language au lieu de language qui risque d'avoir un impact avec d'autres mods ?

crabman77 commented

2022-07-09 15:10:12 +02:00

C'est toi qui voit.
Après si y a pas la variable il se peut que le serveur se mette dans la langue de l'OS tout seul donc même sans il se peut que ça passe quand même en fr ou autre et que le serveur tourne déjà en fr.

il faudrait voir si y a un minestest.lang et tester ce que ça dit.

C'est toi qui voit. Après si y a pas la variable il se peut que le serveur se mette dans la langue de l'OS tout seul donc même sans il se peut que ça passe quand même en fr ou autre et que le serveur tourne déjà en fr. il faudrait voir si y a un minestest.lang et tester ce que ça dit.

sys4 commented

2022-07-09 15:32:33 +02:00

Ça me va, en effet je ne pense pas avoir besoin de rajouter la variable language dans le minetest.conf

sys4 added 1 commit 2022-07-09 16:14:39 +02:00

3cc9f5cb8d Merge branch 'master' into translate_irc

sys4 commented

2022-07-09 16:17:47 +02:00

oups je pensais que ça rebaserai. Pas grave, j'essaierai d'y penser pour la prochaine fois.

sys4 closed this pull request

2022-07-09 16:17:47 +02:00

crabman77 commented

2022-07-09 16:42:45 +02:00

Oui en fait en y repensant c'est peut être mal foutu le test, et il faudrait peut être plutot tester si y a irc_language ou bien passer sur os.getenv("LANG") ?

https://sys4.fr/gitea/nalc/death_messages/src/branch/master/init.lua#L46

local lang = minetest.settings:get("irc_language") or os.getenv("LANG")
if not lang or lang == "" then
	lang = "fr"
end

Oui en fait en y repensant c'est peut être mal foutu le test, et il faudrait peut être plutot tester si y a irc_language ou bien passer sur os.getenv("LANG") ? https://sys4.fr/gitea/nalc/death_messages/src/branch/master/init.lua#L46 ``` local lang = minetest.settings:get("irc_language") or os.getenv("LANG") if not lang or lang == "" then lang = "fr" end ```

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: nalc/death_messages#1