Skip to content

Commit ed0940b

Browse files
committed
Merge pull request #68 from d-Rickyy-b/hotfix-NoneUser
FIX: current_player was None in some cases
2 parents 0bcb9f2 + badf109 commit ed0940b

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

game/blackJackGame.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -286,14 +286,18 @@ def analyze_message(self, update):
286286
self.start_game()
287287
if self.game_running:
288288
if command.startswith(translate("oneMore", self.lang_id)):
289-
current_player = self.players[self.current_player]
290-
if self.current_player >= 0 and user_id == current_player.user_id:
289+
if self.current_player is None or self.current_player < 0:
290+
return
291+
292+
if user_id == self.players[self.current_player].user_id:
291293
self.give_player_one()
292294
else:
293295
self.send_message(self.chat_id, translate("notYourTurn", self.lang_id).format(first_name), game_id=self.__game_id)
294296
elif command.startswith(translate("noMore", self.lang_id)):
295-
current_player = self.players[self.current_player]
296-
if self.current_player >= 0 and user_id == current_player.user_id:
297+
if self.current_player is None or self.current_player < 0:
298+
return
299+
300+
if user_id == self.players[self.current_player].user_id:
297301
self.logger.debug("User doesn't want another card")
298302
self.next_player()
299303
elif command.startswith(translate("stopCmd", self.lang_id)):

0 commit comments

Comments
 (0)