delete_cascade_collabs à faire en 2ème #16

Merged
Clement.Ferrere merged 4 commits from delete_cascade_collabs into develop 2 years ago
There is no content yet.
Clement.Ferrere added 2 commits 2 years ago
AlexandreRuiz reviewed 2 years ago
}
[HttpDelete("{id:int:min(1)}")]
public override ActionResult Delete(uint id)

Pourquoi override la méthode alors que c'est la même ?

Pourquoi override la méthode alors que c'est la même ?

Dans l'override on utilise le CollaborateursServices, si on override pas on utilise un GenericServices de Collaborateur. Pour supprimer en cascade, j'ai modifié le CollaborateursServices pour essayer de rester cohérent dans les dépendances, et le fait d'override permet de forcer l'utilisation de ce service.

Dans l'override on utilise le CollaborateursServices, si on override pas on utilise un GenericServices de Collaborateur. Pour supprimer en cascade, j'ai modifié le CollaborateursServices pour essayer de rester cohérent dans les dépendances, et le fait d'override permet de forcer l'utilisation de ce service.
Clement.Ferrere marked this conversation as resolved
AlexandreRuiz reviewed 2 years ago
public new bool Delete(uint apiDtoId)
{
var referencements = ReferencementRepository.GetAll();

Préciser le type au lieu d'un var

Préciser le type au lieu d'un var
Clement.Ferrere marked this conversation as resolved
AlexandreRuiz reviewed 2 years ago
public new bool Delete(uint apiDtoId)
{
var referencements = ReferencementRepository.GetAll();
var periodeEssais = PeriodeEssaiRepository.GetAll();

Préciser le type au lieu d'un var

Préciser le type au lieu d'un var
Clement.Ferrere marked this conversation as resolved
Clement.Ferrere added 1 commit 2 years ago
Clement.Ferrere added 1 commit 2 years ago
Clement.Ferrere added 1 commit 2 years ago
Clement.Ferrere merged commit ccf9a24049 into develop 2 years ago
Clement.Ferrere deleted branch delete_cascade_collabs 2 years ago
The pull request has been merged as ccf9a24049.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: AlexandreRuiz/Collaborateur_Epa_Back#16
Loading…
There is no content yet.