Skip to content

Correction du bouton de retour qui marchait mal ds les menus paginées#1293

Merged
iambibi merged 2 commits into
ServerOpenMC:masterfrom
iambibi:fix/back-button-clear-history
May 18, 2026
Merged

Correction du bouton de retour qui marchait mal ds les menus paginées#1293
iambibi merged 2 commits into
ServerOpenMC:masterfrom
iambibi:fix/back-button-clear-history

Conversation

@iambibi
Copy link
Copy Markdown
Member

@iambibi iambibi commented May 18, 2026

#1250

Petit résumé de la PR:

É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 iambibi added this to the 2.5.0-beta-1 milestone May 18, 2026
@iambibi iambibi self-assigned this May 18, 2026
@iambibi iambibi added the ✨ Fixes Quelque chose a été patch label May 18, 2026
@PuppyTransGirl
Copy link
Copy Markdown
Member

vu que PaginatedMenu extends Menu t'as juste besoin de faire le check pour Menu

@iambibi iambibi changed the title Correction du bouton de retour qui marchait mal ds les menus pagninées Correction du bouton de retour qui marchait mal ds les menus paginées May 18, 2026
@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 18, 2026

oui c'est logique mais y'avait deja eu une histoire comme celle la, sur le meme bout de code
fin qd tu regarde le bout de code tu te dis c'est débile de check si c un paginatedmenu mais ça le detecte pas si tu fais que Menu, donc je suis parti sur la meme logique, je vais qd meme retenter

if (e.getInventory().getHolder(false) instanceof PaginatedMenu paginatedMenu) {
            paginatedMenu.onClose(e);
            Bukkit.getScheduler().runTaskLater(OMCPlugin.getInstance(), () -> {
                if (!(e.getPlayer().getOpenInventory().getTopInventory().getHolder() instanceof PaginatedMenu)) {
                if (!(e.getPlayer().getOpenInventory().getTopInventory().getHolder() instanceof PaginatedMenu) &&
                        !(e.getPlayer().getOpenInventory().getTopInventory().getHolder() instanceof Menu)) {
                    MenuLib.clearHistory(player);
                }
            }, 1L);
            return;
        }

        if (e.getInventory().getHolder(false) instanceof Menu menu) {
            menu.onClose(e);
            Bukkit.getScheduler().runTaskLater(OMCPlugin.getInstance(), () -> {
                if (!(e.getPlayer().getOpenInventory().getTopInventory().getHolder() instanceof Menu)) {
                    MenuLib.clearHistory(player);
                }
            }, 1L);
        }

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 18, 2026

mh ça passe, je suis curieux de voir si je remets tout au propre si ça marche, mais je doute pourquoi ça aurait pas marcher avant et mtn ça marcherais ptdr

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 18, 2026

au final ça revient au meme (ce qui est 100% logique) mais je suis perplexe, j'ai mis un debug lorsque les menus se ferment, tout est bien appelé, sombre

@iambibi
Copy link
Copy Markdown
Member Author

iambibi commented May 18, 2026

euh j'ai besoin de ça pour corriger le lore

@iambibi iambibi merged commit 83f1fad into ServerOpenMC:master May 18, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Fixes Quelque chose a été patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Le bouton de retour ne fonctionne pas lorsque l'on a ouvert une fiche de ville

2 participants