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

multiserver: allow `!release` as an alias for `!forfeit` (#693) 

* multiserver: allow `!release` as an alias for `!forfeit`

* create `/release` command. Add some periods to messages that print in console and point users to release

* Add a missing space on line 1135

Co-authored-by: Chris Wilson <chris@legendserver.info>
This commit is contained in:
alwaysintreble 2022-06-27 19:59:42 -05:00 committed by GitHub
parent cd9f8f3119
commit 2a0198b618
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 24 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
MultiServer.py
View File

@ -766,7 +766,7 @@ def update_checked_locations(ctx: Context, team: int, slot: int):
def forfeit_player(ctx: Context, team: int, slot: int): def forfeit_player(ctx: Context, team: int, slot: int):
"""register any locations that are in the multidata""" """register any locations that are in the multidata"""
all_locations = set(ctx.locations[slot]) all_locations = set(ctx.locations[slot])
ctx.notify_all("%s (Team #%d) has forfeited" % (ctx.player_names[(team, slot)], team + 1)) ctx.notify_all("%s (Team #%d) has released all remaining items from their world." % (ctx.player_names[(team, slot)], team + 1))
register_location_checks(ctx, team, slot, all_locations) register_location_checks(ctx, team, slot, all_locations)
update_checked_locations(ctx, team, slot) update_checked_locations(ctx, team, slot)
@ -779,7 +779,7 @@ def collect_player(ctx: Context, team: int, slot: int, is_group: bool = False):
if values[1] == slot: if values[1] == slot:
all_locations[source_slot].add(location_id) all_locations[source_slot].add(location_id)
ctx.notify_all("%s (Team #%d) has collected" % (ctx.player_names[(team, slot)], team + 1)) ctx.notify_all("%s (Team #%d) has collected their items from other worlds." % (ctx.player_names[(team, slot)], team + 1))
for source_player, location_ids in all_locations.items(): for source_player, location_ids in all_locations.items():
register_location_checks(ctx, team, source_player, location_ids, count_activity=False) register_location_checks(ctx, team, source_player, location_ids, count_activity=False)
update_checked_locations(ctx, team, source_player) update_checked_locations(ctx, team, source_player)
@ -1106,7 +1106,7 @@ class ClientMessageProcessor(CommonCommandProcessor):
return self.ctx.commandprocessor(command) return self.ctx.commandprocessor(command)
def _cmd_players(self) -> bool: def _cmd_players(self) -> bool:
"""Get information about connected and missing players""" """Get information about connected and missing players."""
if len(self.ctx.player_names) < 10: if len(self.ctx.player_names) < 10:
self.ctx.notify_all(get_players_string(self.ctx)) self.ctx.notify_all(get_players_string(self.ctx))
else: else:
@ -1118,8 +1118,12 @@ class ClientMessageProcessor(CommonCommandProcessor):
self.output(get_status_string(self.ctx, self.client.team)) self.output(get_status_string(self.ctx, self.client.team))
return True return True
def _cmd_release(self) -> bool:
"""Sends remaining items in your world to their recipients."""
return self._cmd_forfeit()
def _cmd_forfeit(self) -> bool: def _cmd_forfeit(self) -> bool:
"""Surrender and send your remaining items out to their recipients""" """Surrender and send your remaining items out to their recipients. Use release in the future."""
if self.ctx.allow_forfeits.get((self.client.team, self.client.slot), False): if self.ctx.allow_forfeits.get((self.client.team, self.client.slot), False):
forfeit_player(self.ctx, self.client.team, self.client.slot) forfeit_player(self.ctx, self.client.team, self.client.slot)
return True return True
@ -1128,7 +1132,7 @@ class ClientMessageProcessor(CommonCommandProcessor):
return True return True
elif "disabled" in self.ctx.forfeit_mode: elif "disabled" in self.ctx.forfeit_mode:
self.output( self.output(
"Sorry, client forfeiting has been disabled on this server. You can ask the server admin for a /forfeit") "Sorry, client item releasing has been disabled on this server. You can ask the server admin for a /release")
return False return False
else: # is auto or goal else: # is auto or goal
if self.ctx.client_game_state[self.client.team, self.client.slot] == ClientStatus.CLIENT_GOAL: if self.ctx.client_game_state[self.client.team, self.client.slot] == ClientStatus.CLIENT_GOAL:
@ -1136,8 +1140,8 @@ class ClientMessageProcessor(CommonCommandProcessor):
return True return True
else: else:
self.output( self.output(
"Sorry, client forfeiting requires you to have beaten the game on this server." "Sorry, client item releasing requires you to have beaten the game on this server."
" You can ask the server admin for a /forfeit") " You can ask the server admin for a /release")
return False return False
def _cmd_collect(self) -> bool: def _cmd_collect(self) -> bool:
@ -1698,43 +1702,48 @@ class ServerCommandProcessor(CommonCommandProcessor):
self.output(f"Could not find player {player_name} to collect") self.output(f"Could not find player {player_name} to collect")
return False return False
@mark_raw
def _cmd_release(self, player_name: str) -> bool:
"""Send out the remaining items from a player to their intended recipients."""
return self._cmd_forfeit(player_name)
@mark_raw @mark_raw
def _cmd_forfeit(self, player_name: str) -> bool: def _cmd_forfeit(self, player_name: str) -> bool:
"""Send out the remaining items from a player to their intended recipients""" """Send out the remaining items from a player to their intended recipients."""
seeked_player = player_name.lower() seeked_player = player_name.lower()
for (team, slot), name in self.ctx.player_names.items(): for (team, slot), name in self.ctx.player_names.items():
if name.lower() == seeked_player: if name.lower() == seeked_player:
forfeit_player(self.ctx, team, slot) forfeit_player(self.ctx, team, slot)
return True return True
self.output(f"Could not find player {player_name} to forfeit") self.output(f"Could not find player {player_name} to release")
return False return False
@mark_raw @mark_raw
def _cmd_allow_forfeit(self, player_name: str) -> bool: def _cmd_allow_forfeit(self, player_name: str) -> bool:
"""Allow the specified player to use the !forfeit command""" """Allow the specified player to use the !release command."""
seeked_player = player_name.lower() seeked_player = player_name.lower()
for (team, slot), name in self.ctx.player_names.items(): for (team, slot), name in self.ctx.player_names.items():
if name.lower() == seeked_player: if name.lower() == seeked_player:
self.ctx.allow_forfeits[(team, slot)] = True self.ctx.allow_forfeits[(team, slot)] = True
self.output(f"Player {player_name} is now allowed to use the !forfeit command at any time.") self.output(f"Player {player_name} is now allowed to use the !release command at any time.")
return True return True
self.output(f"Could not find player {player_name} to allow the !forfeit command for.") self.output(f"Could not find player {player_name} to allow the !release command for.")
return False return False
@mark_raw @mark_raw
def _cmd_forbid_forfeit(self, player_name: str) -> bool: def _cmd_forbid_forfeit(self, player_name: str) -> bool:
""""Disallow the specified player from using the !forfeit command""" """"Disallow the specified player from using the !release command."""
seeked_player = player_name.lower() seeked_player = player_name.lower()
for (team, slot), name in self.ctx.player_names.items(): for (team, slot), name in self.ctx.player_names.items():
if name.lower() == seeked_player: if name.lower() == seeked_player:
self.ctx.allow_forfeits[(team, slot)] = False self.ctx.allow_forfeits[(team, slot)] = False
self.output( self.output(
f"Player {player_name} has to follow the server restrictions on use of the !forfeit command.") f"Player {player_name} has to follow the server restrictions on use of the !release command.")
return True return True
self.output(f"Could not find player {player_name} to forbid the !forfeit command for.") self.output(f"Could not find player {player_name} to forbid the !release command for.")
return False return False
def _cmd_send_multiple(self, amount: typing.Union[int, str], player_name: str, *item_name: str) -> bool: def _cmd_send_multiple(self, amount: typing.Union[int, str], player_name: str, *item_name: str) -> bool: