Add api and redux action to delete a sound file source

Author: Joltras

lib/api/service/client/sound_client_api.dart

@@ -16,11 +16,11 @@ class SoundClientApi extends BaseApi<SoundEventModel> {
   /// Serialization and deserialization for [SoundEventModel] are handled
   /// by [SoundEventModel.fromJson] and [SoundEventModel.toJson] respectively.
   SoundClientApi({required super.apiClient})
-      : super(
-    endpoint: 'sound',
-    fromJson: SoundEventModel.fromJson,
-    toJson: (p0) => p0.toJson(),
-  );
+    : super(
+        endpoint: 'sound',
+        fromJson: SoundEventModel.fromJson,
+        toJson: (p0) => p0.toJson(),
+      );
 
   /// Fetches a paginated list of [SoundFileSource] objects associated with a specific sound event.
   ///
@@ -33,10 +33,10 @@ class SoundClientApi extends BaseApi<SoundEventModel> {
   /// Returns a [Future] that completes with a [PaginatedResult] containing
   /// a list of [SoundFileSource] and pagination details.
   Future<PaginatedResult<SoundFileSource>> getFiles(
-      String id, [
-        int page = 1,
-        int items = 20,
-      ]) async {
+    String id, [
+    int page = 1,
+    int items = 20,
+  ]) async {
     final baseUri = Uri.parse(apiClient.baseUrl);
     final uri = baseUri.replace(
       path: '${baseUri.path}/$endpoint/$id/sources',
@@ -49,7 +49,8 @@ class SoundClientApi extends BaseApi<SoundEventModel> {
     final result = await apiClient.dio.getUri(uri);
     return PaginatedResult.fromJson(
       result.data,
-          (jsonSource) => SoundFileSource.fromJson(jsonSource as Map<String, dynamic>),
+      (jsonSource) =>
+          SoundFileSource.fromJson(jsonSource as Map<String, dynamic>),
     );
   }
 
@@ -58,7 +59,10 @@ class SoundClientApi extends BaseApi<SoundEventModel> {
   /// The request is the updated entry from the backend
   /// - [modelId]: The unique id of the sound event
   /// - [fileToLink]: The [SoundFileSource] which should be linked
-  Future<SoundFileSource> linkFile(String modelId, SoundFileSource fileToLink) async {
+  Future<SoundFileSource> linkFile(
+    String modelId,
+    SoundFileSource fileToLink,
+  ) async {
     final baseUri = Uri.parse(apiClient.baseUrl);
     final uri = baseUri.replace(
       path: '${baseUri.path}/$endpoint/$modelId/sources',
@@ -67,12 +71,35 @@ class SoundClientApi extends BaseApi<SoundEventModel> {
     return SoundFileSource.fromJson(result.data);
   }
 
-  Future<SoundFileSource> updateFile(String modelId, SoundFileSource file) async {
+  Future<SoundFileSource> updateFile(
+    String modelId,
+    SoundFileSource file,
+  ) async {
     final baseUri = Uri.parse(apiClient.baseUrl);
     final uri = baseUri.replace(
       path: '${baseUri.path}/$endpoint/$modelId/sources/update',
     );
     final result = await apiClient.dio.postUri(uri, data: file.toJson());
     return SoundFileSource.fromJson(result.data);
   }
+
+  /// Deletes a [SoundFileSource] associated with a specific sound event.
+  ///
+  /// The request is made to the `/sound/{id}/sources/delete` endpoint.
+  ///
+  /// - [modelId]: The unique identifier of the sound event.
+  /// - [file]: The [SoundFileSource] to be deleted.
+  ///
+  /// Returns a [Future] that completes with the deleted [SoundFileSource].
+  Future<SoundFileSource> deleteFile(
+    String modelId,
+    SoundFileSource file,
+  ) async {
+    final baseUri = Uri.parse(apiClient.baseUrl);
+    final uri = baseUri.replace(
+      path: '${baseUri.path}/$endpoint/$modelId/sources/delete',
+    );
+    final result = await apiClient.dio.deleteUri(uri, data: file.toJson());
+    return SoundFileSource.fromJson(result.data);
+  }
 }

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

@@ -7,7 +7,6 @@ import 'package:stelaris/api/service/client/sound_client_api.dart';
 import 'package:stelaris/api/state/app_state.dart';
 
 class InitSoundFileAction extends ReduxAction<AppState> {
-
   @override
   Future<AppState?> reduce() async {
     if (state.selectedSoundEvent == null) return null;
@@ -17,7 +16,9 @@ class InitSoundFileAction extends ReduxAction<AppState> {
     if (model.files.items.isNotEmpty) return null;
 
     final soundClient = ApiService().soundApi as SoundClientApi;
-    final PaginatedResult<SoundFileSource> files = await soundClient.getFiles(model.id!);
+    final PaginatedResult<SoundFileSource> files = await soundClient.getFiles(
+      model.id!,
+    );
     return state.copyWith(
       selectedSoundEvent: model.copyWith(
         files: model.files.copyWith(
@@ -33,7 +34,6 @@ class InitSoundFileAction extends ReduxAction<AppState> {
 }
 
 class SoundFileLinkAction extends ReduxAction<AppState> {
-
   final SoundFileSource source;
 
   SoundFileLinkAction(this.source);
@@ -43,20 +43,28 @@ class SoundFileLinkAction extends ReduxAction<AppState> {
     if (state.selectedSoundEvent == null) return null;
 
     final SoundEventModel soundModel = state.selectedSoundEvent!;
-    final List<SoundFileSource> list = List.of(soundModel.files.items, growable: true);
+    final List<SoundFileSource> list = List.of(
+      soundModel.files.items,
+      growable: true,
+    );
     final SoundClientApi soundApi = ApiService().soundApi as SoundClientApi;
-    final SoundFileSource linkedFile = await soundApi.linkFile(soundModel.id!, source);
+    final SoundFileSource linkedFile = await soundApi.linkFile(
+      soundModel.id!,
+      source,
+    );
 
     list.add(linkedFile);
 
     // Create a new PaginatedResult with the new list and updated counts
-    final PaginatedResult<SoundFileSource> updatedSources = soundModel.files.copyWith(
+    final PaginatedResult<SoundFileSource>
+    updatedSources = soundModel.files.copyWith(
       items: list,
       totalItems: soundModel.files.totalItems + 1, // Adjust counts as necessary
       // Potentially update totalPages if this new item pushes it to a new page
     );
-    return state.copyWith(selectedSoundEvent: soundModel.copyWith(files: updatedSources));
-
+    return state.copyWith(
+      selectedSoundEvent: soundModel.copyWith(files: updatedSources),
+    );
   }
 }
 
@@ -73,7 +81,10 @@ class SoundFileUpdateAction extends ReduxAction<AppState> {
     final SoundClientApi soundApi = ApiService().soundApi as SoundClientApi;
 
     // Call the API to update the file
-    final SoundFileSource updatedFile = await soundApi.updateFile(soundModel.id!, soundFile);
+    final SoundFileSource updatedFile = await soundApi.updateFile(
+      soundModel.id!,
+      soundFile,
+    );
 
     // Create a new list with the updated item
     final List<SoundFileSource> list = List.of(soundModel.files.items);
@@ -83,14 +94,52 @@ class SoundFileUpdateAction extends ReduxAction<AppState> {
     }
 
     // Create a new PaginatedResult with the updated list
-    final PaginatedResult<SoundFileSource> updatedSources = soundModel.files.copyWith(
-      items: list,
-    );
+    final PaginatedResult<SoundFileSource> updatedSources = soundModel.files
+        .copyWith(items: list);
 
-    return state.copyWith(selectedSoundEvent: soundModel.copyWith(files: updatedSources));
+    return state.copyWith(
+      selectedSoundEvent: soundModel.copyWith(files: updatedSources),
+    );
   }
 }
 
+class SoundFileSourceDeleteAction extends ReduxAction<AppState> {
+  final SoundFileSource soundFile;
+
+  SoundFileSourceDeleteAction(this.soundFile);
+
+  @override
+  Future<AppState?> reduce() async {
+    if (state.selectedSoundEvent == null) return null;
+
+    final SoundEventModel model = state.selectedSoundEvent!;
+    final soundClient = ApiService().soundApi as SoundClientApi;
+
+    final SoundFileSource deletedFile = await soundClient.deleteFile(
+      model.id!,
+      soundFile,
+    );
+
+    final PaginatedResult<SoundFileSource> entries = model.files;
+
+    final List<SoundFileSource> updatedFiles = List.of(
+      entries.items,
+      growable: true,
+    );
+    updatedFiles.removeWhere((file) => file.id == deletedFile.id);
+
+    final updatedSoundEvent = model.copyWith(
+      files: entries.copyWith(
+        items: updatedFiles,
+        currentPage: entries.currentPage,
+        totalPages: entries.totalPages,
+        totalItems: entries.totalItems - 1,
+      ),
+    );
+
+    return state.copyWith(selectedSoundEvent: updatedSoundEvent);
+  }
+}
 
 class LoadMoreSoundFiles extends ReduxAction<AppState> {
   LoadMoreSoundFiles({required this.pageToLoad, this.pageSize = 1});
@@ -143,6 +192,8 @@ class SetSelectedSoundLoading extends ReduxAction<AppState> {
   AppState? reduce() {
     final sel = state.selectedSoundEvent;
     if (sel == null) return null;
-    return state.copyWith(selectedSoundEvent: sel.copyWith(isLoading: isLoading));
+    return state.copyWith(
+      selectedSoundEvent: sel.copyWith(isLoading: isLoading),
+    );
   }
 }