OpenSubsonic API: root music folders are returned using ids

epoupon · Dec 18, 2024 · dfa03ab · dfa03ab
1 parent a72171d
commit dfa03ab
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 9 deletions.
diff --git a/src/libs/subsonic/impl/SubsonicId.cpp b/src/libs/subsonic/impl/SubsonicId.cpp
@@ -33,12 +33,6 @@ namespace lms::api::subsonic
         return "dir-" + id.toString();
     }
 
-    std::string idToString(db::MediaLibraryId id)
-    {
-        // No need to prefix as this is only used at well known places
-        return id.toString();
-    }
-
     std::string idToString(db::ReleaseId id)
     {
         return "al-" + id.toString();

diff --git a/src/libs/subsonic/impl/SubsonicId.hpp b/src/libs/subsonic/impl/SubsonicId.hpp
@@ -31,7 +31,6 @@ namespace lms::api::subsonic
 {
     std::string idToString(db::ArtistId id);
     std::string idToString(db::DirectoryId id);
-    std::string idToString(db::MediaLibraryId id);
     std::string idToString(db::ReleaseId id);
     std::string idToString(db::TrackId id);
     std::string idToString(db::TrackListId id);

diff --git a/src/libs/subsonic/impl/endpoints/Browsing.cpp b/src/libs/subsonic/impl/endpoints/Browsing.cpp
@@ -228,7 +228,7 @@ namespace lms::api::subsonic
         MediaLibrary::find(context.dbSession, [&](const MediaLibrary::pointer& library) {
             Response::Node& musicFolderNode{ musicFoldersNode.createArrayChild("musicFolder") };
 
-            musicFolderNode.setAttribute("id", idToString(library->getId()));
+            musicFolderNode.setAttribute("id", library->getId().getValue());
             musicFolderNode.setAttribute("name", library->getName());
         });
 

diff --git a/src/libs/subsonic/impl/responses/User.cpp b/src/libs/subsonic/impl/responses/User.cpp
@@ -47,7 +47,7 @@ namespace lms::api::subsonic
 
         // users can access all libraries
         db::MediaLibrary::find(context.dbSession, [&](const db::MediaLibrary::pointer& library) {
-            userNode.addArrayValue("folder", idToString(library->getId()));
+            userNode.addArrayValue("folder", library->getId().getValue());
         });
 
         return userNode;