From d3b1ea899624bfc9c7c0c9bffbcf29badb269c8e Mon Sep 17 00:00:00 2001 From: Clement FERRERE Date: Tue, 9 Aug 2022 14:15:10 +0200 Subject: [PATCH] review de code --- .../agence-add/agence-add.component.html | 12 ++-- .../agence/agence-add/agence-add.component.ts | 6 +- .../agence-edit/agence-edit.component.html | 12 ++-- .../agence-edit/agence-edit.component.ts | 6 +- src/app/components/agence/agence.component.ts | 10 +-- .../businessunit-add.component.html | 6 +- .../businessunit-add.component.ts | 2 +- .../businessunit-edit.component.html | 6 +- .../businessunit-edit.component.ts | 2 +- .../businessunit/businessunit.component.ts | 4 +- .../collaborateur-add.component.ts | 14 ++-- .../collaborateur-edit.component.html | 72 +++++++++---------- .../collaborateur-edit.component.ts | 10 +-- .../collaborateur.component.html | 51 ++++++------- .../collaborateur/collaborateur.component.ts | 5 +- src/app/components/home/home.component.scss | 1 - src/app/components/home/home.component.ts | 12 ++-- .../periode-essai-add.component.html | 20 +++--- .../periode-essai-add.component.ts | 8 +-- .../periode-essai-edit.component.html | 18 ++--- .../periode-essai-edit.component.ts | 12 ++-- .../periode-essai/periode-essai.component.ts | 16 ++--- .../referencement-add.component.ts | 4 +- .../referencement-edit.component.ts | 6 +- .../referencement/referencement.component.ts | 6 +- src/app/services/collaborateur.service.ts | 1 - 26 files changed, 157 insertions(+), 165 deletions(-) diff --git a/src/app/components/agence/agence-add/agence-add.component.html b/src/app/components/agence/agence-add/agence-add.component.html index f3a9d5e..59fc093 100644 --- a/src/app/components/agence/agence-add/agence-add.component.html +++ b/src/app/components/agence/agence-add/agence-add.component.html @@ -4,22 +4,22 @@
- -
-
Le nom d'une agence est obligatoire
+ +
+
Le nom d'une agence est obligatoire
- -
-
Vous devez choisir la business unit à laquelle appartient +
+
Vous devez choisir la business unit à laquelle appartient l'agence
diff --git a/src/app/components/agence/agence-add/agence-add.component.ts b/src/app/components/agence/agence-add/agence-add.component.ts index 953a49a..55bb850 100644 --- a/src/app/components/agence/agence-add/agence-add.component.ts +++ b/src/app/components/agence/agence-add/agence-add.component.ts @@ -28,19 +28,19 @@ export class AgenceAddComponent implements OnInit { ) { } - get f() { + get form() { return this.registerForm.controls; } ngOnInit(): void { - this.getBusinessUnits(); + this.initBusinessUnits(); this.registerForm = this.formBuilder.group({ name: ['', Validators.required], businessUnitId: ['', Validators.required] }); } - getBusinessUnits() { + initBusinessUnits() { this.businessUnitService.getBusinessunits() .subscribe(businessunits => this.businessUnits = businessunits); } diff --git a/src/app/components/agence/agence-edit/agence-edit.component.html b/src/app/components/agence/agence-edit/agence-edit.component.html index 2c97cf2..f843e0d 100644 --- a/src/app/components/agence/agence-edit/agence-edit.component.html +++ b/src/app/components/agence/agence-edit/agence-edit.component.html @@ -8,23 +8,23 @@
- -
-
Le nom d'une agence est obligatoire
+
+
Le nom d'une agence est obligatoire
- -
-
Vous devez choisir la business unit à laquelle appartient +
+
Vous devez choisir la business unit à laquelle appartient l'agence
diff --git a/src/app/components/agence/agence-edit/agence-edit.component.ts b/src/app/components/agence/agence-edit/agence-edit.component.ts index a1f7fa6..50b4980 100644 --- a/src/app/components/agence/agence-edit/agence-edit.component.ts +++ b/src/app/components/agence/agence-edit/agence-edit.component.ts @@ -36,12 +36,12 @@ export class AgenceEditComponent implements OnInit { this.id = Number(this.route.snapshot.paramMap.get('id')) } - get f() { + get form() { return this.registerForm.controls; } async ngOnInit() { - this.getBusinessUnits() + this.initBusinessUnits() this.registerForm = this.formBuilder.group({ name: ['', Validators.required], businessUnitId: ['', Validators.required] @@ -50,7 +50,7 @@ export class AgenceEditComponent implements OnInit { this.agence = await this.agenceObservable.pipe(take(1)).toPromise() } - getBusinessUnits() { + initBusinessUnits() { this.businessUnitService.getBusinessunits() .subscribe(businessunits => this.businessUnits = businessunits); } diff --git a/src/app/components/agence/agence.component.ts b/src/app/components/agence/agence.component.ts index 93e2f6e..0db1cbe 100644 --- a/src/app/components/agence/agence.component.ts +++ b/src/app/components/agence/agence.component.ts @@ -24,16 +24,16 @@ export class AgenceComponent implements OnInit { ngOnInit(): void { - this.getAgences(); - this.getBusinessUnits(); + this.initAgences(); + this.initBusinessUnits(); } - getAgences(): void { + initAgences(): void { this.agenceService.getAgences() .subscribe(agences => this.agences = agences); } - getBusinessUnits(): void { + initBusinessUnits(): void { this.businessUnitService.getBusinessunits() .subscribe(bus => this.businessUnits = bus); } @@ -41,7 +41,7 @@ export class AgenceComponent implements OnInit { getBusinessUnitById(id: number): Businessunit { let bu = {} as Businessunit; this.businessUnits.forEach(c => { - if (c.id == id) { + if (c.id === id) { bu = c; return; } diff --git a/src/app/components/businessunit/businessunit-add/businessunit-add.component.html b/src/app/components/businessunit/businessunit-add/businessunit-add.component.html index e8f418a..3e0633f 100644 --- a/src/app/components/businessunit/businessunit-add/businessunit-add.component.html +++ b/src/app/components/businessunit/businessunit-add/businessunit-add.component.html @@ -4,9 +4,9 @@
- -
-
Le nom d'une business unit est obligatoire
+ +
+
Le nom d'une business unit est obligatoire
diff --git a/src/app/components/businessunit/businessunit-add/businessunit-add.component.ts b/src/app/components/businessunit/businessunit-add/businessunit-add.component.ts index 3cdb975..bb8b018 100644 --- a/src/app/components/businessunit/businessunit-add/businessunit-add.component.ts +++ b/src/app/components/businessunit/businessunit-add/businessunit-add.component.ts @@ -25,7 +25,7 @@ export class BusinessunitAddComponent implements OnInit { ) { } - get f() { + get form() { return this.registerForm.controls; } diff --git a/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.html b/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.html index b3a2e8b..91b4adc 100644 --- a/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.html +++ b/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.html @@ -8,9 +8,9 @@
- -
-
Le nom d'une business unit est obligatoire
+ +
+
Le nom d'une business unit est obligatoire
diff --git a/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.ts b/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.ts index f2d6286..75cf386 100644 --- a/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.ts +++ b/src/app/components/businessunit/businessunit-edit/businessunit-edit.component.ts @@ -32,7 +32,7 @@ export class BusinessunitEditComponent implements OnInit { this.id = Number(this.route.snapshot.paramMap.get('id')) } - get f() { + get form() { return this.registerForm.controls; } diff --git a/src/app/components/businessunit/businessunit.component.ts b/src/app/components/businessunit/businessunit.component.ts index 9486219..f8c9a13 100644 --- a/src/app/components/businessunit/businessunit.component.ts +++ b/src/app/components/businessunit/businessunit.component.ts @@ -25,10 +25,10 @@ export class BusinessunitComponent implements OnInit { ngOnInit(): void { - this.getBusinessunits(); + this.initBusinessunits(); } - getBusinessunits(): void { + initBusinessunits(): void { this.businessunitService.getBusinessunits() .subscribe(businessunits => this.businessunits = businessunits); } diff --git a/src/app/components/collaborateur/collaborateur-add/collaborateur-add.component.ts b/src/app/components/collaborateur/collaborateur-add/collaborateur-add.component.ts index 40c618c..7177702 100644 --- a/src/app/components/collaborateur/collaborateur-add/collaborateur-add.component.ts +++ b/src/app/components/collaborateur/collaborateur-add/collaborateur-add.component.ts @@ -22,7 +22,7 @@ export class CollaborateurAddComponent implements OnInit { errorValue: number = 0; registerForm!: FormGroup; - submitted = false; + submitted : boolean = false; constructor(private http: HttpClient, private businessunitService: BusinessunitService, @@ -37,8 +37,8 @@ export class CollaborateurAddComponent implements OnInit { } ngOnInit(): void { - this.getCollaborateurs(); - this.getBusinessunits(); + this.initCollaborateurs(); + this.initBusinessunits(); this.registerForm = this.formBuilder.group({ name: ['', Validators.required], firstName: ['', Validators.required], @@ -55,17 +55,17 @@ export class CollaborateurAddComponent implements OnInit { }); } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } - getBusinessunits(): void { + initBusinessunits(): void { this.businessunitService.getBusinessunits() .subscribe(businessunits => this.businessUnits = businessunits); } - onSubmit() { + onSubmit(): void { this.submitted = true; if (this.registerForm.invalid) { return; @@ -84,7 +84,7 @@ export class CollaborateurAddComponent implements OnInit { } } - onReset() { + onReset():void { this.submitted = false; this.registerForm.reset(); } diff --git a/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.html b/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.html index 1f71c3a..3b98ab0 100644 --- a/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.html +++ b/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.html @@ -9,37 +9,37 @@
- -
-
Le nom d'un collaborateur est obligatoire
+
+
Le nom d'un collaborateur est obligatoire
- -
-
Le prénom d'un collaborateur est obligatoire
+
+
Le prénom d'un collaborateur est obligatoire
- -
-
La date de naissance d'un collaborateur est obligatoire
+
+
La date de naissance d'un collaborateur est obligatoire
- -
-
Vous devez préciser le genre du collaborateur
+
+
Vous devez préciser le genre du collaborateur
@@ -58,7 +58,7 @@
- -
-
Vous devez préciser le statut du collaborateur
+
+
Vous devez préciser le statut du collaborateur
- -
-
Le nombre d'enfant d'un collaborateur est obligatoire
+
+
Le nombre d'enfant d'un collaborateur est obligatoire
- -
-
L'adresse d'un collaborateur est obligatoire
+
+
L'adresse d'un collaborateur est obligatoire
- -
-
Le numéro de téléphone d'un collaborateur est obligatoire
+
+
Le numéro de téléphone d'un collaborateur est obligatoire
@@ -108,13 +108,13 @@ (pas Apside)
- -
-
Le mail personnel d'un collaborateur est obligatoire
-
Le mail personnel d'un collaborateur doit être au format +
+
Le mail personnel d'un collaborateur est obligatoire
+
Le mail personnel d'un collaborateur doit être au format "exemple@nom.de.domaine"
@@ -122,13 +122,13 @@
- -
-
Le mail Apside d'un collaborateur est obligatoire
-
Le mail Apside d'un collaborateur doit être au format +
+
Le mail Apside d'un collaborateur est obligatoire
+
Le mail Apside d'un collaborateur doit être au format "exemple@apside-groupe.com" ou "exemple@apside.fr"
@@ -141,7 +141,7 @@
@@ -153,13 +153,13 @@
- -
-
Vous devez préciser la Business Unit du collaborateur
+
+
Vous devez préciser la Business Unit du collaborateur
diff --git a/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.ts b/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.ts index 71a124c..de8fa35 100644 --- a/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.ts +++ b/src/app/components/collaborateur/collaborateur-edit/collaborateur-edit.component.ts @@ -39,13 +39,13 @@ export class CollaborateurEditComponent implements OnInit { this.id = Number(this.route.snapshot.paramMap.get('id')) } - get f() { + get form() { return this.registerForm.controls; } async ngOnInit() { - this.getCollaborateurs() - this.getBusinessunits(); + this.initCollaborateurs() + this.initBusinessunits(); this.registerForm = this.formBuilder.group({ name: ['', Validators.required], firstName: ['', Validators.required], @@ -64,12 +64,12 @@ export class CollaborateurEditComponent implements OnInit { this.collaborateur = await this.collaborateurObservable.pipe(take(1)).toPromise() } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } - getBusinessunits(): void { + initBusinessunits(): void { this.businessunitService.getBusinessunits() .subscribe(businessunits => this.businessUnits = businessunits); } diff --git a/src/app/components/collaborateur/collaborateur.component.html b/src/app/components/collaborateur/collaborateur.component.html index 3f87a6d..f0a68ec 100644 --- a/src/app/components/collaborateur/collaborateur.component.html +++ b/src/app/components/collaborateur/collaborateur.component.html @@ -1,33 +1,28 @@ -
+

Collaborateurs

-

Collaborateurs

- -
-

Pour ajouter un nouveau collaborateur, cliquez ici :

- -
- - -
- - - - - - - - +
+

Pour ajouter un nouveau collaborateur, cliquez ici :

+ +
- - - - - - - - -
NomPrénomMail Apside
{{collaborateur.name}} {{collaborateur.firstName}} {{collaborateur.apsideMail}} Modifier
-
+
+ + + + + + + + + + + + + + + + +
NomPrénomMail Apside
{{collaborateur.name}} {{collaborateur.firstName}} {{collaborateur.apsideMail}} Modifier
diff --git a/src/app/components/collaborateur/collaborateur.component.ts b/src/app/components/collaborateur/collaborateur.component.ts index 1a5f339..7a2597c 100644 --- a/src/app/components/collaborateur/collaborateur.component.ts +++ b/src/app/components/collaborateur/collaborateur.component.ts @@ -20,11 +20,10 @@ export class CollaborateurComponent implements OnInit { } ngOnInit(): void { - this.getCollaborateurs(); - + this.initCollaborateurs(); } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } diff --git a/src/app/components/home/home.component.scss b/src/app/components/home/home.component.scss index d317285..5e5b4d4 100644 --- a/src/app/components/home/home.component.scss +++ b/src/app/components/home/home.component.scss @@ -1,5 +1,4 @@ .home-card { - //width: 25%; margin-right: 20px; margin-left: 20px; height: fit-content; diff --git a/src/app/components/home/home.component.ts b/src/app/components/home/home.component.ts index fda21ca..ab52bc7 100644 --- a/src/app/components/home/home.component.ts +++ b/src/app/components/home/home.component.ts @@ -44,7 +44,7 @@ export class HomeComponent implements OnInit { ) { } - getDateJoursRestant() { + getDateJoursRestant(): Date { let result = new Date(); result.setDate(new Date().getDate() + this.joursRestant); return result; @@ -71,7 +71,7 @@ export class HomeComponent implements OnInit { }) }); }); - this.getPeriodeEssais(); + this.initPeriodeEssais(); this.businessUnitService.getBusinessunits().subscribe( businessUnits => { this.businessUnits = businessUnits @@ -87,7 +87,7 @@ export class HomeComponent implements OnInit { getCollaborateurById(id: number): Collaborateur { let collab = {} as Collaborateur; this.collaborateurs.forEach(c => { - if (c.id == id) { + if (c.id === id) { collab = c; return; } @@ -96,7 +96,7 @@ export class HomeComponent implements OnInit { return collab; } - getPeriodeEssais() { + initPeriodeEssais():void { this.periodeEssaisEnCours = [] this.periodeEssaiService.getPeriodeEssais() .subscribe(periodeEssais => { @@ -119,13 +119,13 @@ export class HomeComponent implements OnInit { incrementJoursRestant() { this.joursRestant += 7; - this.getPeriodeEssais(); + this.initPeriodeEssais(); } decrementJoursRestant() { if (this.joursRestant >= 7) { this.joursRestant -= 7; - this.getPeriodeEssais(); + this.initPeriodeEssais(); } } } diff --git a/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.html b/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.html index 5bbaaf6..180ae68 100644 --- a/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.html +++ b/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.html @@ -4,14 +4,14 @@
- -
-
Vous devez choisir le collaborateur concerné par cette période +
+
Vous devez choisir le collaborateur concerné par cette période d'essai
@@ -20,22 +20,22 @@
- -
-
La date de début d'une période d'essai est obligatoire
+
+
La date de début d'une période d'essai est obligatoire
-
-
La date de fin prévue d'une période d'essai est obligatoire +
+
La date de fin prévue d'une période d'essai est obligatoire
@@ -51,7 +51,7 @@
- +
diff --git a/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.ts b/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.ts index 7cc0946..eb2c0d3 100644 --- a/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.ts +++ b/src/app/components/periode-essai/periode-essai-add/periode-essai-add.component.ts @@ -35,7 +35,7 @@ export class PeriodeEssaiAddComponent implements OnInit { ) { } - get f() { + get form() { return this.registerForm.controls; } @@ -50,17 +50,17 @@ export class PeriodeEssaiAddComponent implements OnInit { startingDate: ['', Validators.required] }); - this.getCollaborateurs(); + this.initCollaborateurs(); } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => { this.collaborateurs = collaborateurs; if (this.apsideMail != undefined) { this.collaborateurs.forEach(collab => { - if (this.apsideMail == collab.apsideMail) { + if (this.apsideMail === collab.apsideMail) { this.registerForm.get("collaborateurId")?.setValue(collab.id) this.collaborateur = collab } diff --git a/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.html b/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.html index 9bb0985..26854df 100644 --- a/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.html +++ b/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.html @@ -7,15 +7,15 @@
- -
-
Vous devez choisir le collaborateur concerné par cette période +
+
Vous devez choisir le collaborateur concerné par cette période d'essai
@@ -25,12 +25,12 @@
- -
-
La date de début d'une période d'essai est obligatoire
+
+
La date de début d'une période d'essai est obligatoire
@@ -38,13 +38,13 @@
-
-
La date de fin prévue d'une période d'essai est obligatoire +
+
La date de fin prévue d'une période d'essai est obligatoire
diff --git a/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.ts b/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.ts index 4421e23..eeb1650 100644 --- a/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.ts +++ b/src/app/components/periode-essai/periode-essai-edit/periode-essai-edit.component.ts @@ -40,13 +40,13 @@ export class PeriodeEssaiEditComponent implements OnInit { this.id = Number(this.route.snapshot.paramMap.get('id')) } - get f() { + get form() { return this.registerForm.controls; } async ngOnInit() { - this.getCollaborateurs(); - this.getPeriodeEssais(); + this.initCollaborateurs(); + this.initPeriodeEssais(); this.registerForm = this.formBuilder.group({ comment: [], collaborateurId: ['', Validators.required], @@ -59,12 +59,12 @@ export class PeriodeEssaiEditComponent implements OnInit { this.periodeEssai = await this.periodeEssaiObservable.pipe(take(1)).toPromise() } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } - getPeriodeEssais() { + initPeriodeEssais() { this.periodeEssaiService.getPeriodeEssais() .subscribe(periodeEssais => this.periodeEssais = periodeEssais); } @@ -94,7 +94,7 @@ export class PeriodeEssaiEditComponent implements OnInit { } else { this.periodeEssai.issue = this.registerForm.value.issue; } - if (this.registerForm.value.realEndingDate == undefined || this.registerForm.value.realEndingDate == '') { + if (this.registerForm.value.realEndingDate == undefined || this.registerForm.value.realEndingDate === '') { this.periodeEssai.realEndingDate = null; } else { this.periodeEssai.realEndingDate = this.registerForm.value.realEndingDate; diff --git a/src/app/components/periode-essai/periode-essai.component.ts b/src/app/components/periode-essai/periode-essai.component.ts index a051674..5e7a811 100644 --- a/src/app/components/periode-essai/periode-essai.component.ts +++ b/src/app/components/periode-essai/periode-essai.component.ts @@ -28,17 +28,17 @@ export class PeriodeEssaiComponent implements OnInit { } ngOnInit(): void { - this.getCollaborateurs(); - this.getPeriodeEssais(); + this.initCollaborateurs(); + this.initPeriodeEssais(); } onIsOldChange() { this.isOld = !this.isOld; - this.initPeriodeEssai(); + this.definePeriodeEssais(); } - initPeriodeEssai() { + definePeriodeEssais() { if (this.isOld) { this.periodeEssais = this.periodeEssaisPassees; } else { @@ -46,7 +46,7 @@ export class PeriodeEssaiComponent implements OnInit { } } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } @@ -54,7 +54,7 @@ export class PeriodeEssaiComponent implements OnInit { getCollaborateurById(id: number): Collaborateur { let collab = {} as Collaborateur; this.collaborateurs.forEach(c => { - if (c.id == id) { + if (c.id === id) { collab = c; return; } @@ -63,7 +63,7 @@ export class PeriodeEssaiComponent implements OnInit { return collab; } - getPeriodeEssais() { + initPeriodeEssais() { this.periodeEssaiService.getPeriodeEssais() .subscribe(periodeEssais => { periodeEssais.forEach(pe => { @@ -75,7 +75,7 @@ export class PeriodeEssaiComponent implements OnInit { }) this.periodeEssaisEnCours.sort((b, a) => new Date(b.plannedEndingDate).getTime() - new Date(a.plannedEndingDate).getTime()); this.periodeEssaisPassees.sort((a, b) => new Date(b.plannedEndingDate).getTime() - new Date(a.plannedEndingDate).getTime()); - this.initPeriodeEssai(); + this.definePeriodeEssais(); }); } diff --git a/src/app/components/referencement/referencement-add/referencement-add.component.ts b/src/app/components/referencement/referencement-add/referencement-add.component.ts index ad1ce68..1ccb3d1 100644 --- a/src/app/components/referencement/referencement-add/referencement-add.component.ts +++ b/src/app/components/referencement/referencement-add/referencement-add.component.ts @@ -37,7 +37,7 @@ export class ReferencementAddComponent implements OnInit { } ngOnInit(): void { - this.getCollaborateurs(); + this.initCollaborateurs(); this.registerForm = this.formBuilder.group({ referredId: ['', Validators.required], referrerId: ['', Validators.required], @@ -52,7 +52,7 @@ export class ReferencementAddComponent implements OnInit { } } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } diff --git a/src/app/components/referencement/referencement-edit/referencement-edit.component.ts b/src/app/components/referencement/referencement-edit/referencement-edit.component.ts index 8edcb6f..bc455d7 100644 --- a/src/app/components/referencement/referencement-edit/referencement-edit.component.ts +++ b/src/app/components/referencement/referencement-edit/referencement-edit.component.ts @@ -45,7 +45,7 @@ export class ReferencementEditComponent implements OnInit { } async ngOnInit() { - this.getCollaborateurs(); + this.initCollaborateurs(); this.registerForm = this.formBuilder.group({ referredId: ['', Validators.required], referrerId: ['', Validators.required], @@ -58,7 +58,7 @@ export class ReferencementEditComponent implements OnInit { } - getCollaborateurs(): void { + initCollaborateurs(): void { this.collaborateurService.getCollaborateurs() .subscribe(collaborateurs => this.collaborateurs = collaborateurs); } @@ -73,7 +73,7 @@ export class ReferencementEditComponent implements OnInit { this.referencement.referrerId = this.registerForm.value.referrerId; this.referencement.startingDate = this.registerForm.value.startingDate; - if (this.registerForm.value.endingDate == undefined || this.registerForm.value.endingDate == '') { + if (this.registerForm.value.endingDate == undefined || this.registerForm.value.endingDate === '') { this.referencement.endingDate = null; } else { this.referencement.endingDate = this.registerForm.value.endingDate; diff --git a/src/app/components/referencement/referencement.component.ts b/src/app/components/referencement/referencement.component.ts index 7831b2d..f37b0f4 100644 --- a/src/app/components/referencement/referencement.component.ts +++ b/src/app/components/referencement/referencement.component.ts @@ -37,14 +37,14 @@ export class ReferencementComponent implements OnInit { this.referencements = referencements; this.referencements.forEach(ref => { - if (this.idReferredTab.find(id => ref.id == id) == undefined) { + if (this.idReferredTab.find(id => ref.id === id) == undefined) { this.idReferredTab.push(ref.referredId); } }); this.collaborateurs.forEach(collab => { - if (this.idReferredTab.find(id => collab.id == id) == undefined) { + if (this.idReferredTab.find(id => collab.id === id) == undefined) { this.collaborateursSansRef.push(collab); } }) @@ -60,7 +60,7 @@ export class ReferencementComponent implements OnInit { getCollaborateurById(id: number): Collaborateur { let collab = {} as Collaborateur; this.collaborateurs.forEach(c => { - if (c.id == id) { + if (c.id === id) { collab = c; return; } diff --git a/src/app/services/collaborateur.service.ts b/src/app/services/collaborateur.service.ts index 4e4d0f7..d9ba5c6 100644 --- a/src/app/services/collaborateur.service.ts +++ b/src/app/services/collaborateur.service.ts @@ -28,7 +28,6 @@ export class CollaborateurService { updateCollaborateur(collaborateur: Collaborateur): Observable { let body = JSON.stringify(collaborateur); return this.http.put(this.collaborateursUrl + "/" + collaborateur.id, body, this.httpOptions); - } deleteCollaborateur(collaborateur: Collaborateur) {