fix: add optional chaining(?) to manage undefined id case in setValue() and...

fix: add optional chaining(?) to manage undefined id case in setValue() and rename cancelValue function() to emptySelectedList
Refs #637

src/app/components/generic-select/generic-select.component.html

@@ -11,7 +11,7 @@
         </mat-option>
     </mat-select>
     <button mat-button *ngIf="showClearButton" matSuffix mat-icon-button aria-label="Clear"
-        (click)="cancelValue($event)">
+        (click)="emptySelectedList($event)">
         <mat-icon>close</mat-icon>
     </button>
     <div *ngIf="enableHelpButton" class="overlap-select">

src/app/components/select-field-line/select-field-line.component.ts

@@ -59,15 +59,14 @@ export class SelectFieldLineComponent implements OnInit {
         return this._select.getValue();
     }
 
-    public onValueChange($event) {
+    public onValueChange($event: any) {
         this.selectedValue = $event.value
         this.valueChanged.emit(this.selectedValue)
     }
 
-    public cancelValue($event): any {
+    public emptySelectedList($event:any) {
         this.selectedValue = [];
         this.valueChanged.emit(this.selectedValue)
-        // this.validChange.emit(false)
         $event.stopPropagation();
     }
 

src/app/formulaire/elements/select/select-field.ts

@@ -82,7 +82,7 @@ export abstract class SelectField extends Field {
                 || (
                     !Array.isArray(previousSelectedEntry)
                     && !Array.isArray(v)
-                    && previousSelectedEntry.id !== v.id
+                    && previousSelectedEntry?.id !== v?.id
                 )
                 || (
                     Array.isArray(previousSelectedEntry)