improve music release

packages/server/services/music/classes/release/index.js

@@ -12,6 +12,23 @@ const AllowedUpdateFields = [
 
 export default class Release {
     static async create(payload) {
+        if (!payload.title) {
+            throw new OperationError(400, "Release title is required")
+        }
+
+        if (!payload.list) {
+            throw new OperationError(400, "Release list is required")
+        }
+
+        // ensure list is an array of strings with tracks ids only
+        playload.list = playload.list.map((item) => {
+            if (typeof item !== "string") {
+                item = item._id
+            }
+
+            return item
+        })
+
         const release = new MusicRelease({
             user_id: payload.user_id,
             created_at: Date.now(),
@@ -48,13 +65,21 @@ export default class Release {
             }
         }
 
+        // ensure list is an array of strings with tracks ids only
+        release.list = release.list.map((item) => {
+            if (typeof item !== "string") {
+                item = item._id
+            }
+
+            return item
+        })
+
         release = await MusicRelease.findByIdAndUpdate(id, release)
 
         return release
     }
 
     static async fullfillItemData(release) {
-
         return release
     }
 }

packages/server/services/music/classes/track/methods/create.js

@@ -15,6 +15,8 @@ export default async (payload = {}) => {
         return await ModifyTrack(payload._id, payload)
     }
 
+    let metadata = Object()
+
     try {
         const sourceStream = await axios({
             url: payload.source,
@@ -24,25 +26,26 @@ export default async (payload = {}) => {
 
         stream = sourceStream.data
         headers = sourceStream.headers
-    } catch (error) {
-        throw new OperationError(500, `Failed to process fetching source: ${error.message}`)
-    }
 
-    const fileMetadata = await MusicMetadata.parseStream(stream, {
+        const streamMetadata = await MusicMetadata.parseStream(stream, {
             mimeType: headers["content-type"],
         })
 
-    let metadata = {
+        metadata = {
-        format: fileMetadata.format.codec,
+            ...metadata,
-        channels: fileMetadata.format.numberOfChannels,
+            format: streamMetadata.format.codec,
-        sampleRate: fileMetadata.format.sampleRate,
+            channels: streamMetadata.format.numberOfChannels,
-        bits: fileMetadata.format.bitsPerSample,
+            sampleRate: streamMetadata.format.sampleRate,
-        lossless: fileMetadata.format.lossless,
+            bits: streamMetadata.format.bitsPerSample,
-        duration: fileMetadata.format.duration,
+            lossless: streamMetadata.format.lossless,
+            duration: streamMetadata.format.duration,
 
-        title: fileMetadata.common.title,
+            title: streamMetadata.common.title,
-        artists: fileMetadata.common.artists,
+            artists: streamMetadata.common.artists,
-        album: fileMetadata.common.album,
+            album: streamMetadata.common.album,
+        }
+    } catch (error) {
+        // sowy :(
     }
 
     if (typeof payload.metadata === "object") {
@@ -52,6 +55,16 @@ export default async (payload = {}) => {
         }
     }
 
+    metadata.format = metadata.format.toUpperCase()
+
+    if (
+        metadata.format === "FLAC" || 
+        metadata.format === "WAV" || 
+        metadata.format === "ALAC"
+    ) {
+        metadata.lossless = true
+    }
+
     const obj = {
         title: payload.title,
         album: payload.album,
@@ -70,6 +83,10 @@ export default async (payload = {}) => {
         obj.artists.push(payload.artists)
     }
 
+    if (typeof payload.artist === "string") {
+        obj.artists.push(payload.artist)
+    }
+
     if (obj.artists.length === 0 || !obj.artists) {
         obj.artists = metadata.artists
     }