Use dataclasses clientside

common/structs.py

@@ -636,6 +636,15 @@ class TimelineEvent:
     type: TimelineEventType
 
 
+@dataclasses.dataclass(slots=True)
+class Review:
+    user: str
+    score: int
+    message: str
+    likes: int
+    timestamp: int
+
+
 @dataclasses.dataclass(slots=True)
 class Label:
     id: int
@@ -934,7 +943,7 @@ class Game:
     previews_urls      : list[str]
     downloads          : tuple[tuple[str, list[tuple[str, str]]]]
     reviews_total      : int
-    reviews            : list[dict]
+    reviews            : list[Review]
     selected           : bool = False
     image              : "imagehelper.ImageHelper" = None
     executables_valids : list[bool] = None
@@ -1110,7 +1119,9 @@ def __setattr__(self, name: str, value: typing.Any):
             "notes",
             "image_url",
             "previews_urls",
-            "downloads"
+            "downloads",
+            "reviews_total",
+            "reviews",
         ]:
             if isinstance(attr := getattr(self, name), Timestamp):
                 attr.update(value)

modules/api.py

@@ -34,6 +34,7 @@
     OldGame,
     Os,
     ProxyType,
+    Review,
     SearchResult,
     Status,
     Tag,
@@ -776,6 +777,7 @@ async def full_check(game: Game, last_changed: int):
         thread["downloads"] = tuple(thread["downloads"])
         thread["reviews_total"] = int(thread.get("reviews_total", "0"))
         thread["reviews"] = json.loads(thread.get("reviews", "[]"))
+        thread["reviews"] = [Review(**review) for review in thread["reviews"]]
 
         old_name = game.name
         old_version = game.version

modules/db.py

@@ -1,6 +1,7 @@
 import asyncio
 import configparser
 import contextlib
+import dataclasses
 import enum
 import json
 import pathlib
@@ -388,7 +389,10 @@ def sql_to_py(value: str | int | float, data_type: typing.Type):
                     value = data_type([value]) if value else data_type()
                 if args:
                     content_type = args[0]
-                    value = data_type(x for x in (content_type(x) for x in value) if x is not None)
+                    if hasattr(content_type, "__dataclass_fields__"):
+                        value = data_type(x for x in (content_type(**x) for x in value) if x is not None)
+                    else:
+                        value = data_type(x for x in (content_type(x) for x in value) if x is not None)
         case _:
             if isinstance(data_type, types.UnionType):
                 if (
@@ -492,12 +496,16 @@ def py_to_sql(value: enum.Enum | Timestamp | bool | list | tuple | typing.Any):
     elif isinstance(value, list):
         value = value.copy()
         value = [getattr(item, "value", getattr(item, "id", item)) for item in value]
+        if value and hasattr(value[0], "__dataclass_fields__"):
+            value = [dataclasses.asdict(item) for item in value]
         value = json.dumps(value)
     elif isinstance(value, tuple):
         if 3 <= len(value) <= 4 and all(type(item) in (float, int) for item in value):
             value = colors.rgba_0_1_to_hex(value)
         else:
             value = [getattr(item, "value", getattr(item, "id", item)) for item in value]
+            if value and hasattr(value[0], "__dataclass_fields__"):
+                value = [dataclasses.asdict(item) for item in value]
             value = json.dumps(value)
     return value