From dc9ce79c2d3f089df274ca2da814580bec3036c5 Mon Sep 17 00:00:00 2001
From: shenlong-tanwen <139912620+shalong-tanwen@users.noreply.github.com>
Date: Wed, 12 Aug 2026 20:14:44 +0530
Subject: [PATCH] minor fixes

---
 .../repositories/person_api.repository.dart   |  7 +---
 mobile/lib/services/person.service.dart       | 32 -------------------
 .../search/search_filter/people_picker.dart   |  2 +-
 3 files changed, 2 insertions(+), 39 deletions(-)
 delete mode 100644 mobile/lib/services/person.service.dart

diff --git a/mobile/lib/repositories/person_api.repository.dart b/mobile/lib/repositories/person_api.repository.dart
index 5804e1da3b..262868eea1 100644
--- a/mobile/lib/repositories/person_api.repository.dart
+++ b/mobile/lib/repositories/person_api.repository.dart
@@ -11,11 +11,6 @@ class PersonApiRepository extends ApiRepository {
 
   PersonApiRepository(this._api);
 
-  Future<List<Person>> getAll() async {
-    final dto = await checkNull(_api.getAllPeople());
-    return dto.people.map(_toPerson).toList();
-  }
-
   Future<Person> update(String id, {String? name, DateTime? birthday}) async {
     final birthdayUtc = birthday == null ? null : DateTime.utc(birthday.year, birthday.month, birthday.day);
     final dto = PersonUpdateDto(
@@ -27,5 +22,5 @@ class PersonApiRepository extends ApiRepository {
   }
 
   static Person _toPerson(PersonResponseDto dto) =>
-      .new(birthDate: dto.birthDate, id: dto.id, name: dto.name, updatedAt: dto.updatedAt.value);
+      .new(birthDate: dto.birthDate, id: dto.id, name: dto.name, updatedAt: dto.updatedAt.orElse(null));
 }
diff --git a/mobile/lib/services/person.service.dart b/mobile/lib/services/person.service.dart
deleted file mode 100644
index 6a9226a893..0000000000
--- a/mobile/lib/services/person.service.dart
+++ /dev/null
@@ -1,32 +0,0 @@
-import 'package:hooks_riverpod/hooks_riverpod.dart';
-import 'package:immich_mobile/domain/models/person.model.dart';
-import 'package:immich_mobile/repositories/person_api.repository.dart';
-import 'package:logging/logging.dart';
-
-final personServiceProvider = Provider.autoDispose<PersonService>(
-  (ref) => PersonService(ref.watch(personApiRepositoryProvider)),
-);
-
-class PersonService {
-  final Logger _log = Logger("PersonService");
-  final PersonApiRepository _personApiRepository;
-  PersonService(this._personApiRepository);
-
-  Future<List<Person>> getAllPeople() async {
-    try {
-      return await _personApiRepository.getAll();
-    } catch (error, stack) {
-      _log.severe("Error while fetching curated people", error, stack);
-      return [];
-    }
-  }
-
-  Future<Person?> updateName(String id, String name) async {
-    try {
-      return await _personApiRepository.update(id, name: name);
-    } catch (error, stack) {
-      _log.severe("Error while updating person name", error, stack);
-    }
-    return null;
-  }
-}
diff --git a/mobile/lib/widgets/search/search_filter/people_picker.dart b/mobile/lib/widgets/search/search_filter/people_picker.dart
index 1db15a36c1..ee738265b2 100644
--- a/mobile/lib/widgets/search/search_filter/people_picker.dart
+++ b/mobile/lib/widgets/search/search_filter/people_picker.dart
@@ -65,7 +65,7 @@ class PeoplePicker extends HookConsumerWidget {
                     padding: const EdgeInsets.only(bottom: 2.0),
                     child: LargeLeadingTile(
                       title: Text(
-                        person.name,
+                        person.name.nullIfEmpty ?? context.t.no_name,
                         style: context.textTheme.bodyLarge?.copyWith(
                           fontSize: 20,
                           fontWeight: FontWeight.w500,