OpenSubsonic API: root music folders are returned using ids

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();

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);

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());
         });
 

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;