feat: get music name from filename automatically

mebtte · Jul 21, 2024 · 1243aac · 1243aac
1 parent cf53ed7
commit 1243aac
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 25 deletions.
diff --git a/apps/pwa/src/pages/player/pages/my_music/create_music_dialog/index.tsx b/apps/pwa/src/pages/player/pages/my_music/create_music_dialog/index.tsx
@@ -40,7 +40,7 @@ import playerEventemitter, {
 } from '../../../eventemitter';
 import { Singer } from './constants';
 import upperCaseFirstLetter from '#/utils/upper_case_first_letter';
-import { base64ToCover, canAudioPlay } from './utils';
+import { base64ToCover, canAudioPlay, getMusicNameFromFilename } from './utils';
 
 const maskProps: { style: CSSProperties } = {
   style: { zIndex: ZIndex.DIALOG },
@@ -103,8 +103,8 @@ function CreateMusicDialog() {
       getMusicFileMetadata(a)
         .then((metadata) => {
           const { title, artist } = metadata;
-          if (!name && title) {
-            setName(title);
+          if (!name) {
+            setName(title || getMusicNameFromFilename(a.name));
           }
           if (!singerList.length && artist) {
             searchSingerRequest({
@@ -155,11 +155,7 @@ function CreateMusicDialog() {
       });
 
       try {
-        const {
-          // lyric,
-          picture,
-          year,
-        } = await getMusicFileMetadata(asset);
+        const { picture, year } = await getMusicFileMetadata(asset);
         const updateCover = async (pb: string) => {
           const coverBlob = await base64ToCover(pb);
           const { id: assetId } = await uploadAsset(
@@ -175,14 +171,6 @@ function CreateMusicDialog() {
         };
 
         await Promise.all([
-          // musicType === MusicType.SONG && lyric
-          //   ? await updateMusic({
-          //       id,
-          //       key: AllowUpdateKey.LYRIC,
-          //       value: [lyric],
-          //       requestMinimalDuration: 0,
-          //     })
-          //   : null,
           picture ? updateCover(picture.dataURI) : null,
           year
             ? updateMusic({

diff --git a/apps/pwa/src/pages/player/pages/my_music/create_music_dialog/utils.ts b/apps/pwa/src/pages/player/pages/my_music/create_music_dialog/utils.ts
@@ -40,3 +40,12 @@ export function canAudioPlay(file: File) {
     URL.revokeObjectURL(url);
   });
 }
+
+export function getMusicNameFromFilename(filename: string) {
+  const lastIndex = filename.lastIndexOf('.');
+  return lastIndex === -1
+    ? filename
+    : lastIndex === 0
+    ? ''
+    : filename.slice(0, lastIndex);
+}
diff --git a/shared/utils/get_music_file_metadata.ts b/shared/utils/get_music_file_metadata.ts
@@ -1,7 +1,6 @@
 import jsmediatags from 'jsmediatags';
 
 interface Metadata {
-  // lyric?: string;
   title?: string;
   artist?: string;
   picture?: {
@@ -21,18 +20,11 @@ function getMusicFileMetadata(file: File | string) {
   return new Promise<Metadata>((resolve, reject) => {
     jsmediatags.read(file, {
       onSuccess: async (metadata) => {
-        const {
-          // lyrics,
-          picture,
-          title,
-          artist,
-          year,
-        } = metadata.tags;
+        const { picture, title, artist, year } = metadata.tags;
 
         return resolve({
           title,
           artist,
-          // lyric: lyrics,
           picture: picture
             ? {
                 dataURI: pictureToDataURI(picture),