Correctif des lettres qui n'étaient pas récupérable à certains moments

[gtolontop]

Corrige le bug où les items de la mailbox disparaissaient lors de l'acceptation d'une lettre.

Problème

Dans LetterMenu.accept(), les items étaient récupérés après deleteLetter(), qui supprime la lettre du cache en mémoire. MailboxManager.getById() retournait alors null dans le ClaimLetterEvent, et letter.getCachedItems() pouvait échouer silencieusement — les items étaient perdus sans message d'erreur.

Le problème se manifestait principalement quand l'expéditeur était déconnecté et que la lettre avait été envoyée avant un redémarrage serveur.

Correction

• Extraction d'une méthode resolveItems() qui cache les items désérialisés dans un champ resolvedItems, évitant la double désérialisation entre getContent() et accept()
• Appel de resolveItems() avant deleteLetter() pour garantir que les items sont disponibles
• Les items sont donnés au joueur avant le fire du ClaimLetterEvent
• ClaimLetterEvent reçoit directement l'objet letter au lieu de getById() post-suppression

Fixes #1167, fixes #1084

[AxenoDev]

Pourquoi tu modifie la class OMCPlugin.java ?

[gtolontop]

Il manque des accolade c'est écrit :)

[iambibi]

on peut avoir des reproductions in game? (en sachant que ça touche une feature majeure, je pref avoir des screens)

[iambibi]

@gtolontop merci de répondre sous délai de 3 semaines
sinon on prendra en relai ton travail

[gtolontop]

#1220 :)

[gtolontop]

sachant que cest dja tous prêt jvois pas trop sque tu vas reprendre mais az

[PuppyTransGirl]

on peut avoir des reproductions in game? (en sachant que ça touche une feature majeure, je pref avoir des screens)

il parle de ça

[gtolontop]

je sais

[iambibi]

je sais

Bah alors ? 🤡

[iambibi]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.

Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

[gtolontop]

je sais

Bah alors ? 🤡

💀💀💀💀

[gtolontop]

je sais

Bah alors ? 🤡

Bah alors 🥴🥴🥴🥴😬😬😬😬😬👿👿👿👿🥶🥶🥶🥶🤯🤯🤯🤯🤯😋😋😋😋😋

[gtolontop]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.

Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

cool

[gtolontop]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.

Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

c'est marrant jpensais que t'avais lu l'autre pr ta pas compris quoi dans le msg sur l'autre pr ?

[gtolontop]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.

Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

le #1220 (comment) dja te pose soucis tu veux de l'aide ou qlq chose ?

[iambibi]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.
Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

c'est marrant jpensais que t'avais lu l'autre pr ta pas compris quoi dans le msg sur l'autre pr ?

mb tu avais spam tellement de messages que je l'avais pas vu ça marche ^^

[iambibi]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.
Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

le #1220 (comment) dja te pose soucis tu veux de l'aide ou qlq chose ?

a vrai dire pas besoin de l'aide de qlq qui teste rien ptdr, l'IA aide surement bien a corriger les choses mais pas a lancer un serveur et se connecter ptdr

[gtolontop]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.
Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

c'est marrant jpensais que t'avais lu l'autre pr ta pas compris quoi dans le msg sur l'autre pr ?

mb tu avais spam tellement de messages que je l'avais pas vu ça marche ^^

hoo bichette c'est vrai j'ai tlm spam sur l'autre pr c'est vrai que une seule ligne a lire sur le dernier msg de la pr c'est tellement compliqué pauvre chou, tu veux un tuto?

[gtolontop]

Donc c'est toujours valable hein comme dit toutouchien on attends une démonstration in game (video). Sinon on reprendra le relais. C'est valable pour l'autre PR.
Ce n'est pas tout de coder et de faire des PR. Fin bon je vois pas comment tu testes des fixes sans les tester in game. Est ce que tu as un serveur local même ? La théorie c'est bien mais ça va avec la pratique aussi...

le #1220 (comment) dja te pose soucis tu veux de l'aide ou qlq chose ?

a vrai dire pas besoin de l'aide de qlq qui teste rien ptdr, l'IA aide surement bien a corriger les choses mais pas a lancer un serveur et se connecter ptdr

tlmmm cold 🥶🥶🥶🥶🥶

[gtolontop]

tes tlm stylé mec stp apprend moi