Show "You cannot apply that rune" when a rune is refused

applyRune returns false on every failed requirement without telling the
player, so using a rune the character doesn't qualify for did nothing. The
caller now sends popup 161 on refusal, matching popup 160 on success. The id
comes from the server's ErrorPopupMsg list; the popup strings aren't in the
client exe, so it isn't confirmed in IDA.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
MagicBotandClaude Opus 5.5 committed 2026-10-21 08:53:52 -04:00
1 parent 0ff7e7736c
commit 8f87778c43
1 file changed
+2 -1
@@ -266,7 +266,8 @@ public class ObjectActionMsgHandler extends AbstractClientMsgHandler {
case RUNE:
if(ApplyRuneMsg.applyRune(uuid, origin, player)) {
itemMan.consume(item);
}
} else
ErrorPopupMsg.sendErrorPopup(player, 161);
break;
default: //shouldn't be here, consume item
dispatch = Dispatch.borrow(player, msg);