diff --git a/mobile/lib/pages/album/album_additional_shared_user_selection.page.dart b/mobile/lib/pages/album/album_additional_shared_user_selection.page.dart index 194f749a32..146493382e 100644 --- a/mobile/lib/pages/album/album_additional_shared_user_selection.page.dart +++ b/mobile/lib/pages/album/album_additional_shared_user_selection.page.dart @@ -105,7 +105,8 @@ class AlbumAdditionalSharedUserSelectionPage extends HookConsumerWidget { if (sharedUsersList.value.contains(users[index])) { sharedUsersList.value = sharedUsersList.value .where( - (selectedUser) => selectedUser.id != users[index].id, + (selectedUser) => + selectedUser.uid != users[index].uid, ) .toSet(); } else { diff --git a/mobile/lib/pages/album/album_shared_user_selection.page.dart b/mobile/lib/pages/album/album_shared_user_selection.page.dart index 1c797aa449..e06ad5d016 100644 --- a/mobile/lib/pages/album/album_shared_user_selection.page.dart +++ b/mobile/lib/pages/album/album_shared_user_selection.page.dart @@ -118,7 +118,8 @@ class AlbumSharedUserSelectionPage extends HookConsumerWidget { if (sharedUsersList.value.contains(users[index])) { sharedUsersList.value = sharedUsersList.value .where( - (selectedUser) => selectedUser.id != users[index].id, + (selectedUser) => + selectedUser.uid != users[index].uid, ) .toSet(); } else { diff --git a/mobile/lib/pages/common/activities.page.dart b/mobile/lib/pages/common/activities.page.dart index 776ee9977b..bf59716119 100644 --- a/mobile/lib/pages/common/activities.page.dart +++ b/mobile/lib/pages/common/activities.page.dart @@ -51,7 +51,7 @@ class ActivitiesPage extends HookConsumerWidget { final liked = data.firstWhereOrNull( (a) => a.type == ActivityType.like && - a.user.id == user?.id && + a.user.uid == user?.uid && a.assetId == asset?.remoteId, ); diff --git a/mobile/lib/pages/library/partner/partner_detail.page.dart b/mobile/lib/pages/library/partner/partner_detail.page.dart index 618d31affa..c35311fc44 100644 --- a/mobile/lib/pages/library/partner/partner_detail.page.dart +++ b/mobile/lib/pages/library/partner/partner_detail.page.dart @@ -111,7 +111,7 @@ class PartnerDetailPage extends HookConsumerWidget { ), ), ), - renderListProvider: singleUserTimelineProvider(partner.id), + renderListProvider: singleUserTimelineProvider(partner.uid), onRefresh: () => ref.read(assetProvider.notifier).getAllAsset(), deleteEnabled: false, favoriteEnabled: false, diff --git a/mobile/lib/pages/photos/photos.page.dart b/mobile/lib/pages/photos/photos.page.dart index c9211e984d..6095b4a99e 100644 --- a/mobile/lib/pages/photos/photos.page.dart +++ b/mobile/lib/pages/photos/photos.page.dart @@ -110,7 +110,7 @@ class PhotosPage extends HookConsumerWidget { : const SizedBox(), renderListProvider: timelineUsers.length > 1 ? multiUsersTimelineProvider(timelineUsers) - : singleUserTimelineProvider(currentUser?.id), + : singleUserTimelineProvider(currentUser?.uid), buildLoadingIndicator: buildLoadingIndicator, onRefresh: refreshAssets, stackEnabled: true, diff --git a/mobile/lib/providers/album/suggested_shared_users.provider.dart b/mobile/lib/providers/album/suggested_shared_users.provider.dart index 3c8dcb6733..120d21e7f1 100644 --- a/mobile/lib/providers/album/suggested_shared_users.provider.dart +++ b/mobile/lib/providers/album/suggested_shared_users.provider.dart @@ -10,6 +10,6 @@ final otherUsersProvider = final currentUser = ref.watch(currentUserProvider); final allUsers = await userService.getAll(); - allUsers.removeWhere((u) => currentUser?.id == u.id); + allUsers.removeWhere((u) => currentUser?.uid == u.uid); return allUsers; }); diff --git a/mobile/lib/services/asset.service.dart b/mobile/lib/services/asset.service.dart index d187284d07..7bbb7571a9 100644 --- a/mobile/lib/services/asset.service.dart +++ b/mobile/lib/services/asset.service.dart @@ -314,7 +314,7 @@ class AssetService { await refreshRemoteAssets(); final owner = _userService.getMyUser(); final remoteAssets = await _assetRepository.getAll( - ownerId: owner.id, + ownerId: owner.uid, state: AssetState.merged, ); @@ -516,11 +516,11 @@ class AssetService { Future> getRecentlyAddedAssets() { final me = _userService.getMyUser(); - return _assetRepository.getRecentlyAddedAssets(me.id); + return _assetRepository.getRecentlyAddedAssets(me.uid); } Future> getMotionAssets() { final me = _userService.getMyUser(); - return _assetRepository.getMotionAssets(me.id); + return _assetRepository.getMotionAssets(me.uid); } } diff --git a/mobile/lib/services/backup_verification.service.dart b/mobile/lib/services/backup_verification.service.dart index 9aa021a324..6c1d1fbe4b 100644 --- a/mobile/lib/services/backup_verification.service.dart +++ b/mobile/lib/services/backup_verification.service.dart @@ -38,7 +38,7 @@ class BackupVerificationService { /// Returns at most [limit] assets that were backed up without exif Future> findWronglyBackedUpAssets({int limit = 100}) async { - final owner = _userService.getMyUser().id; + final owner = _userService.getMyUser().uid; final List onlyLocal = await _assetRepository.getAll( ownerId: owner, state: AssetState.local, diff --git a/mobile/lib/services/sync.service.dart b/mobile/lib/services/sync.service.dart index 0574dc283b..d0a1b8d076 100644 --- a/mobile/lib/services/sync.service.dart +++ b/mobile/lib/services/sync.service.dart @@ -183,7 +183,7 @@ class SyncService { return false; }, onlyFirst: (UserDto a) => toUpsert.add(a), - onlySecond: (UserDto b) => toDelete.add(b.id), + onlySecond: (UserDto b) => toDelete.add(b.uid), ); if (changes) { await _userRepository.transaction(() async { @@ -222,7 +222,7 @@ class SyncService { ) async { final currentUser = _userService.getMyUser(); final DateTime? since = - (await _eTagRepository.get(currentUser.id))?.time?.toUtc(); + (await _eTagRepository.get(currentUser.uid))?.time?.toUtc(); if (since == null) return null; final DateTime now = DateTime.now(); final (toUpsert, toDelete) = await getChangedAssets(users, since); @@ -321,7 +321,7 @@ class SyncService { return false; } final List inDb = await _assetRepository.getAll( - ownerId: user.id, + ownerId: user.uid, sortBy: AssetSort.checksum, ); assert(inDb.isSorted(Asset.compareByChecksum), "inDb not sorted!"); @@ -427,15 +427,15 @@ class SyncService { // update shared users final List sharedUsers = album.sharedUsers.map((u) => u.toDto()).toList(growable: false); - sharedUsers.sort((a, b) => a.id.compareTo(b.id)); + sharedUsers.sort((a, b) => a.uid.compareTo(b.uid)); final List users = dto.remoteUsers.map((u) => u.toDto()).toList() - ..sort((a, b) => a.id.compareTo(b.id)); + ..sort((a, b) => a.uid.compareTo(b.uid)); final List userIdsToAdd = []; final List usersToUnlink = []; diffSortedListsSync( users, sharedUsers, - compare: (UserDto a, UserDto b) => a.id.compareTo(b.id), + compare: (UserDto a, UserDto b) => a.uid.compareTo(b.uid), both: (a, b) => false, onlyFirst: (UserDto a) => userIdsToAdd.add(a.id), onlySecond: (UserDto a) => usersToUnlink.add(a), @@ -622,7 +622,7 @@ class SyncService { // general case, e.g. some assets have been deleted or there are excluded albums on iOS final inDb = await _assetRepository.getByAlbum( dbAlbum, - ownerId: (_userService.getMyUser()).id, + ownerId: (_userService.getMyUser()).uid, sortBy: AssetSort.checksum, ); diff --git a/mobile/lib/services/timeline.service.dart b/mobile/lib/services/timeline.service.dart index 4e91d27a7c..c3c591c7e4 100644 --- a/mobile/lib/services/timeline.service.dart +++ b/mobile/lib/services/timeline.service.dart @@ -30,12 +30,12 @@ class TimelineService { Future> getTimelineUserIds() async { final me = _userService.getMyUser(); - return _timelineRepository.getTimelineUserIds(me.id); + return _timelineRepository.getTimelineUserIds(me.uid); } Stream> watchTimelineUserIds() async* { final me = _userService.getMyUser(); - yield* _timelineRepository.watchTimelineUsers(me.id); + yield* _timelineRepository.watchTimelineUsers(me.uid); } Stream watchHomeTimeline(String userId) { @@ -52,13 +52,13 @@ class TimelineService { Stream watchArchiveTimeline() async* { final user = _userService.getMyUser(); - yield* _timelineRepository.watchArchiveTimeline(user.id); + yield* _timelineRepository.watchArchiveTimeline(user.uid); } Stream watchFavoriteTimeline() async* { final user = _userService.getMyUser(); - yield* _timelineRepository.watchFavoriteTimeline(user.id); + yield* _timelineRepository.watchFavoriteTimeline(user.uid); } Stream watchAlbumTimeline(Album album) async* { @@ -71,7 +71,7 @@ class TimelineService { Stream watchTrashTimeline() async* { final user = _userService.getMyUser(); - yield* _timelineRepository.watchTrashTimeline(user.id); + yield* _timelineRepository.watchTrashTimeline(user.uid); } Stream watchAllVideosTimeline() { @@ -98,7 +98,7 @@ class TimelineService { Stream watchAssetSelectionTimeline() async* { final user = _userService.getMyUser(); - yield* _timelineRepository.watchAssetSelectionTimeline(user.id); + yield* _timelineRepository.watchAssetSelectionTimeline(user.uid); } GroupAssetsBy _getGroupByOption() { diff --git a/mobile/lib/services/trash.service.dart b/mobile/lib/services/trash.service.dart index 478ad65db0..eda960c988 100644 --- a/mobile/lib/services/trash.service.dart +++ b/mobile/lib/services/trash.service.dart @@ -46,7 +46,7 @@ class TrashService { await _apiService.trashApi.emptyTrash(); - final trashedAssets = await _assetRepository.getTrashAssets(user.id); + final trashedAssets = await _assetRepository.getTrashAssets(user.uid); final ids = trashedAssets.map((e) => e.remoteId!).toList(); await _assetRepository.transaction(() async { @@ -77,7 +77,7 @@ class TrashService { await _apiService.trashApi.restoreTrash(); - final trashedAssets = await _assetRepository.getTrashAssets(user.id); + final trashedAssets = await _assetRepository.getTrashAssets(user.uid); final updatedAssets = trashedAssets.map((asset) { asset.isTrashed = false; return asset; diff --git a/mobile/lib/widgets/common/user_avatar.dart b/mobile/lib/widgets/common/user_avatar.dart index a5a6fa2bdd..7af2efa07d 100644 --- a/mobile/lib/widgets/common/user_avatar.dart +++ b/mobile/lib/widgets/common/user_avatar.dart @@ -16,7 +16,7 @@ Widget userAvatar(BuildContext context, UserDto u, {double? radius}) { foregroundImage: CachedNetworkImageProvider( url, headers: ApiService.getRequestHeaders(), - cacheKey: "user-${u.id}-profile", + cacheKey: "user-${u.uid}-profile", ), // silence errors if user has no profile image, use initials as fallback onForegroundImageError: (exception, stackTrace) {}, diff --git a/mobile/test/modules/shared/sync_service_test.dart b/mobile/test/modules/shared/sync_service_test.dart index 47bc1b9544..caaf669e0a 100644 --- a/mobile/test/modules/shared/sync_service_test.dart +++ b/mobile/test/modules/shared/sync_service_test.dart @@ -125,7 +125,7 @@ void main() { when(() => userRepository.getAll()).thenAnswer((_) async => [owner]); when( () => assetRepository.getAll( - ownerId: owner.id, + ownerId: owner.uid, sortBy: AssetSort.checksum, ), ).thenAnswer((_) async => initialAssets); @@ -200,7 +200,7 @@ void main() { expect(c1, isTrue); when( () => assetRepository.getAll( - ownerId: owner.id, + ownerId: owner.uid, sortBy: AssetSort.checksum, ), ).thenAnswer((_) async => remoteAssets); @@ -213,7 +213,7 @@ void main() { final currentState = [...remoteAssets]; when( () => assetRepository.getAll( - ownerId: owner.id, + ownerId: owner.uid, sortBy: AssetSort.checksum, ), ).thenAnswer((_) async => currentState);