Skip to content

Conversation

@gab4000
Copy link
Contributor

@gab4000 gab4000 commented Nov 1, 2025

Petit résumé de la PR:

  • On peut désormais ajouter et retirer toutes les perms d'un joueur/rank en 1 fois
  • Correctif du reload du menu des permissions
  • On ne peut plus gérer un grades de même niveau/plus important que le sien

É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

Decrivez vos changements

  • Ajout des boutons dans les menus de perms
  • Retrait de closeInventory sur le menu des permissions
  • Vérification à chaque modifications si le rank du joueur y est autorisé

Copy link
Member

@iambibi iambibi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Arrete voir avec tes fourres tout (c'est vraiment pas du tout mais du tout compréhesible par les joueurs)
Met ta pr en draft si pas encore testé

Comment on lines 118 to 143
if (edit) {
map.put(52, new ItemBuilder(this, Material.RED_DYE, itemMeta -> {
itemMeta.displayName(Component.text("§cTout retirer"));
itemMeta.lore(List.of(Component.text("§7Cliquez pour retirer toutes les permissions du membre")));
}).setOnClick(inventoryClickEvent -> {
if (!edit)
MessagesManager.sendMessage(getOwner(), Component.text("§cVous n'avez pas la permission de modifier les permissions des membres"), Prefix.CITY, MessageType.ERROR, true);
else {
CityPermsCommands.removeAll(getOwner(), CacheOfflinePlayer.getOfflinePlayer(memberUUID));
new CityPermsMenu(getOwner(), memberUUID, true).open();
}
}));

map.put(53, new ItemBuilder(this, Material.GREEN_DYE, itemMeta -> {
itemMeta.displayName(Component.text("§aTout ajouter"));
itemMeta.lore(List.of(Component.text("§7Cliquez pour ajouter toutes les permissions au membre")));
}).setOnClick(inventoryClickEvent -> {
if (!edit)
MessagesManager.sendMessage(getOwner(), Component.text("§cVous n'avez pas la permission de modifier les permissions des membres"), Prefix.CITY, MessageType.ERROR, true);
else {
CityPermsCommands.addAll(getOwner(), CacheOfflinePlayer.getOfflinePlayer(memberUUID));
new CityPermsMenu(getOwner(), memberUUID, true).open();
}
}));
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fait un unique bouton (tout ajouter, tout retirer) genre clique gauche tu ajoutes tout, clique droit tu retires tt

Comment on lines 111 to 135
if (canEdit) {
map.put(52, new ItemBuilder(this, Material.RED_DYE, itemMeta -> {
itemMeta.displayName(Component.text("§cTout retirer"));
itemMeta.lore(List.of(Component.text("§7Cliquez pour retirer toutes les permissions du membre")));
}).setOnClick(inventoryClickEvent -> {
if (!canEdit)
MessagesManager.sendMessage(getOwner(), Component.text("§cVous n'avez pas la permission de modifier les permissions des grades"), Prefix.CITY, MessageType.ERROR, true);
else {
CityRankCommands.removeAllPermissions(getOwner(), newRank);
new CityRankPermsMenu(getOwner(), oldRank, newRank, true, page).open();
}
}));

map.put(53, new ItemBuilder(this, Material.GREEN_DYE, itemMeta -> {
itemMeta.displayName(Component.text("§aTout ajouter"));
itemMeta.lore(List.of(Component.text("§7Cliquez pour ajouter toutes les permissions au membre")));
}).setOnClick(inventoryClickEvent -> {
if (!canEdit)
MessagesManager.sendMessage(getOwner(), Component.text("§cVous n'avez pas la permission de modifier les permissions des grades"), Prefix.CITY, MessageType.ERROR, true);
else {
CityRankCommands.addAllPermissions(getOwner(), newRank);
new CityRankPermsMenu(getOwner(), oldRank, newRank, true, page).open();
}
}));
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

@gab4000 gab4000 marked this pull request as draft November 1, 2025 18:04
@gab4000 gab4000 changed the title Ajout du "tout ajouter/retirer" sur les permissions CHangements sur les grades Nov 1, 2025
@gab4000 gab4000 changed the title CHangements sur les grades Changements sur les grades Nov 1, 2025
@gab4000
Copy link
Contributor Author

gab4000 commented Nov 2, 2025

PR en pause pendant qq semaines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHANGE] Mettre toutes les permissions d'un coup [BUG] Grades impossible voir perms

2 participants