gaming/lutris
gaming/lutris
1
0
Fork 0
mirror of https://github.com/lutris/lutris synced 2024-09-21 00:31:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add protection against None in steam.match_game()

Browse source 
This is confusing, but it looks like we pass None to match_game() when the Lutris API says some service game you have corresponds to a another service game from the same service you don't have

I don't understand when this would be true, but it's better not to crash over it.

Resolves #5248
This commit is contained in:
Daniel Johnson 2024-01-17 18:47:52 -05:00
parent 8054e6124b
commit d6faf75fb3
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
lutris/services/steam.py
View file

@ -111,11 +111,12 @@ class SteamService(BaseService):
def match_game(self, service_game, lutris_game):
super().match_game(service_game, lutris_game)
# Copy playtimes from Steam's data
for game in get_games(filters={"service": self.id, "service_id": service_game["appid"]}):
steam_game_playtime = json.loads(service_game["details"]).get("playtime_forever")
playtime = steam_game_playtime / 60
sql.db_update(PGA_DB, "games", {"playtime": playtime}, conditions={"id": game["id"]})
if service_game:
# Copy playtimes from Steam's data
for game in get_games(filters={"service": self.id, "service_id": service_game["appid"]}):
steam_game_playtime = json.loads(service_game["details"]).get("playtime_forever")
playtime = steam_game_playtime / 60
sql.db_update(PGA_DB, "games", {"playtime": playtime}, conditions={"id": game["id"]})
def get_installer_files(self, installer, _installer_file_id, _selected_extras):
steam_uri = "$STEAM:%s:."