check if username is already added to the game
This commit is contained in:
		
							parent
							
								
									39377d8c9b
								
							
						
					
					
						commit
						824190348d
					
				@ -56,7 +56,14 @@ class Game:
 | 
				
			|||||||
        self.members.append(self.owner)
 | 
					        self.members.append(self.owner)
 | 
				
			||||||
        return self.owner
 | 
					        return self.owner
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    def get_member(self, username):
 | 
				
			||||||
 | 
					        for member in self.members:
 | 
				
			||||||
 | 
					            if member.username == username:
 | 
				
			||||||
 | 
					                return member
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def add_member(self, username):
 | 
					    def add_member(self, username):
 | 
				
			||||||
 | 
					        if self.get_member(username):
 | 
				
			||||||
 | 
					            return None
 | 
				
			||||||
        member = Member(username)
 | 
					        member = Member(username)
 | 
				
			||||||
        self.members.append(member)
 | 
					        self.members.append(member)
 | 
				
			||||||
        return member
 | 
					        return member
 | 
				
			||||||
 | 
				
			|||||||
@ -39,8 +39,8 @@ def join_game():
 | 
				
			|||||||
    if game == None:
 | 
					    if game == None:
 | 
				
			||||||
        return {"error": 1, "msg": "game does not exist"}
 | 
					        return {"error": 1, "msg": "game does not exist"}
 | 
				
			||||||
    
 | 
					    
 | 
				
			||||||
 | 
					    if not game.add_member(username):
 | 
				
			||||||
    game.add_member(username)
 | 
					        return {"error": 1, "msg": f"Username '{username}' already used in game {game.game_id}"}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    flask.session["game_id"] = game.game_id
 | 
					    flask.session["game_id"] = game.game_id
 | 
				
			||||||
    flask.session["is_owner"] = False
 | 
					    flask.session["is_owner"] = False
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user