@ -158,6 +158,7 @@ namespace API_LoL.Controllers
var champ = await ChampionsManager . GetItemsByName ( champion . Name , 0 , 1 ) ;
if ( champ . Count ( ) ! = 0 & & champ . FirstOrDefault ( ) . Name = = champion . Name )
{
_logger . LogError ( message : "Champion with this id already exists" , "Post" , "/Champion" , 409 , "champion : " + champion . toString , DateTime . Now ) ;
return Conflict ( champion ) ;
}
await ChampionsManager . AddItem ( champion . ToChampion ( ) ) ;
@ -170,10 +171,16 @@ namespace API_LoL.Controllers
[HttpPut("name")]
public async Task < IActionResult > Put ( string name , ChampionDTO championDTO )
{
if ( string . IsNullOrEmpty ( name ) )
if ( string . IsNullOrEmpty ( name ) )
{
_logger . LogError ( message : "Null paramater given for Name" , "Put" , "/Champion/Name" , 400 , "name : " + name , "champion : " + championDTO . toString , DateTime . Now ) ;
return BadRequest ( ) ;
}
if ( championDTO = = null )
{
_logger . LogError ( message : "Null paramater given for Champion" , "Put" , "/Champion/Name" , 422 , "name : " + name , "champion : " + championDTO . toString , DateTime . Now ) ;
return UnprocessableEntity ( ) ;
}
var list = await ChampionsManager . GetItemsByName ( name , 0 , 1 ) ;
if ( list . Count ( ) = = 1 )
{
@ -187,7 +194,10 @@ namespace API_LoL.Controllers
public async Task < IActionResult > Delete ( string name )
{
if ( string . IsNullOrEmpty ( name ) )
return BadRequest ( ) ;
{
_logger . LogError ( message : "Null paramater given for Name" , "Delete" , "/Champion/Name" , 400 , "name : " + name , DateTime . Now ) ;
return BadRequest ( ) ;
}
var list = await ChampionsManager . GetItemsByName ( name , 0 , 1 ) ;
if ( list . Count ( ) = = 1 ) {
return Ok ( await ChampionsManager . DeleteItem ( list . First ( ) ) ) ;