Improve state handling

Author: theEvilReaper

lib/api/state/actions/sound/sound_file_actions.dart

@@ -64,6 +64,16 @@ class LoadMoreSoundFiles extends ReduxAction<AppState> {
   final int pageToLoad;
   final int pageSize;
 
+  @override
+  void before() {
+    dispatch(SetSelectedSoundLoading(true));
+  }
+
+  @override
+  void after() {
+    dispatch(SetSelectedSoundLoading(false));
+  }
+
   @override
   Future<AppState?> reduce() async {
     if (state.selectedSoundEvent == null) return null;
@@ -72,6 +82,8 @@ class LoadMoreSoundFiles extends ReduxAction<AppState> {
     final soundClient = ApiService().soundApi as SoundClientApi;
     final PaginatedResult<SoundFileSource> files = await soundClient.getFiles(
       model.id!,
+      pageToLoad,
+      pageSize,
     );
     final PaginatedResult<SoundFileSource> entries = model.files;
 
@@ -87,3 +99,16 @@ class LoadMoreSoundFiles extends ReduxAction<AppState> {
     return state.copyWith(selectedSoundEvent: updatedSoundEvent);
   }
 }
+
+class SetSelectedSoundLoading extends ReduxAction<AppState> {
+  SetSelectedSoundLoading(this.isLoading);
+
+  final bool isLoading;
+
+  @override
+  AppState? reduce() {
+    final sel = state.selectedSoundEvent;
+    if (sel == null) return null;
+    return state.copyWith(selectedSoundEvent: sel.copyWith(isLoading: isLoading));
+  }
+}

lib/api/state/factory/sound/selected_sound_state.dart

@@ -25,7 +25,12 @@ class SelectedSoundState
             selectedEvent.files.hasNextPage &&
             !selectedEvent.isLoading) {
           dispatch(
-            LoadMoreSoundFiles(pageToLoad: selectedEvent.files.currentPage + 1),
+            LoadMoreSoundFiles(
+              pageToLoad: selectedEvent.files.currentPage + 1,
+              pageSize: selectedEvent.files.pageSize > 0
+                  ? selectedEvent.files.pageSize
+                  : 20,
+            ),
           );
         }
       },