diff --git a/backend/src/album/album.controller.ts b/backend/src/album/album.controller.ts index f65cf48..bb7de97 100644 --- a/backend/src/album/album.controller.ts +++ b/backend/src/album/album.controller.ts @@ -12,27 +12,23 @@ export class AlbumController { @Get() findAll(): Promise { - console.log("AlbumController findAll") return this.albumService.findAll(); } @Get(':id') findOneById(@Param('id') id: number): Promise { - console.log("AlbumController findOnyById: ",id); return this.albumService.findOneById(id); } @Post() @UsePipes(new ValidationPipe({ transform: true })) async create(@Body() createAlbumDto: CreateAlbumDto): Promise { - console.log("AlbumController create: ",createAlbumDto); return this.albumService.create(createAlbumDto); } @Put(':id') @UsePipes(new ValidationPipe({ transform: true })) async update(@Param('id') id: number, @Body() updateAlbumDto: UpdateAlbumDto): Promise { - console.log("AlbumController update: ",id,updateAlbumDto); return this.albumService.update(id, updateAlbumDto); } diff --git a/backend/src/album/album.service.ts b/backend/src/album/album.service.ts index 66a5ebf..e9bc854 100644 --- a/backend/src/album/album.service.ts +++ b/backend/src/album/album.service.ts @@ -43,23 +43,20 @@ export class AlbumService { } async update(id: number, updateAlbumDto: UpdateAlbumDto): Promise { - console.log(id, updateAlbumDto); if (id == updateAlbumDto.id) { const albumToUpdate = await this.albumRepository.findOneBy({ id: updateAlbumDto.id }); if (!albumToUpdate) { - console.log("Didn't find album: ", id); + console.error("AlbumService: update: Didn't find album: ", id); return null; } albumToUpdate.title = updateAlbumDto.title; albumToUpdate.artist = updateAlbumDto.artist; albumToUpdate.genre = updateAlbumDto.genre; - console.log("Saving album:", albumToUpdate); const savedAlbum = await this.albumRepository.save(albumToUpdate); - console.log("Saved album: ", savedAlbum); return savedAlbum; } else { - console.log("ids don't match", id, updateAlbumDto); + console.error("AlbumService: update: IDs do not match", id, updateAlbumDto); return null; } } diff --git a/backend/src/main.ts b/backend/src/main.ts index 3bfd4b1..53ed861 100644 --- a/backend/src/main.ts +++ b/backend/src/main.ts @@ -4,7 +4,7 @@ import { AppModule } from './app.module'; async function bootstrap() { const app = await NestFactory.create(AppModule, { - logger: ['error', 'log', 'warn'], + logger: ['error', 'warn'], }); app.enableCors({ origin: ['https://music.anatid.net'], diff --git a/backend/src/song/song.service.ts b/backend/src/song/song.service.ts index 546dfbd..afb4543 100644 --- a/backend/src/song/song.service.ts +++ b/backend/src/song/song.service.ts @@ -46,8 +46,10 @@ export class SongService { if (id == updateSongDto.id) { const album = await this.albumRepository.findOneBy({id: updateSongDto.albumId}); const songToUpdate = await this.songRepository.findOneBy({ id: updateSongDto.id }); - if (!songToUpdate || !album) + if (!songToUpdate || !album) { + console.error("SongService: update: Song or Album not found"); return null; + } songToUpdate.id = updateSongDto.id; songToUpdate.title = updateSongDto.title; @@ -57,6 +59,7 @@ export class SongService { const song = await this.songRepository.save(songToUpdate); return song; } else { + console.error("SongService: update: IDs do not match"); return null; } } diff --git a/frontend/src/app/actions.tsx b/frontend/src/app/actions.tsx index ae417d9..dc4e078 100644 --- a/frontend/src/app/actions.tsx +++ b/frontend/src/app/actions.tsx @@ -150,7 +150,7 @@ export async function updateSong(formData: FormData) { const title = formData.get('title'); const duration = formData.get('duration'); const trackNumber = formData.get('trackNumber'); - return fetch(`https://api.anatid.net/song/${id}`, { + return fetch(`https://api.anatid.net/song/${album-id}`, { method: "PUT", headers: { "Content-Type": "application/json" }, body: JSON.stringify({