search

beyond-all-reason / teiserver

Middleware server for online gaming
https://www.beyondallreason.info/
MIT License
50 stars 47 forks source link

Bugfix: Allow hosts to rename lobbies with equal priority to players. #382

Closed DeviousNull closed 1 month ago

DeviousNull commented 1 month ago

This PR should only be deployed after all SPADS instances are running the new code from https://github.com/beyond-all-reason/spads_config_bar/pull/136 !

(Otherwise, any non-updated SPADS instances will regularly set the lobby name back to the default)

Background and summary: With the new "!rename" command added to the BarManager SPADS plugin, a host may attempt to rename its lobby at the request of a player. Furthermore, BarManager no longer automatically changes lobby names (instead, it only updates the lobby's Teaser).

However, after the very first rename for a lobby (starting from initial creation), Teiserver currently forbids any further renames originating from the host. This PR removes that restriction, allowing "!rename" to function as expected.

DeviousNull commented 1 month ago

Note that I forgot to include: This fixes https://github.com/beyond-all-reason/teiserver/issues/371