Update inetOrgPerson DTO to include employeeType field

Author: tacxou

docs/technical/Tests.md

@@ -96,6 +96,7 @@ export const inetOrgPersonDtoStub = (): inetOrgPersonDto => {
     sn: 'sn',
     uid: 'uid',
     employeeNumber: 'employeeNumber',
+    employeeType: 'employeeType',
     displayName: 'displayName',
     facsimileTelephoneNumber: 'facsimileTelephoneNumber',
     givenName: 'givenName',

docs/user/IdentitiesCreation.md

@@ -42,13 +42,14 @@ Lors de la création d'une identité, le corps de la requête doit inclure les i
 
 #### Informations de Base (`inetOrgPerson`)
 
-Les informations de base de l'identité sont définies dans l'objet `inetOrgPerson`. Les champs requis sont `cn` (Nom Commun), `sn` (Nom de Famille), `uid` (Identifiant Unique), et `employeeNumber` (Identifiant Identité). Ces champs sont validés en dur.
+Les informations de base de l'identité sont définies dans l'objet `inetOrgPerson`. Les champs requis sont `cn` (Nom Commun), `sn` (Nom de Famille), `uid` (Identifiant Unique), et `employeeNumber` + `employeeType` (Identifiant Identité). Ces champs sont validés en dur.
 
 ##### Champs obligatoires :
 - `cn`  
 - `sn`
 - `uid`
 - `employeeNumber`
+- `employeeType`
 
 ##### Champs facultatifs :
 - `displayName`
@@ -95,6 +96,7 @@ Voici un exemple de corps de requête pour la création d'une identité avec les
     "sn": "Nom de Famille",
     "uid": "Identifiant Unique",
     "employeeNumber": "Identifiant Identité"
+    "employeeType": "Type Identité"
   },
   "additionalFields": {
     "objectClasses": ["supann"],
@@ -121,7 +123,8 @@ curl -X POST "http://<adresse-du-serveur>/identities" \
              "cn": "Nom Commun",
              "sn": "Nom de Famille",
              "uid": "Identifiant Unique",
-             "employeeNumber": "Identifiant Identité"
+             "employeeNumber": "Identifiant Identité",
+             "employeeType": "Type Identité"
            },
            "additionalFields": {
              "objectClasses": ["supann"],
@@ -163,7 +166,7 @@ Même procédure pour la validation, mais ici nous avons :
 
 ### Corps de la Requête (Body)
 
-Lors de la création d'une identité, le corps de la requête doit inclure au moins la données (`inetOrgPerson.uid`) et (`inetOrgPerson.employeeNumber`), ainsi que tout champ additionnel nécessaire selon l'`objectClass` spécifique. Notez que l'objet `inetOrgPerson` est validé en dur et non via un fichier YAML.
+Lors de la création d'une identité, le corps de la requête doit inclure au moins la données (`inetOrgPerson.uid`) et (`inetOrgPerson.employeeNumber` + `inetOrgPerson.employeeType`), ainsi que tout champ additionnel nécessaire selon l'`objectClass` spécifique. Notez que l'objet `inetOrgPerson` est validé en dur et non via un fichier YAML.
 
 De plus pour une modification, il specifier le state a `-1`
 
@@ -172,7 +175,8 @@ De plus pour une modification, il specifier le state a `-1`
     "state": -1,
     "inetOrgPerson": {
       "uid": "Identifiant Unique",
-      "employeeNumber": "Identifiant Identite"
+      "employeeNumber": "Identifiant Identite",
+      "employeeType": "Type Identite"
     },
     "additionalFields": {
       "objectClasses": ["supann"],
@@ -195,7 +199,8 @@ De plus pour une modification, il specifier le state a `-1`
              "state": -1,
              "inetOrgPerson": {
                "uid": "Identifiant Unique",
-               "employeeNumber": "Identifiant Identite"
+               "employeeNumber": "Identifiant Identite",
+               "employeeType": "Type Identite"
              },
              "additionalFields": {
                "objectClasses": ["supann"],

docs/user/IdentitiesValidation.md

@@ -14,6 +14,7 @@ Les champs de base de l'objet `inetOrgPerson` sont validés par défaut.
 - `sn`
 - `uid`
 - `employeeNumber`
+- `employeeType`
 
 ### Champs facultatifs :
 - `displayName`

src/management/identities/_dto/_parts/inetOrgPerson.dto.ts

@@ -14,6 +14,10 @@ export class inetOrgPersonCreateDto {
   @ApiProperty()
   employeeNumber: string;
 
+  @IsString()
+  @ApiProperty()
+  employeeType: string;
+
   @IsString()
   @ApiProperty()
   @IsOptional()

src/management/identities/_schemas/_parts/inetOrgPerson.part.ts

@@ -41,14 +41,20 @@ export class inetOrgPerson {
   @Prop({ required: true, unique: true })
   uid: string;
 
-  @Prop({ required: true, unique: true })
+  @Prop({ required: true })
   employeeNumber: string;
 
+  @Prop({ required: true })
+  employeeType: string;
+
   @Prop()
   userCertificate?: string;
 
   @Prop()
   userPassword?: string;
 }
 
-export const inetOrgPersonSchema = SchemaFactory.createForClass(inetOrgPerson);
+export const inetOrgPersonSchema = SchemaFactory.createForClass(inetOrgPerson).index(
+  { employeeNumber: 1, employeeType: 1 },
+  { unique: true },
+);

src/management/identities/_stubs/_parts/inetOrgPerson.dto.stub.ts

@@ -6,6 +6,7 @@ export const inetOrgPersonDtoStub = (): inetOrgPersonDto => {
     sn: 'sn',
     uid: 'uid',
     employeeNumber: 'employeeNumber',
+    employeeType: 'employeeType',
     displayName: 'displayName',
     facsimileTelephoneNumber: 'facsimileTelephoneNumber',
     givenName: 'givenName',

src/management/identities/identities.service.ts

@@ -70,7 +70,7 @@ export class IdentitiesService extends AbstractServiceSchema {
       };
     }
 
-    //TODO: rechercher par uid ou employeeNumber ?
+    //TODO: rechercher par uid ou employeeNumber + employeeType ?
     const upsert = await super.upsert({ 'inetOrgPerson.uid': data.inetOrgPerson.uid }, data, options);
     return upsert;
     //TODO: add backends service logic here

src/management/identities/jsonforms/_config/inetorgperson.ui.yml

@@ -36,6 +36,14 @@
           "options": {
             "required": true
           }
+        },
+        {
+          "type": "Control",
+          "label": "EmployeeType",
+          "scope": "#/properties/employeeType",
+          "options": {
+            "required": true
+          }
         }
       ]
     },

src/management/identities/validations/_config/inetorgperson.yml

@@ -18,6 +18,10 @@
       "type": "string",
       "description": "EmployeeNumber of the inetOrgPerson."
     },
+    "employeeType": {
+      "type": "string",
+      "description": "EmployeeType of the inetOrgPerson."
+    },
     "displayName": {
       "type": "string",
       "description": "Display name of the inetOrgPerson."
@@ -68,5 +72,5 @@
       "description": "User password of the inetOrgPerson."
     }
   },
-  "required": ["uid", "employeeNumber"]
+  "required": ["uid", "employeeNumber", "employeeType"]
 }

src/management/passwd/passwd.controller.ts

@@ -24,7 +24,7 @@ export class PasswdController {
   public async change(@Body() cpwd: ChangePasswordDto, @Res() res: Response): Promise<Response> {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const [_, data] = await this.passwdService.change(cpwd);
-    //TODO: uid ou employeeNumber ?
+    //TODO: uid ou employeeNumber + employeeType ?
     data.data.uid = cpwd.id;
     this.logger.log(`call passwd change for : ${cpwd.id}`);