Fixes being able to vote for maps which are outside their configured population range (#64619)

* fixes map voting?

* this is worth checking as well

* additional logging
This commit is contained in:
MrMelbert
2022-02-03 00:43:44 -05:00
committed by GitHub
parent 9c24fec4dd
commit ba7c0fab92
2 changed files with 11 additions and 0 deletions
+7
View File
@@ -388,6 +388,13 @@ GLOBAL_LIST_EMPTY(the_station_areas)
message_admins("Failed to set new map with next_map.json for [VM.map_name]! Using default as backup!")
return
if (VM.config_min_users > 0 && GLOB.clients.len < VM.config_min_users)
message_admins("[VM.map_name] was chosen for the next map, despite there being less current players than its set minimum population range!")
log_game("[VM.map_name] was chosen for the next map, despite there being less current players than its set minimum population range!")
if (VM.config_max_users > 0 && GLOB.clients.len > VM.config_max_users)
message_admins("[VM.map_name] was chosen for the next map, despite there being more current players than its set maximum population range!")
log_game("[VM.map_name] was chosen for the next map, despite there being more current players than its set maximum population range!")
next_map_config = VM
return TRUE
+4
View File
@@ -183,6 +183,10 @@ SUBSYSTEM_DEF(vote)
var/datum/map_config/VM = config.maplist[map]
if(!VM.votable || (VM.map_name in SSpersistence.blocked_maps))
continue
if (VM.config_min_users > 0 && GLOB.clients.len < VM.config_min_users)
continue
if (VM.config_max_users > 0 && GLOB.clients.len > VM.config_max_users)
continue
maps += VM.map_name
shuffle_inplace(maps)
for(var/valid_map in maps)