From 29ec198f81f2d0c38d4e2993c5022ad309f45e50 Mon Sep 17 00:00:00 2001 From: Ryan Barlow <7389646+ryanbarlow97@users.noreply.github.com> Date: Thu, 1 Oct 2026 11:19:33 +0000 Subject: [PATCH] fix: reword player messages in character Replace out-of-character wording in gameplay messages (permissions, containers, entities, chunks, plugin tags) with in-world phrasing. Staff commands, usage help and setup errors keep their plain wording. Co-Authored-By: Claude Opus 5.5 (1M context) --- .../tfminecraft/gunsandgadgets/manager/CraftingManager.java | 2 +- .../gunsandgadgets/manager/inventory/InventoryManager.java | 4 ++-- .../gunsandgadgets/manager/CraftingManagerTest.java | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/net/tfminecraft/gunsandgadgets/manager/CraftingManager.java b/src/main/java/net/tfminecraft/gunsandgadgets/manager/CraftingManager.java index 3bb7da7..d21ce7d 100644 --- a/src/main/java/net/tfminecraft/gunsandgadgets/manager/CraftingManager.java +++ b/src/main/java/net/tfminecraft/gunsandgadgets/manager/CraftingManager.java @@ -85,7 +85,7 @@ public void onCraft(InventoryClickEvent e) { } if(!hasInputs(player, parts)) { - player.sendMessage("§cLacking inputs"); + player.sendMessage("§cYou lack the materials for this weapon."); player.playSound(player.getLocation(), Sound.ENTITY_VILLAGER_NO, 1f, 1f); return; } diff --git a/src/main/java/net/tfminecraft/gunsandgadgets/manager/inventory/InventoryManager.java b/src/main/java/net/tfminecraft/gunsandgadgets/manager/inventory/InventoryManager.java index 0dca988..1f48147 100644 --- a/src/main/java/net/tfminecraft/gunsandgadgets/manager/inventory/InventoryManager.java +++ b/src/main/java/net/tfminecraft/gunsandgadgets/manager/inventory/InventoryManager.java @@ -281,7 +281,7 @@ public boolean hasClassConflict(Player player, Collection parts) { Set resolved = resolveClasses(parts); if (resolved != null) return false; // no conflict - player.sendMessage("§cClass conflict! These parts have incompatible class requirements:"); + player.sendMessage("§cThese parts cannot go together. They are made for different callings:"); for (GunPart part : parts) { List reqs = part.getClassRequirements(); if (!reqs.isEmpty()) { @@ -379,7 +379,7 @@ public ItemStack createOutputItem(GunType type, Collection parts, boole if (skin == null) { ItemStack unavailable = new ItemStack(Material.BARRIER); ItemMeta unavailableMeta = unavailable.getItemMeta(); - unavailableMeta.setDisplayName("§cNo compatible skin"); + unavailableMeta.setDisplayName("§cNo known design fits these parts"); unavailable.setItemMeta(unavailableMeta); return unavailable; } diff --git a/src/test/java/net/tfminecraft/gunsandgadgets/manager/CraftingManagerTest.java b/src/test/java/net/tfminecraft/gunsandgadgets/manager/CraftingManagerTest.java index 686a42d..115292f 100644 --- a/src/test/java/net/tfminecraft/gunsandgadgets/manager/CraftingManagerTest.java +++ b/src/test/java/net/tfminecraft/gunsandgadgets/manager/CraftingManagerTest.java @@ -145,7 +145,7 @@ void insufficientMaterialsRejectWithoutConsumption() { manager.onCraft(click); assertEquals(2, material.getAmount()); verify(inventory, never()).addItem(any(ItemStack.class)); - verify(player).sendMessage("§cLacking inputs"); + verify(player).sendMessage("§cYou lack the materials for this weapon."); } @Test