|
1
|
|
|
2
|
1 : BACK
|
|
3
|
Problème :
|
|
4
|
pagination return booléen dans le cas est vide
|
|
5
|
files :getReportData/ReportService
|
|
6
|
getAll/SettingsService
|
|
7
|
|
|
8
|
Retourner false peut entraîner un comportement inattendu côté frontend.
|
|
9
|
==>La pagination doit toujours retourner une réponse structurée (même vide), pas un booléen.
|
|
10
|
|
|
11
|
Cette condition est incorrecte
|
|
12
|
if (totalItemCount === 0) {
|
|
13
|
return false;
|
|
14
|
}
|
|
15
|
|
|
16
|
FRONT
|
|
17
|
Problème :
|
|
18
|
La classe .custom-tab existe dans plusieurs fichiers SCSS :
|
|
19
|
|
|
20
|
company-compensation.scss
|
|
21
|
details/company-compensation.scss
|
|
22
|
report.scss
|
|
23
|
|
|
24
|
Avoir le même nom de classe (.custom-tab) défini dans différents fichiers peut entraîner des conflits et des problèmes de style inattendus.
|
|
25
|
|
|
26
|
Recommandation :
|
|
27
|
Pour éviter ces conflits, envisagez de :
|
|
28
|
|
|
29
|
Utiliser des noms de classe uniques en fonction du contexte
|
|
30
|
(ex. : .company-custom-tab, .details-custom-tab, .report-custom-tab).
|
|
31
|
Appliquer la méthodologie BEM (Block-Element-Modifier) pour une meilleure structuration
|
|
32
|
(ex. : .company-compensation__tab, .report__custom-tab).
|
|
33
|
|
|
34
|
|
|
35
|
Problème :
|
|
36
|
L'utilisation de styles en ligne dans :
|
|
37
|
<mat-tab-group style="background-color: white; text-align: right"> peut entraîner :
|
|
38
|
L'utilisation de styles en ligne (style="") n'est pas une bonne pratique
|
|
39
|
car elle complique la maintenance et peut
|
|
40
|
entraîner des incohérences avec les styles définis dans les fichiers SCSS
|
|
41
|
|
|
42
|
Recommandation :
|
|
43
|
Au lieu d'utiliser des styles en ligne, définissez-les dans un fichier SCSS ou bien utiliser CSS Flexbox(flex justify-center , ....)
|