/sp gui, GUI Complete, need prettification

InventoryInteractEvent was not being fired, although their children do.

Prefix Menu works, `/sp gui` works with many pages.

Author: Peashooter101

README.md

@@ -11,23 +11,30 @@ A plugin aiming to allow for simple player-selected prefixes that can be used in
 > 
 > Documentation is made for existing versions, more for our sake.
 
-## Requirements
+# Requirements
 
 - Paper-based Server (ie: [PurpurMC](https://purpurmc.org/))
   - This plugin takes advantage of Mini-Message from Paper.
 - [PlaceholderAPI Plugin](https://github.com/PlaceholderAPI/PlaceholderAPI)
 
-## Features
+# Features
 
+- Does not interact with any other plugins directly.
+  - The plugin uses and hold prefixes in a placeholder, it will not affect permission groups.
+- Supports PlaceholderAPI Placeholders.
+- Supports Mini-Message formatting.
+- Supports multiple types of requirements: PERMISSION, STATISTIC, ADVANCEMENT, COMPARE_INT
 
-
-## Placeholders, Commands, and Permissions
+# Placeholders, Commands, and Permissions
 
 `%sp_prefix%`
 > Placeholder for the prefix. Use this in your chat plugin.
 > 
 > Example Usage: `%sp_prefix% %player_displayname% » {message}`
 
+`/sp gui`
+> Opens a generated GUI that organizes the prefixes by prefix-id.
+
 `/sp reset`
 > Resets your prefix.
 
@@ -37,7 +44,7 @@ A plugin aiming to allow for simple player-selected prefixes that can be used in
 `/sp reload`
 > Reloads the plugin.
 
-## Configuration
+# Configuration
 
 ```yml
 saving-type: "file"
@@ -59,14 +66,14 @@ prefix-id:
     - "compare_int %placeholder% < 1"
 ```
 
-### Configuration Settings
+## Configuration Settings
 
 | Setting        | Description                                 | Valid Values  |
 |----------------|---------------------------------------------|---------------|
 | saving-type    | What type of saving system should this use? | `PDC`, `FILE` |
 | default-prefix | Fallback prefix, supports Placeholders.     | String        |
 
-#### Saving Types
+### Saving Types
 
 These are the current implementations of the saving types...
 
@@ -181,6 +188,11 @@ Format: `compare_int <placeholder> <operator> <value>`
 >
 > <u>Example</u>: `compare_int %player_absorption% > 0` (Player has absorption value greater than 0).
 
+<u>**Unimplemented**</u>
+
+- Compare String, Case Sensitive
+- Compare String, Case Insensitive
+
 ## WIP Features
 
 - Chest GUI

src/main/java/adhdmc/simpleprefixes/gui/chest/PrefixMenu.java

@@ -26,6 +26,7 @@ public class PrefixMenu {
     public static final NamespacedKey nskPrefixId = new NamespacedKey(SimplePrefixes.getPlugin(), "prefix_id");
     public static final NamespacedKey nskUnlocked = new NamespacedKey(SimplePrefixes.getPlugin(), "prefix_unlocked");
     public static final NamespacedKey nskPrefixMenu = new NamespacedKey(SimplePrefixes.getPlugin(), "prefix_menu");
+    public static final NamespacedKey nskPage = new NamespacedKey(SimplePrefixes.getPlugin(), "prefix_page");
     private final int PER_PAGE = 45;
     private final MiniMessage mini = SimplePrefixes.getMiniMessage();
 
@@ -68,22 +69,23 @@ private List<String> getPlayerPrefixOptions(Player p) {
         return prefixes;
     }
 
+    // TODO: Everything within this section should be configurable.
     private ItemStack generatePrefixItem(Player player, Prefix prefix) {
-        ItemStack item = new ItemStack(Material.NAME_TAG);
+        boolean unlocked = RequirementUtil.getInstance().isEarnedPrefix(player, prefix);
+        ItemStack item = (unlocked) ? new ItemStack(Material.NAME_TAG) : new ItemStack(Material.BARRIER);
         ItemMeta meta = item.getItemMeta();
         assert prefix.displayName != null;
         assert prefix.prefix != null;
-        String papiDisplayName = "<reset>" + PlaceholderAPI.setPlaceholders(player, prefix.displayName);
-        String papiPrefix = "<reset>" + PlaceholderAPI.setPlaceholders(player, prefix.prefix);
-        boolean unlocked = RequirementUtil.getInstance().isEarnedPrefix(player, prefix);
-        String unlockedLore = "<reset>" + (unlocked ? "<aqua><bold>✔ UNLOCKED" : "<red><bold>✗ LOCKED"); // TODO: Make Configurable
+        String papiDisplayName = "<!i><white>" + PlaceholderAPI.setPlaceholders(player, prefix.displayName);
+        String papiPrefix = "<!i><white>" + PlaceholderAPI.setPlaceholders(player, prefix.prefix);
+        String unlockedLore = "<!i><white>" + (unlocked ? "<aqua><bold>✔ UNLOCKED" : "<red><bold>✗ LOCKED");
         meta.displayName(mini.deserialize(papiDisplayName));
         List<Component> lore = new ArrayList<>();
         lore.add(mini.deserialize(papiPrefix));
         lore.add(mini.deserialize(""));
         lore.add(mini.deserialize(unlockedLore));
         for (String line : prefix.description) {
-            String papiLine = "<reset>" + PlaceholderAPI.setPlaceholders(player, line);
+            String papiLine = "<!i><white>" + PlaceholderAPI.setPlaceholders(player, line);
             lore.add(mini.deserialize(papiLine));
         }
         meta.lore(lore);
@@ -101,6 +103,8 @@ private ItemStack generatePageArrowItem(int page, int prefixes, boolean forward)
         ItemStack item = new ItemStack(Material.ARROW);
         ItemMeta meta = item.getItemMeta();
         String displayName = forward ? "<bold>Next Page >>" : "<bold><< Prev Page";
+        int toPage = forward ? page+1 : page-1;
+        meta.getPersistentDataContainer().set(nskPage, PersistentDataType.INTEGER, toPage);
         meta.displayName(mini.deserialize(displayName));
         item.setItemMeta(meta);
         return item;

src/main/java/adhdmc/simpleprefixes/gui/chest/listener/PrefixMenuListener.java

@@ -1,24 +1,35 @@
 package adhdmc.simpleprefixes.gui.chest.listener;
 
-import adhdmc.simpleprefixes.SimplePrefixes;
 import adhdmc.simpleprefixes.gui.chest.PrefixMenu;
-import net.kyori.adventure.text.minimessage.MiniMessage;
+import adhdmc.simpleprefixes.util.PrefixUtil;
+import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
-import org.bukkit.event.inventory.InventoryInteractEvent;
+import org.bukkit.event.inventory.InventoryClickEvent;
+import org.bukkit.event.inventory.InventoryDragEvent;
 import org.bukkit.event.inventory.InventoryType;
 import org.bukkit.inventory.Inventory;
 import org.bukkit.inventory.ItemStack;
+import org.bukkit.persistence.PersistentDataContainer;
+import org.bukkit.persistence.PersistentDataType;
 
 public class PrefixMenuListener implements Listener {
 
     @EventHandler
-    public void onInventoryInteractEvent(InventoryInteractEvent event) {
+    public void onInventoryClick(InventoryClickEvent event) {
+        Inventory inventory = event.getInventory();
+        if (!isPrefixMenu(inventory)) return;
+        if (!(event.getWhoClicked() instanceof Player player)) return;
+        ItemStack item = event.getCurrentItem();
+        if (item == null) return;
         event.setCancelled(true);
-        SimplePrefixes.getPlugin().getServer().broadcast(MiniMessage.miniMessage().deserialize("<red>SERVER READ INPUT"));
-        event.getWhoClicked().sendRichMessage("<green>YOU CLICKED IT!");
-        if (!isPrefixMenu(event.getInventory())) return;
-        event.getWhoClicked().sendRichMessage("<blue>CANCELLED!");
+        evalSelection(event.getCurrentItem(), player);
+    }
+
+    @EventHandler
+    public void onInventoryDrag(InventoryDragEvent event) {
+        Inventory inventory = event.getInventory();
+        if (!isPrefixMenu(inventory)) return;
         event.setCancelled(true);
     }
 
@@ -29,4 +40,17 @@ private boolean isPrefixMenu(Inventory inv) {
         return item.getItemMeta().getPersistentDataContainer().has(PrefixMenu.nskPrefixMenu);
     }
 
+    private void evalSelection(ItemStack item, Player p) {
+        if (item == null) return;
+        Integer newPage = item.getItemMeta().getPersistentDataContainer().get(PrefixMenu.nskPage, PersistentDataType.INTEGER);
+        if (newPage != null) {
+            Inventory newMenu = PrefixMenu.getInstance().generatePrefixMenu(p, newPage);
+            p.openInventory(newMenu);
+            return;
+        }
+        PersistentDataContainer pdc = item.getItemMeta().getPersistentDataContainer();
+        String prefixId = pdc.get(PrefixMenu.nskPrefixId, PersistentDataType.STRING);
+        if (prefixId == null) return;
+        if (pdc.has(PrefixMenu.nskUnlocked)) PrefixUtil.getInstance().setPrefix(p, prefixId);
+    }
 }