diff --git a/Sources/Trek12_API/Controllers/PlayerController.cs b/Sources/Trek12_API/Controllers/PlayerController.cs index eaca2af..5cddb12 100644 --- a/Sources/Trek12_API/Controllers/PlayerController.cs +++ b/Sources/Trek12_API/Controllers/PlayerController.cs @@ -72,26 +72,25 @@ namespace Trek12_API.Controllers return Ok(playerToCreate?.toDTO()); } - [HttpDelete] - public async Task Delete(PlayerDTO player) + [HttpDelete(Name= "DeletePlayerById")] + public async Task Delete(int idPlayer) { - var playerToDelete = player.toModel(); - //faire recherche pour voir si player existe - await playersManager.DeleteItem(playerToDelete); - return Ok(); - } + var playerToDelete = await playersManager.GetItemsById(idPlayer); + if (playerToDelete == null) + { + return NotFound("Joueur non trouvé"); + } - [HttpDelete("pseudo/{pseudo}")] - public async Task DeleteByPseudo(PlayerDTO player) - { - var playerToDelete = player.toModel(); - //faire recherche pour voir si player existe - await playersManager.DeleteItem(playerToDelete); - return Ok(); + if (!await playersManager.DeleteItem(playerToDelete.SingleOrDefault(p => p.Id == idPlayer))) + { + return BadRequest("Erreur lors de la suppression du joueur"); + } + + return Ok("Joueur bien supprimé"); } - [HttpPut(Name= "UpdatePlayer")] + [HttpPut(Name= "UpdatePlayerById")] public async Task Update(int id, PlayerDTO newPlayer) { await playersManager.UpdateItem(playersManager.GetItems(0,1).Result.FirstOrDefault(), newPlayer.toModel());