feat: adapt frontend update playlist metadata with only provided fields | [AntonioMrtz#246]

Author: AntonioMrtz

.pre-commit-config.yaml

@@ -11,7 +11,7 @@ repos:
       - id: check-merge-conflict
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.3.7
+    rev: v0.8.5
     hooks:
       - id: ruff
         args: [--fix]

Backend/app/spotify_electron/playlist/playlist_service.py

@@ -162,11 +162,10 @@ def create_playlist(
 
 def update_playlist_metadata(  # noqa: C901
     name: str,
+    token: TokenData,
     new_name: str | None = None,
     photo: str | None = None,
     description: str | None = None,
-    *,
-    token: TokenData,
 ) -> None:
     """
     Updates specified fields of a playlist's metadata.

Electron/src/pages/Playlist/Playlist.tsx

@@ -269,30 +269,24 @@ export default function Playlist({ refreshSidebarData }: PropsPlaylist) {
     event.preventDefault();
 
     try {
-      // TODO add song to playlist instead of fetching already present songs
-      const playlistData =
-        await PlaylistsService.getPlaylistPlaylistsNameGet(playlistName);
-      const newSongsPutPlaylist = [...playlistData.song_names];
-
       const formPhoto =
         formData.photo && formData.photo.includes('http') ? formData.photo : '';
 
       const formDescription = formData.description;
 
       if (formData.name !== playlistName && formData.name !== '') {
-        PlaylistsService.updatePlaylistPlaylistsNamePut(
+        PlaylistsService.updatePlaylistMetadataPlaylistsNamePut(
           playlistName,
+          formData.name,
           formPhoto,
           formDescription,
-          newSongsPutPlaylist,
-          formData.name,
         );
       } else {
-        PlaylistsService.updatePlaylistPlaylistsNamePut(
+        PlaylistsService.updatePlaylistMetadataPlaylistsNamePut(
           playlistName,
+          null,
           formPhoto,
           formDescription,
-          newSongsPutPlaylist,
         );
       }
 
@@ -304,7 +298,6 @@ export default function Playlist({ refreshSidebarData }: PropsPlaylist) {
           refreshSidebarData();
           navigate(`/playlist/${formData.name}`, { replace: true });
         }, 500);
-        refreshPlaylistData();
         return;
       }
 

Electron/src/swagger/api/services/PlaylistsService.ts

@@ -31,28 +31,27 @@ export class PlaylistsService {
         });
     }
     /**
-     * Update Playlist
-     * Update playlist
+     * Update Playlist Metadata
+     * Update playlist metadata
      *
      * Args:
-     * photo (str): playlist new photo
-     * description (str): playlist new description
-     * song_names (list[str], optional): playlist new song names. Defaults to Body(...).
-     * new_name (str | None, optional): playlist new name. Defaults to None.
+     * name (str): playlist name
+     * new_name (str | None): new playlist name
+     * photo (str | None): new photo URL
+     * description (str | None): new description
+     * token (TokenData): User authentication token
      * @param name
+     * @param newName
      * @param photo
      * @param description
-     * @param requestBody
-     * @param newName
      * @returns any Successful Response
      * @throws ApiError
      */
-    public static updatePlaylistPlaylistsNamePut(
+    public static updatePlaylistMetadataPlaylistsNamePut(
         name: string,
-        photo: string,
-        description: string,
-        requestBody: Array<string>,
         newName?: (string | null),
+        photo?: (string | null),
+        description?: (string | null),
     ): CancelablePromise<any> {
         return __request(OpenAPI, {
             method: 'PUT',
@@ -61,12 +60,10 @@ export class PlaylistsService {
                 'name': name,
             },
             query: {
+                'new_name': newName,
                 'photo': photo,
                 'description': description,
-                'new_name': newName,
             },
-            body: requestBody,
-            mediaType: 'application/json',
             errors: {
                 422: `Validation Error`,
             },