Code Cleanup

pull/2/head
Clement FERRERE 3 years ago
parent 34a5c4783f
commit 4c7647ac27
  1. 25
      Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs
  2. 4
      Collaborateur_Epa_Back/espacecollab.backend.appservices/CollaborateursServices.cs
  3. 6
      Collaborateur_Epa_Back/espacecollab.backend.appservices/GenericsServices.cs
  4. 2
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/BusinessUnitSqlDto.cs
  5. 2
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/CollaborateurAppartientBusinessUnitSqlDto.cs
  6. 4
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/CollaborateurEstFonctionSqlDto.cs
  7. 2
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/FonctionSqlDto.cs
  8. 4
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/SiteDeveloppeProjetSqlDto.cs
  9. 4
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql.dtos/SiteSqlDto.cs
  10. 2
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql/Repository/CollaborateurSqlRepository.cs
  11. 2
      Collaborateur_Epa_Back/espacecollab.backend.infrastructure.sql/Repository/GenericSqlRepository.cs

@ -6,7 +6,7 @@ namespace espacecollab.backend.api.Controllers
{ {
[Route("api/[controller]")] [Route("api/[controller]")]
[ApiController] [ApiController]
public class CollaborateursController : ControllerBase public class CollaborateursController : ControllerBase
{ {
private CollaborateursServices CollaborateursServices { get; } private CollaborateursServices CollaborateursServices { get; }
@ -15,14 +15,12 @@ namespace espacecollab.backend.api.Controllers
CollaborateursServices = collaborateursServices; CollaborateursServices = collaborateursServices;
} }
// GET: api/collaborateurs
[HttpGet] [HttpGet]
public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateurs() public ActionResult<IEnumerable<CollaborateurApiDto>> GetCollaborateurs()
{ {
return Ok(CollaborateursServices.GetAll()); return Ok(CollaborateursServices.GetAll());
} }
// GET: api/collaborateurs/{collaborateurId}
[HttpGet("{collaborateurId}")] [HttpGet("{collaborateurId}")]
public ActionResult<CollaborateurApiDto> GetCollaborateurById(int collaborateurId) public ActionResult<CollaborateurApiDto> GetCollaborateurById(int collaborateurId)
{ {
@ -61,7 +59,7 @@ namespace espacecollab.backend.api.Controllers
return Problem(); return Problem();
return Ok(updatedCollaborateur); return Ok(updatedCollaborateur);
} }
[HttpGet("businessunit/{businessId}")] [HttpGet("businessunit/{businessId}")]
@ -93,22 +91,5 @@ namespace espacecollab.backend.api.Controllers
return Ok(collaborateur); return Ok(collaborateur);
} }
//public IEnumerable<CollaborateurApiDto> GetCollaborateursByBusinessUnit(int businessUnitId)
//{
// return CollaborateurRepository.GetCollaborateursByBusinessUnit(businessUnitId).Select(collaborateurSql => collaborateurSql.ToApi());
//}
//public IEnumerable<CollaborateurApiDto> GetCollaborateursByReferrer(int referrerId)
//{
// return CollaborateurRepository.GetCollaborateursByReferrer(referrerId).Select(collaborateurSql => collaborateurSql.ToApi());
//}
//public CollaborateurApiDto? GetCollaborateurByApsideMail(string apsideMail)
//{
// if (string.IsNullOrEmpty(apsideMail))
// return null;
// return CollaborateurRepository.GetCollaborateurByApsideMail(apsideMail).ToApi();
//}
} }
} }

@ -10,7 +10,7 @@ namespace espacecollab.backend.appservices
private ICollaborateurRepository CollaborateurRepository { get; } private ICollaborateurRepository CollaborateurRepository { get; }
public CollaborateursServices(ICollaborateurRepository collaborateurRepository) public CollaborateursServices(ICollaborateurRepository collaborateurRepository)
:base(collaborateurRepository, CollaborateurMapper.ToApi, CollaborateurMapper.ToSql) : base(collaborateurRepository, CollaborateurMapper.ToApi, CollaborateurMapper.ToSql)
{ {
CollaborateurRepository = collaborateurRepository; CollaborateurRepository = collaborateurRepository;
} }
@ -30,7 +30,7 @@ namespace espacecollab.backend.appservices
if (string.IsNullOrEmpty(apsideMail)) if (string.IsNullOrEmpty(apsideMail))
return null; return null;
return CollaborateurRepository.GetCollaborateurByApsideMail(apsideMail)?.ToApi(); return CollaborateurRepository.GetCollaborateurByApsideMail(apsideMail)?.ToApi();
} }
} }
} }

@ -44,15 +44,15 @@ namespace espacecollab.backend.appservices
} }
public TO? Update(TO apiDto) public TO? Update(TO apiDto)
{ {
if(GetById(apiDto.Id)==null) if (GetById(apiDto.Id) == null)
{ {
return null; return null;
} }
T sqlDto = MapperToSqlDto(apiDto); T sqlDto = MapperToSqlDto(apiDto);
T? sqlDtoValidation = GenericRepository.Update(sqlDto); T? sqlDtoValidation = GenericRepository.Update(sqlDto);
if (sqlDtoValidation == null) if (sqlDtoValidation == null)
return null; return null;
return MapperToApiDto(sqlDtoValidation); return MapperToApiDto(sqlDtoValidation);

@ -22,6 +22,6 @@ namespace espacecollab.backend.infrastructure.sql.dtos
AgenceId = agenceId; AgenceId = agenceId;
} }
} }
} }

@ -19,6 +19,6 @@ namespace espacecollab.backend.infrastructure.sql.dtos
BusinessUnitId = businessUnitId; BusinessUnitId = businessUnitId;
} }
} }
} }

@ -2,7 +2,7 @@
namespace espacecollab.backend.infrastructure.sql.dtos namespace espacecollab.backend.infrastructure.sql.dtos
{ {
public class CollaborateurEstFonctionSqlDto public class CollaborateurEstFonctionSqlDto
{ {
public int CollaborateurId { get; set; } public int CollaborateurId { get; set; }
public int FonctionId { get; set; } public int FonctionId { get; set; }
@ -18,6 +18,6 @@ namespace espacecollab.backend.infrastructure.sql.dtos
FonctionId = fonctionId; FonctionId = fonctionId;
} }
} }
} }

@ -19,6 +19,6 @@ namespace espacecollab.backend.infrastructure.sql.dtos
Name = name; Name = name;
} }
} }
} }

@ -18,7 +18,7 @@ namespace espacecollab.backend.infrastructure.sql.dtos
ProjetId = projetId; ProjetId = projetId;
} }
} }
} }

@ -23,7 +23,7 @@ namespace espacecollab.backend.infrastructure.sql.dtos
BusinessUnitId = businessUnitId; BusinessUnitId = businessUnitId;
} }
} }
} }

@ -4,7 +4,7 @@ using espacecollab.backend.infrastructure.sql.dtos;
namespace espacecollab.backend.infrastructure.sql.Repository namespace espacecollab.backend.infrastructure.sql.Repository
{ {
public class CollaborateurSqlRepository : GenericSqlRepository<CollaborateurSqlDto>, ICollaborateurRepository public class CollaborateurSqlRepository : GenericSqlRepository<CollaborateurSqlDto>, ICollaborateurRepository
{ {
private MainDbContext Context { get; } private MainDbContext Context { get; }
public CollaborateurSqlRepository(MainDbContext context) : base(context) public CollaborateurSqlRepository(MainDbContext context) : base(context)

@ -12,7 +12,7 @@ namespace espacecollab.backend.infrastructure.sql.Repository
Context = context; Context = context;
} }
public virtual T? Add(T entity) public virtual T? Add(T entity)
{ {
return Context.Set<T>().Add(entity) as T; return Context.Set<T>().Add(entity) as T;
} }

Loading…
Cancel
Save