Often times sortAllPlayers() is used when the sorting isn't necessary at all and the intention is rather to just get the player IDs.
Especially true in playerPlacementRandom() where they are automatically sorted again.
therefore split the two functionalities.
Note that this only changes the playerPlacementRandom(sortAllPlayers()) combination, but there are probably other places as well, where sortAllPlayers() could be replaced by getPlayerIDs() but I don't want to check in each map if the sorting is actually intended or not. That may be done in the future.