force filename to be correctly formatted on new playlist

Alaanor · Jun 6, 2020 · ea1ef37 · ea1ef37
1 parent 97db776
commit ea1ef37
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/libraries/os/beatSaber/installer/PlaylistInstaller.ts b/src/libraries/os/beatSaber/installer/PlaylistInstaller.ts
@@ -11,6 +11,7 @@ import PlaylistScanner from "@/libraries/scanner/playlist/PlaylistScanner";
 import PlaylistFormatType from "@/libraries/playlist/PlaylistFormatType";
 import store from "@/plugins/store";
 import PlaylistFilenameExtension from "@/libraries/playlist/PlaylistFilenameExtension";
+import PlaylistFilename from "@/libraries/playlist/PlaylistFilename";
 
 declare const __static: string;
 const defaultCoverPath = path.join(__static, "defaultCover.jpg");
@@ -38,7 +39,10 @@ export default class PlaylistInstaller {
     ] as PlaylistFormatType;
     const extension = PlaylistFilenameExtension.GetFor(format);
     const filepath = path
-      .join(await BeatSaber.getPlaylistFolder(), `${name}.${extension}`)
+      .join(
+        await BeatSaber.getPlaylistFolder(),
+        `${PlaylistFilename.computeFilenameFor(name)}.${extension}`
+      )
       .toLowerCase();
 
     const emptyPlaylist = {

diff --git a/src/libraries/playlist/PlaylistFilename.ts b/src/libraries/playlist/PlaylistFilename.ts
@@ -26,7 +26,7 @@ export default class PlaylistFilename {
     return newPlaylistPath;
   }
 
-  private static computeFilenameFor(title: string) {
+  public static computeFilenameFor(title: string) {
     return title.replace(/[^a-zA-Z0-9]+/g, "_").toLowerCase();
   }
 }
diff --git a/src/libraries/playlist/loader/serializer/JsonSerializer.ts b/src/libraries/playlist/loader/serializer/JsonSerializer.ts
@@ -4,7 +4,6 @@ import { PlaylistBase, PlaylistMap } from "@/libraries/playlist/PlaylistLocal";
 import Base64SrcLoader from "@/libraries/os/utils/Base64SrcLoader";
 
 export default class JsonSerializer extends PlaylistSerializer {
-  // eslint-disable-next-line class-methods-use-this
   public async serialize(playlist: PlaylistBase): Promise<void> {
     const data = JsonSerializer.format(playlist);
     await fs.writeFile(this.filepath, JSON.stringify(data));