summaryrefslogtreecommitdiff
path: root/player-android/lib/providers/progress_queue_provider.dart
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-05-22 09:52:26 +0300
committerPaul Buetow <paul@buetow.org>2026-05-22 09:52:26 +0300
commitb0f07dd0a4535a330553e323417ccdd354fae22f (patch)
treea39fb53a0dc6206d57c690ea11d317b9a20772cc /player-android/lib/providers/progress_queue_provider.dart
parentaaf61a9e3c0e1439faee36c21462e0788639b7f1 (diff)
Fix bb review issues: stale snapshot, double-tap guard, button dedup, comments
- _downloadEpisodeAt: read _episodes fresh inside setState instead of using the pre-await snapshot, and guard that the index is still valid and the row still lacks a mediaId, preventing silent overwrites of data refreshed by _load() during the await. - Add Set<int> _pendingDownloads to prevent concurrent download API calls when the user double-taps; visually disable the button while in-flight via new isLoading parameter on _DownloadButton. - Extract _EpisodeActionButton shared primitive to eliminate structural duplication between _PlayButton and _DownloadButton (DRY). - Fix misleading mediaId null-safety comment to describe the actual guarantee. - Add comments to episodeToggleErrorMessage and episodeDownloadErrorMessage explaining why they use action-specific fallback strings. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'player-android/lib/providers/progress_queue_provider.dart')
0 files changed, 0 insertions, 0 deletions