From f495bd019e300426aa254efded472ef40bfc7b0f Mon Sep 17 00:00:00 2001 From: Clement FERRERE Date: Mon, 6 Dec 2021 10:28:35 +0100 Subject: [PATCH] Fix "Utilisation des type unsigned dans la couche API + AppSvc" --- .../Controllers/CollaborateursController.cs | 10 +++++----- .../CollaborateurApiDto.cs | 4 ++-- .../Interfaces/IGenericIdSqlDto.cs | 2 +- .../Mappers/CollaborateurApiDtoMapper.cs | 4 ++-- .../IGenericsServices.cs | 2 +- .../CollaborateursServices.cs | 8 ++++---- .../GenericsServices.cs | 10 +++++----- 7 files changed, 20 insertions(+), 20 deletions(-) diff --git a/Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs b/Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs index f74c0cd..df9e5ac 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.api/Controllers/CollaborateursController.cs @@ -22,7 +22,7 @@ namespace espacecollab.backend.api.Controllers } [HttpGet("{collaborateurId:int:min(1)}")] - public ActionResult GetCollaborateurById(int collaborateurId) + public ActionResult GetCollaborateurById(uint collaborateurId) { CollaborateurApiDto? collaborateur = CollaborateursServices.GetById(collaborateurId); if (collaborateur == null) @@ -42,7 +42,7 @@ namespace espacecollab.backend.api.Controllers } [HttpDelete("{collaborateurId:int:min(1)}")] - public ActionResult DeleteCollaborateur(int collaborateurId) + public ActionResult DeleteCollaborateur(uint collaborateurId) { bool isCollaborateurDeleted = CollaborateursServices.Delete(collaborateurId); if (!isCollaborateurDeleted) @@ -52,7 +52,7 @@ namespace espacecollab.backend.api.Controllers } [HttpPut("update/{collaborateurId:int:min(1)}")] - public ActionResult UpdateCollaborateur(int collaborateurId, CollaborateurApiDto collaborateurApi) + public ActionResult UpdateCollaborateur(uint collaborateurId, CollaborateurApiDto collaborateurApi) { if (collaborateurApi.Id != collaborateurId) return Unauthorized(); @@ -65,7 +65,7 @@ namespace espacecollab.backend.api.Controllers } [HttpGet("businessunit/{businessUnitId:int:min(1)}")] - public ActionResult> GetCollaborateursByBusinessUnit(int businessUnitId) + public ActionResult> GetCollaborateursByBusinessUnit(uint businessUnitId) { IEnumerable collaborateurs = CollaborateursServices.GetCollaborateursByBusinessUnit(businessUnitId); if (collaborateurs == null) @@ -75,7 +75,7 @@ namespace espacecollab.backend.api.Controllers } [HttpGet("referrer/{referrerId:int:min(1)}")] - public ActionResult> GetCollaborateursByReferrer(int referrerId) + public ActionResult> GetCollaborateursByReferrer(uint referrerId) { IEnumerable collaborateurs = CollaborateursServices.GetCollaborateursByReferrer(referrerId); if (collaborateurs == null) diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/CollaborateurApiDto.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/CollaborateurApiDto.cs index 7b5ac19..af539c5 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/CollaborateurApiDto.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/CollaborateurApiDto.cs @@ -4,7 +4,7 @@ namespace espacecollab.backend.appservices.dtos { public class CollaborateurApiDto : IGenericIdApiDto { - public int Id { get; set; } + public uint Id { get; set; } public string Name { get; set; } public string FirstName { get; set; } public DateTime BirthDate { get; set; } @@ -23,7 +23,7 @@ namespace espacecollab.backend.appservices.dtos { } - public CollaborateurApiDto(int id, string name, string firstName, DateTime birthDate, EnumGenreApi gender, EnumStatutApi status, int childrenNumber, string address, string telephone, string personalMail, string apsideMail, DateTime resignationDate, int referrerId, int businessUnitId) + public CollaborateurApiDto(uint id, string name, string firstName, DateTime birthDate, EnumGenreApi gender, EnumStatutApi status, int childrenNumber, string address, string telephone, string personalMail, string apsideMail, DateTime resignationDate, int referrerId, int businessUnitId) { Id = id; Name = name; diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Interfaces/IGenericIdSqlDto.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Interfaces/IGenericIdSqlDto.cs index af0c0ec..167465b 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Interfaces/IGenericIdSqlDto.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Interfaces/IGenericIdSqlDto.cs @@ -2,6 +2,6 @@ { public interface IGenericIdApiDto { - public int Id { get; set; } + public uint Id { get; set; } } } diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Mappers/CollaborateurApiDtoMapper.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Mappers/CollaborateurApiDtoMapper.cs index 7a72dab..b604f55 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Mappers/CollaborateurApiDtoMapper.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices.dtos/Mappers/CollaborateurApiDtoMapper.cs @@ -6,7 +6,7 @@ namespace espacecollab.backend.appservices.dtos.Mappers { public static CollaborateurApiDto ToApi(this CollaborateurSqlDto collaborateurSql) { - return new CollaborateurApiDto(collaborateurSql.Id, collaborateurSql.Name, collaborateurSql.FirstName, + return new CollaborateurApiDto((uint)collaborateurSql.Id, collaborateurSql.Name, collaborateurSql.FirstName, collaborateurSql.BirthDate, collaborateurSql.Gender.ToEnumGenreApi(), collaborateurSql.Status.ToEnumStatutApi(), collaborateurSql.ChildrenNumber, collaborateurSql.Address, collaborateurSql.Telephone, collaborateurSql.PersonalMail, collaborateurSql.ApsideMail, collaborateurSql.ResignationDate, collaborateurSql.ReferrerId, collaborateurSql.BusinessUnitId); @@ -14,7 +14,7 @@ namespace espacecollab.backend.appservices.dtos.Mappers public static CollaborateurSqlDto ToSql(this CollaborateurApiDto collaborateurApi) { - return new CollaborateurSqlDto(collaborateurApi.Id, collaborateurApi.Name, collaborateurApi.FirstName, + return new CollaborateurSqlDto((int)collaborateurApi.Id, collaborateurApi.Name, collaborateurApi.FirstName, collaborateurApi.BirthDate, collaborateurApi.Gender.ToEnumGenreSql(), collaborateurApi.Status.ToEnumStatutSql(), collaborateurApi.ChildrenNumber, collaborateurApi.Address, collaborateurApi.Telephone, collaborateurApi.PersonalMail, collaborateurApi.ApsideMail, collaborateurApi.ResignationDate, collaborateurApi.ReferrerId, collaborateurApi.BusinessUnitId); diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices.interfaces/IGenericsServices.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices.interfaces/IGenericsServices.cs index 83402d2..5f14f76 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices.interfaces/IGenericsServices.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices.interfaces/IGenericsServices.cs @@ -8,7 +8,7 @@ namespace espacecollab.backend.appservices.interfaces IEnumerable GetAll(); - T? GetById(int id); + T? GetById(uint id); T? Update(T apiDto); } diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices/CollaborateursServices.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices/CollaborateursServices.cs index 388ff77..f9039c7 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices/CollaborateursServices.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices/CollaborateursServices.cs @@ -15,14 +15,14 @@ namespace espacecollab.backend.appservices CollaborateurRepository = collaborateurRepository; } - public IEnumerable GetCollaborateursByBusinessUnit(int businessUnitId) + public IEnumerable GetCollaborateursByBusinessUnit(uint businessUnitId) { - return CollaborateurRepository.GetCollaborateursByBusinessUnit(businessUnitId).Select(collaborateurSql => collaborateurSql.ToApi()); + return CollaborateurRepository.GetCollaborateursByBusinessUnit((int)businessUnitId).Select(collaborateurSql => collaborateurSql.ToApi()); } - public IEnumerable GetCollaborateursByReferrer(int referrerId) + public IEnumerable GetCollaborateursByReferrer(uint referrerId) { - return CollaborateurRepository.GetCollaborateursByReferrer(referrerId).Select(collaborateurSql => collaborateurSql.ToApi()); + return CollaborateurRepository.GetCollaborateursByReferrer((int)referrerId).Select(collaborateurSql => collaborateurSql.ToApi()); } public CollaborateurApiDto? GetCollaborateurByApsideMail(string apsideMail) diff --git a/Collaborateur_Epa_Back/espacecollab.backend.appservices/GenericsServices.cs b/Collaborateur_Epa_Back/espacecollab.backend.appservices/GenericsServices.cs index 76715d8..d1c65c2 100644 --- a/Collaborateur_Epa_Back/espacecollab.backend.appservices/GenericsServices.cs +++ b/Collaborateur_Epa_Back/espacecollab.backend.appservices/GenericsServices.cs @@ -10,7 +10,7 @@ namespace espacecollab.backend.appservices { private Func MapperToApiDto { get; } private Func MapperToSqlDto { get; } - + private IGenericRepository GenericRepository { get; } protected GenericsServices(IGenericRepository genericRepository, Func mapperToApiDto, Func mapperToSqlDto) @@ -25,9 +25,9 @@ namespace espacecollab.backend.appservices return GenericRepository.GetAll().Select(entity => MapperToApiDto(entity)).ToList(); } - public TO? GetById(int id) + public TO? GetById(uint id) { - T? entity = GenericRepository.GetById(id); + T? entity = GenericRepository.GetById((int)id); if (entity == null) return null; @@ -60,9 +60,9 @@ namespace espacecollab.backend.appservices return MapperToApiDto(sqlDtoValidation); } - public bool Delete(int apiDtoId) + public bool Delete(uint apiDtoId) { - return GenericRepository.Delete(apiDtoId); + return GenericRepository.Delete((int)apiDtoId); } } }