TerribleDev/ente
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[mob][photos] Better log and handle clusterperson edge case

Browse Source
This commit is contained in:
laurenspriem
2025-01-08 13:47:52 +01:00
parent 8fed95f3dc
commit 2d34e9e97c
1 changed files with 18 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
mobile/lib/services/search_service.dart
View File

@@ -902,10 +902,24 @@ class SearchService {
final String clusterName = clusterId;
if (clusterIDToPersonID[clusterId] != null) {
// This should not happen, means a clusterID is assigned to a personID of a person that no longer exists
_logger.severe(
"`getAllFace`: Cluster $clusterId should not have person id ${clusterIDToPersonID[clusterId]}",
);
final String personID = clusterIDToPersonID[clusterId]!;
final PersonEntity? p = personIdToPerson[personID];
if (p != null) {
// This should not be possible since it should be handled in the above loop, logging just in case
_logger.severe(
"`getAllFace`: Something unexpected happened, Cluster $clusterId should not have person id $personID",
);
} else {
// This should not happen, means a clusterID is still assigned to a personID of a person that no longer exists
// Logging the error and deleting the clusterID to personID mapping
_logger.severe(
"`getAllFace`: Cluster $clusterId should not have person id ${clusterIDToPersonID[clusterId]}, deleting the mapping",
);
await MLDataDB.instance.removeClusterToPerson(
personID: personID,
clusterID: clusterId,
);
}
}
if (files.length < minClusterSize) continue;
facesResult.add(