From 816c1bf41be901441a8b9cb258125b18d482ba74 Mon Sep 17 00:00:00 2001 From: Yaros Date: Wed, 28 Jan 2026 14:44:48 +0100 Subject: [PATCH] refactor: implement suggestions --- .../repositories/people.repository.dart | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/mobile/lib/infrastructure/repositories/people.repository.dart b/mobile/lib/infrastructure/repositories/people.repository.dart index d08c2d5a0b..97b7bab7e3 100644 --- a/mobile/lib/infrastructure/repositories/people.repository.dart +++ b/mobile/lib/infrastructure/repositories/people.repository.dart @@ -2,7 +2,6 @@ import 'package:drift/drift.dart'; import 'package:immich_mobile/domain/models/asset/base_asset.model.dart'; import 'package:immich_mobile/domain/models/person.model.dart'; import 'package:immich_mobile/infrastructure/entities/person.entity.drift.dart'; -import 'package:immich_mobile/infrastructure/entities/remote_asset.entity.drift.dart'; import 'package:immich_mobile/infrastructure/repositories/db.repository.dart'; class DriftPeopleRepository extends DriftDatabaseRepository { @@ -24,17 +23,11 @@ class DriftPeopleRepository extends DriftDatabaseRepository { final query = _db.select(_db.personEntity).join([ innerJoin(_db.assetFaceEntity, _db.assetFaceEntity.personId.equalsExp(_db.personEntity.id)), - innerJoin( - _db.remoteAssetEntity, - _db.remoteAssetEntity.id.equalsExp(_db.assetFaceEntity.assetId) & - _db.remoteAssetEntity.visibility.equals( - $RemoteAssetEntityTable.$convertervisibility.toSql(AssetVisibility.timeline), - ) & - _db.remoteAssetEntity.deletedAt.isNull(), - ), + innerJoin(_db.remoteAssetEntity, _db.remoteAssetEntity.id.equalsExp(_db.assetFaceEntity.assetId)), ]) ..where(_db.personEntity.isHidden.equals(false)) ..where(_db.remoteAssetEntity.deletedAt.isNull()) + ..where(_db.remoteAssetEntity.visibility.equalsValue(AssetVisibility.timeline)) ..groupBy( [_db.personEntity.id], having: Expression.or([