fixed API rate limit exit bug on TrackTile

This commit is contained in:
Kingkor Roy Tirtho 2022-06-16 22:59:21 +06:00
parent 8f200d6488
commit 038856a1f7
4 changed files with 90 additions and 73 deletions

View File

@ -8,6 +8,7 @@ import 'package:spotube/models/Logger.dart';
import 'package:spotube/provider/Auth.dart'; import 'package:spotube/provider/Auth.dart';
import 'package:spotube/provider/Playback.dart'; import 'package:spotube/provider/Playback.dart';
import 'package:spotube/provider/SpotifyDI.dart'; import 'package:spotube/provider/SpotifyDI.dart';
import 'package:spotube/provider/SpotifyRequests.dart';
class PlayerActions extends HookConsumerWidget { class PlayerActions extends HookConsumerWidget {
final MainAxisAlignment mainAxisAlignment; final MainAxisAlignment mainAxisAlignment;
@ -51,6 +52,10 @@ class PlayerActions extends HookConsumerWidget {
logger.e("FavoriteButton.onPressed", e, stack); logger.e("FavoriteButton.onPressed", e, stack);
} finally { } finally {
update(); update();
ref.refresh(currentUserSavedTracksQuery);
ref.refresh(
playlistTracksQuery("user-liked-tracks"),
);
} }
}); });
}), }),

View File

@ -2,7 +2,6 @@ import 'dart:convert';
import 'package:flutter/services.dart'; import 'package:flutter/services.dart';
import 'package:flutter_hooks/flutter_hooks.dart'; import 'package:flutter_hooks/flutter_hooks.dart';
import 'package:go_router/go_router.dart';
import 'package:hooks_riverpod/hooks_riverpod.dart'; import 'package:hooks_riverpod/hooks_riverpod.dart';
import 'package:spotube/components/Shared/HeartButton.dart'; import 'package:spotube/components/Shared/HeartButton.dart';
import 'package:spotube/components/Shared/TrackCollectionView.dart'; import 'package:spotube/components/Shared/TrackCollectionView.dart';

View File

@ -12,6 +12,7 @@ import 'package:spotube/models/Logger.dart';
import 'package:spotube/provider/Auth.dart'; import 'package:spotube/provider/Auth.dart';
import 'package:spotube/provider/Playback.dart'; import 'package:spotube/provider/Playback.dart';
import 'package:spotube/provider/SpotifyDI.dart'; import 'package:spotube/provider/SpotifyDI.dart';
import 'package:spotube/provider/SpotifyRequests.dart';
class TrackTile extends HookConsumerWidget { class TrackTile extends HookConsumerWidget {
final Playback playback; final Playback playback;
@ -41,6 +42,13 @@ class TrackTile extends HookConsumerWidget {
final spotify = ref.watch(spotifyProvider); final spotify = ref.watch(spotifyProvider);
final update = useForceUpdate(); final update = useForceUpdate();
final savedTracksSnapshot = ref.watch(currentUserSavedTracksQuery);
final isSaved = savedTracksSnapshot.asData?.value.any(
(e) => track.value.id! == e.id,
) ??
false;
final actionFavorite = useCallback((bool isLiked) async { final actionFavorite = useCallback((bool isLiked) async {
try { try {
isLiked isLiked
@ -50,6 +58,8 @@ class TrackTile extends HookConsumerWidget {
logger.e("FavoriteButton.onPressed", e, stack); logger.e("FavoriteButton.onPressed", e, stack);
} finally { } finally {
update(); update();
ref.refresh(currentUserSavedTracksQuery);
ref.refresh(playlistTracksQuery("user-liked-tracks"));
} }
}, [track.value.id, spotify]); }, [track.value.id, spotify]);
@ -229,10 +239,7 @@ class TrackTile extends HookConsumerWidget {
Text(duration), Text(duration),
], ],
const SizedBox(width: 10), const SizedBox(width: 10),
FutureBuilder<bool>( PopupMenuButton(
future: spotify.tracks.me.containsOne(track.value.id!),
builder: (context, snapshot) {
return PopupMenuButton(
icon: const Icon(Icons.more_horiz_rounded), icon: const Icon(Icons.more_horiz_rounded),
itemBuilder: (context) { itemBuilder: (context) {
return [ return [
@ -262,7 +269,7 @@ class TrackTile extends HookConsumerWidget {
PopupMenuItem( PopupMenuItem(
child: Row( child: Row(
children: [ children: [
Icon(snapshot.data == true Icon(isSaved
? Icons.favorite_rounded ? Icons.favorite_rounded
: Icons.favorite_border_rounded), : Icons.favorite_border_rounded),
const SizedBox(width: 10), const SizedBox(width: 10),
@ -286,7 +293,7 @@ class TrackTile extends HookConsumerWidget {
onSelected: (value) { onSelected: (value) {
switch (value) { switch (value) {
case "favorite": case "favorite":
actionFavorite(snapshot.data == true); actionFavorite(isSaved);
break; break;
case "add-playlist": case "add-playlist":
actionAddToPlaylist(); actionAddToPlaylist();
@ -299,8 +306,7 @@ class TrackTile extends HookConsumerWidget {
break; break;
} }
}, },
); ),
})
], ],
); );
} }

View File

@ -98,6 +98,13 @@ final artistRelatedArtistsQuery =
}, },
); );
final currentUserSavedTracksQuery = FutureProvider<List<Track>>((ref) {
final spotify = ref.watch(spotifyProvider);
return spotify.tracks.me.saved.all().then(
(tracks) => tracks.map((e) => e.track!).toList(),
);
});
final playlistTracksQuery = FutureProvider.family<List<Track>, String>( final playlistTracksQuery = FutureProvider.family<List<Track>, String>(
(ref, id) { (ref, id) {
final spotify = ref.watch(spotifyProvider); final spotify = ref.watch(spotifyProvider);