Skip to content

Commit 1d84741

Browse files
authored
fix invalidating wrong provider + removing unneeded providers (#2339)
1 parent 098f916 commit 1d84741

2 files changed

Lines changed: 2 additions & 19 deletions

File tree

lib/src/model/user/user_repository_providers.dart

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -18,25 +18,11 @@ Future<User> user(Ref ref, {required UserId id}) {
1818
return ref.read(userRepositoryProvider).getUser(id, withCanChallenge: true);
1919
}
2020

21-
@riverpod
22-
Future<(User, UserStatus)> userAndStatus(Ref ref, {required UserId id}) {
23-
final repo = ref.read(userRepositoryProvider);
24-
return Future.wait([
25-
repo.getUser(id, withCanChallenge: true),
26-
repo.getUsersStatuses({id}.lock),
27-
], eagerError: true).then((value) => (value[0] as User, (value[1] as IList<UserStatus>).first));
28-
}
29-
3021
@riverpod
3122
Future<UserPerfStats> userPerfStats(Ref ref, {required UserId id, required Perf perf}) {
3223
return ref.read(userRepositoryProvider).getPerfStats(id, perf);
3324
}
3425

35-
@riverpod
36-
Future<IList<UserStatus>> userStatuses(Ref ref, {required ISet<UserId> ids}) {
37-
return ref.read(userRepositoryProvider).getUsersStatuses(ids);
38-
}
39-
4026
@riverpod
4127
Future<IList<Streamer>> liveStreamers(Ref ref) {
4228
return ref.withAggregatorCacheFor(

lib/src/view/user/user_screen.dart

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import 'package:lichess_mobile/src/model/game/game_filter.dart';
88
import 'package:lichess_mobile/src/model/relation/relation_repository.dart';
99
import 'package:lichess_mobile/src/model/user/user.dart';
1010
import 'package:lichess_mobile/src/model/user/user_repository.dart';
11-
import 'package:lichess_mobile/src/model/user/user_repository_providers.dart';
1211
import 'package:lichess_mobile/src/network/connectivity.dart';
1312
import 'package:lichess_mobile/src/network/http.dart';
1413
import 'package:lichess_mobile/src/styles/lichess_icons.dart';
@@ -125,7 +124,7 @@ class _UserScreenState extends ConsumerState<UserScreen> {
125124
);
126125
}
127126
return FullScreenRetryRequest(
128-
onRetry: () => ref.invalidate(userProvider(id: widget.user.id)),
127+
onRetry: () => ref.invalidate(_userScreenDataProvider(widget.user.id)),
129128
);
130129
},
131130
),
@@ -166,9 +165,7 @@ class _UserProfileListView extends ConsumerWidget {
166165
Future<void> userAction(Future<void> Function(LichessClient client) action) async {
167166
setIsLoading(true);
168167
try {
169-
await ref
170-
.withClient(action)
171-
.then((_) => ref.invalidate(userAndStatusProvider(id: user.id)));
168+
await ref.withClient(action).then((_) => ref.invalidate(_userScreenDataProvider(user.id)));
172169
} finally {
173170
setIsLoading(false);
174171
}

0 commit comments

Comments
 (0)