Ajout des Traductions en anglais

[iambibi]

Petit résumé de la PR:

Ajout des traductions en anglais, fait avec IA (uniquement la génération des .properties), majoritairement voté dans le staff.

Étape nécessaire afin que la PR soit fini (si PR en draft)

• Suivre le Code de Conduite
• Enlever tous les imports non utilisés
• Bien documenter la feature
• Fournir un profileur (si besoin/demandé par un admin)
• Avoir une milestone associée à la PR
• Valider tout les checks
• Tester et valider la feature/changement

• Les Issues corrigée(s) en commun :

Decrivez vos changements

[iambibi]

Je mets en draft, y'a des bugs, certains utilisent bcp translation string or ça prends le fallback donc pas la langue anglaise.

[iambibi]

Y'a juste les translations des notations (translationString) que j'arrive pas a changer. Mais le principal est traduit

[iambibi]

cette PR passera surement dans la 2.5.0-beta-1 car y'a beaucoup de changements qui peuvent casser les systemes (dans les .yml, dans la db, ...)

[iambibi]

oyéoyé, faudrait review qd meme, je sais que je suis le principal dev mais j'aime bien aussi les reviews de mon code..., on peut tjr s'améliorer... (d'ailleurs j'ai remarqué que j'utilise les bossbar de bukkit qui utilisent que un string comme nom, faudrait que je change ça)

[gtolontop]

Coucouuuu comme personne a fais je me permet 🤓☝️

[gtolontop]

Les champs persistés camp1 et camp2 ont été renommés en keyCamp1 et keyCamp2.

mais je crois que l’initialisation utilise uniquement TableUtils.createTableIfNotExists, qui ne modifie pas une table existante. et la documentation ORMLite confirme que cette méthode crée seulement la table si elle n’existe

Sur un serveur existant, le DAO cherchera donc keyCamp1/keyCamp2 alors que la DB contient encore camp1/camp2. L’appel à queryForFirst() risque de niquer l'init du pl

je pense qu'il faut soit

• conserver columnName = "camp1" et columnName = "camp2" ;
• soit ajouter une migration SQL versionnée qui renomme/crée les colonnes et convertit les valeurs existantes en clés de traduction

[gtolontop]

Les surcharges de InputUtils.pluralize prennent maintenant un Component.

sauf que les tests il appellent toujours l’ancienne API avec des String :

• InputUtilsTest.java:69
• InputUtilsTest.java:75
• InputUtilsTest.java:81
• InputUtilsTest.java:87-L88

donc le job Test échoue avec 5 erreurs no suitable method found for pluralize(String, ...).

[iambibi]

Les champs persistés camp1 et camp2 ont été renommés en keyCamp1 et keyCamp2.

mais je crois que l’initialisation utilise uniquement TableUtils.createTableIfNotExists, qui ne modifie pas une table existante. et la documentation ORMLite confirme que cette méthode crée seulement la table si elle n’existe

Sur un serveur existant, le DAO cherchera donc keyCamp1/keyCamp2 alors que la DB contient encore camp1/camp2. L’appel à queryForFirst() risque de niquer l'init du pl

je pense qu'il faut soit

• conserver columnName = "camp1" et columnName = "camp2" ;
• soit ajouter une migration SQL versionnée qui renomme/crée les colonnes et convertit les valeurs existantes en clés de traduction

Plop le tag Change DB a été mis et oui ct pour cela, non je compte pas faire de migration surtout parce que :

• les bases de données vont être reset en prod
• les bugs sur les serveurs de dev, vont arriver, faudra juste que les développeurs delete toute leur db, pas de quoi s'inquiéter

[gtolontop]

J'ai noter plein d'autre truc je sais pas si tu veux ?

[iambibi]

Les surcharges de InputUtils.pluralize prennent maintenant un Component.

sauf que les tests il appellent toujours l’ancienne API avec des String :

• InputUtilsTest.java:69
• InputUtilsTest.java:75
• InputUtilsTest.java:81
• InputUtilsTest.java:87-L88

donc le job Test échoue avec 5 erreurs no suitable method found for pluralize(String, ...).

Je verrais cela, même si actuellement le workflow de test marche pas 👍
Mais je suis d'accord sur le fait de rajouter des bugs supplémentaires au test ne sert a pas grand chose

[iambibi]

J'ai noter plein d'autre truc je sais pas si tu veux ?

Oui par contre faut faire une review et non pas des commentaires pls.
(Tu vas dans la catégorie code, et tu peux empiler des commentaires associé au lignes de code, c'est ça une review)

[gtolontop]

Il manque des trad
Exemples

• en_US/mailboxes.properties:2 : feature.mailboxes.letter=lettre
• en_GB/city.properties:486 : Peace/Guerre
• en_GB/city.properties:1286 : The Ruée Minière event...
• en_GB/milestones.properties:82 : Create/Rejoindre une city
• en_GB/quests.properties:35 : Miner {target} stone blocks
• en_GB/quests.properties:63 : Miner {target} Aywenite{s}
• en_GB/tpa.properties:17 : /tpaccept <joueur>
• en_GB/tpa.properties:25 : /tpadeny <joueur>

Les mêmes erreurs se retrouvent dans en_US. Jpense l'AI qui a fais les trad à c'est bavé dessus lv

Merci l'IA pour avoir détecté les fautes de l'IA dans quel monde on part purée

Avec tous le respect tu dis ça mais ta mm pas relu alors que vrm on scroll un peu il manque full trad

[iambibi]

Je connais je savais juste pas qu'on pouvais sans être Membre du projet xd

Bref arrête toi là je pourrais généré ma review seul

[gtolontop]

Je connais je savais juste pas qu'on pouvais sans être Membre du projet xd

Bref arrête toi là je pourrais généré ma review seul

C'est toi tu as demandé une review mdrrrr

[gtolontop]

pitier même

git diff --check

relève des soucis EXEMPLE

• en_GB/city.properties:977-L979
• en_US/city.properties:977-L979
• en_GB/displays.properties:10
• en_US/displays.properties:10

[iambibi]

Oui je T'ES demandé une review, pas d'une personne tiers. Honnêtement c'est surtout ça qui m'énerve au sujet de l'IA, c'est l'humanisme de ce machin. Les features sont de moins en moins originale, ça perds des côtes créatives et ça fera toujours la même chose car elle sait que reprendre ce qu'elle connait déjà. Et là dans ton cas, on trouve même pas ta personnalité derrière tes messages, on perd du contact, de la communication.
Je suis tout de même reconnaissant que tu dépenses tes tokens pour ça, mais la prochaine fois donne moi la réponse du prompt 🫪.

[iambibi]

bonne partie des review faites, juste pour les problemes de clé manquantes je le ferais lorsque y'aura les features profil et dim opener

[gtolontop]

PluginV2/src/main/resources/translations/en_US/mailboxes.properties

Line 2 in d7420dd

feature.mailboxes.letter=lettre

y'a cette clé stv jcrois y'en a aucune autre

[iambibi]

e partie des review faites, juste pour les problemes de clé manquantes je le ferais lorsque y'aura les features profil et dim opener

j'ai pas encore fait le problemes des clés manquantes

[gab4000]

Bon j'ai fait une """petite""" review mais 7000 lignes...