monorail
/
Archipelago
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't leak info about what exists or not if player can't afford the hint (#1146)

This commit is contained in:
CaitSith2 2022-10-28 02:45:18 -07:00 committed by GitHub
parent e6c6b00109
commit 80db8a33af
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
MultiServer.py
View File

@ -1332,6 +1332,8 @@ class ClientMessageProcessor(CommonCommandProcessor):
def get_hints(self, input_text: str, for_location: bool = False) -> bool: def get_hints(self, input_text: str, for_location: bool = False) -> bool:
points_available = get_client_points(self.ctx, self.client) points_available = get_client_points(self.ctx, self.client)
cost = self.ctx.get_hint_cost(self.client.slot)
if not input_text: if not input_text:
hints = {hint.re_check(self.ctx, self.client.team) for hint in hints = {hint.re_check(self.ctx, self.client.team) for hint in
self.ctx.hints[self.client.team, self.client.slot]} self.ctx.hints[self.client.team, self.client.slot]}
@ -1386,7 +1388,6 @@ class ClientMessageProcessor(CommonCommandProcessor):
return False return False
if hints: if hints:
cost = self.ctx.get_hint_cost(self.client.slot)
new_hints = set(hints) - self.ctx.hints[self.client.team, self.client.slot] new_hints = set(hints) - self.ctx.hints[self.client.team, self.client.slot]
old_hints = set(hints) - new_hints old_hints = set(hints) - new_hints
if old_hints: if old_hints:
@ -1436,7 +1437,12 @@ class ClientMessageProcessor(CommonCommandProcessor):
return True return True
else: else:
self.output("Nothing found. Item/Location may not exist.") if points_available >= cost:
self.output("Nothing found. Item/Location may not exist.")
else:
self.output(f"You can't afford the hint. "
f"You have {points_available} points and need at least "
f"{self.ctx.get_hint_cost(self.client.slot)}.")
return False return False
@mark_raw @mark_raw