Fixes

backend/src/album/album.entity.ts

@@ -1,4 +1,5 @@
 import { Entity, Column, OneToMany, PrimaryGeneratedColumn } from 'typeorm';
+import { IsNotEmpty, IsString } from 'class-validator';
 import { Song } from '../song/song.entity';
 
 @Entity()
@@ -7,12 +8,18 @@ export class Album {
   id: number
 
   @Column()
+  @IsString()
+  @IsNotEmpty()
   title: string
 
   @Column()
+  @IsString()
+  @IsNotEmpty()
   artist: string
 
   @Column()
+  @IsString()
+  @IsNotEmpty
   genre: string
 
   @OneToMany(() => Song, (song) => song.album, { eager: true, onDelete: 'CASCADE' })

backend/src/album/album.service.ts

@@ -50,11 +50,11 @@ export class AlbumService {
         return null;
       }
 
-      albumToUpdate.title = updateAlbumDto.title;
+      const savedAlbum = await this.albumRepository.update({ id: updateAlbumDto.id }, {
-      albumToUpdate.artist = updateAlbumDto.artist;
+        title: updateAlbumDto.title,
-      albumToUpdate.genre = updateAlbumDto.genre;
+        artist: updateAlbumDto.artist,
-      const savedAlbum = await this.albumRepository.save(albumToUpdate);
+        genre: updateAlbumDto.genre
-      return savedAlbum;
+      });
     } else {
       console.error("AlbumService: update: IDs do not match", id, updateAlbumDto);
       return null;

backend/src/song/song.entity.ts

@@ -1,4 +1,5 @@
 import { Entity, Column, ManyToOne, PrimaryGeneratedColumn } from 'typeorm';
+import { IsNotEmpty, IsNumber, IsString } from 'class-validator';
 import { Album } from '../album/album.entity';
 
 @Entity()
@@ -7,12 +8,16 @@ export class Song {
   id: number
 
   @Column()
+  @IsString()
+  @IsNotEmpty()
   title: string
 
   @Column()
+  @IsNumber()
   duration: number
 
   @Column()
+  @IsNumber()
   trackNumber: number
 
   @ManyToOne(() => Album, (album) => album.songs, { cascade: true })

backend/src/song/song.service.ts

@@ -51,12 +51,12 @@ export class SongService {
         return null;
       }
 
-      songToUpdate.id = updateSongDto.id;
+      const song = await this.songRepository.update({ id: updateSongDto.id }, {
-      songToUpdate.title = updateSongDto.title;
+        title: updateSongDto.title,
-      songToUpdate.duration = updateSongDto.duration;
+        duration: updateSongDto.duration,
-      songToUpdate.trackNumber = updateSongDto.trackNumber;
+        trackNumber: updateSongDto.trackNumber,
-      songToUpdate.album = album;
+        album: album,
-      const song = await this.songRepository.save(songToUpdate);
+      });
       return song;
     } else {
       console.error("SongService: update: IDs do not match");

frontend/src/app/album/[id]/page.tsx

@@ -44,13 +44,10 @@ export default function Page() {
 
     const formData = new FormData(event.currentTarget);
     try {
-      console.log(formData);
       if (formData.get('id') == "") {
-        const data = await createSong(formData);
+        await createSong(formData);
-        console.log(data);
       } else {
-        const data = await updateSong(formData);
+        await updateSong(formData);
-        console.log(data);
       }
     } catch (error) {
       console.error("Error creating Song: ", error);
@@ -75,7 +72,7 @@ export default function Page() {
         const data = await getSong(parseInt(songId));
         setFormModalTitle("Edit Song");
         setFormModalButtonLabel("Save");
-        setFormSongId(songId)
+        setFormSongId(songId);
         setFormAlbumId(albumId);
         setFormSongTitle(data.title);
         setFormSongDuration(data.duration);

frontend/src/app/album/page.tsx

@@ -39,7 +39,6 @@ export default function Page() {
 
     const formData = new FormData(event.currentTarget);
     try {
-      console.log(formData);
       if (formData.get('id') == "") {
         await createAlbum(formData);
       } else {