Better update in memory cache

mobile/lib/db/ml/db.dart

@@ -1286,6 +1286,8 @@ class MLDataDB with SqlDbBase implements IMLDataDB<int> {
     ];
   }
 
+  /// WARNING: Better to use the similarly named [putFaceIdCachedForPersonOrCluster]
+  /// method from face_thumbnail_cache instead!
   Future<void> putFaceIdCachedForPersonOrCluster(
     String personOrClusterId,
     String faceID,

mobile/lib/services/machine_learning/face_ml/person/person_service.dart

@@ -443,10 +443,7 @@ class PersonService {
       data: person.data.copyWith(avatarFaceId: face.faceID),
     );
     await updatePerson(updatedPerson);
-    await faceMLDataDB.putFaceIdCachedForPersonOrCluster(
-      p.remoteID,
-      face.faceID,
-    );
+    await putFaceIdCachedForPersonOrCluster(p.remoteID, face.faceID);
     return updatedPerson;
   }
 

mobile/lib/utils/face/face_box_crop.dart

@@ -65,6 +65,17 @@ Future<String?> checkUsedFaceIDForPersonOrClusterId(
   return faceIDFromDB;
 }
 
+Future<void> putFaceIdCachedForPersonOrCluster(
+  String personOrClusterID,
+  String faceID,
+) async {
+  await MLDataDB.instance.putFaceIdCachedForPersonOrCluster(
+    personOrClusterID,
+    faceID,
+  );
+  _personOrClusterIdToCachedFaceID.put(personOrClusterID, faceID);
+}
+
 Future<void> _putCachedCropForFaceID(
   String faceID,
   Uint8List data, [
@@ -72,11 +83,7 @@ Future<void> _putCachedCropForFaceID(
 ]) async {
   _faceCropCache.put(faceID, data);
   if (personOrClusterID != null) {
-    await MLDataDB.instance.putFaceIdCachedForPersonOrCluster(
-      personOrClusterID,
-      faceID,
-    );
-    _personOrClusterIdToCachedFaceID.put(personOrClusterID, faceID);
+    await putFaceIdCachedForPersonOrCluster(personOrClusterID, faceID);
   }
 }