fixup! feat(metadata): add possibility to fill af form from query params

Author: VincentCauchois

frontend/src/app/metadataModule/actors/actors.component.ts

@@ -1,6 +1,6 @@
 import { Component, OnInit, Input, Output, EventEmitter } from '@angular/core';
 import { UntypedFormArray, UntypedFormGroup, Validators } from '@angular/forms';
-import { BehaviorSubject } from 'rxjs';
+import { BehaviorSubject, combineLatest, Subject } from 'rxjs';
 import { MatDialog } from '@angular/material/dialog';
 
 import { ConfirmationDialog } from '@geonature_common/others/modal-confirmation/confirmation.dialog';
@@ -102,6 +102,13 @@ export class ActorComponent implements OnInit {
     }
 
     this.setToggleButtonValue();
+
+    combineLatest([
+      this.actorForm.get('id_organism').valueChanges,
+      this.actorForm.get('id_role').valueChanges,
+    ]).subscribe(([id_organism, id_role]) => {
+      this.setToggleButtonValue();
+    });
   }
 
   toggleActorOrganismChoiceChange(event) {

frontend/src/app/metadataModule/af/af-form.component.ts

@@ -1,4 +1,4 @@
-import { Component, OnInit } from '@angular/core';
+import { AfterViewInit, Component, OnInit } from '@angular/core';
 import { UntypedFormArray, UntypedFormGroup, Validators } from '@angular/forms';
 import { Router, ActivatedRoute } from '@angular/router';
 import { NgbDateParserFormatter } from '@ng-bootstrap/ng-bootstrap';
@@ -20,7 +20,7 @@ import { TranslateService } from '@librairies/@ngx-translate/core';
   styleUrls: ['../form.component.scss'],
   providers: [AcquisitionFrameworkFormService],
 })
-export class AfFormComponent implements OnInit {
+export class AfFormComponent implements OnInit, AfterViewInit {
   public form: UntypedFormGroup;
   //observable pour la liste déroulantes HTML des AF parents
   public acquisitionFrameworkParents: Observable<any>;

frontend/src/app/metadataModule/services/af-form.service.ts

@@ -1,6 +1,6 @@
 import { Injectable } from '@angular/core';
 import { UntypedFormGroup, UntypedFormArray, UntypedFormBuilder, Validators } from '@angular/forms';
-import { BehaviorSubject, forkJoin, Observable, of } from 'rxjs';
+import { BehaviorSubject, combineLatest, forkJoin, Observable, of } from 'rxjs';
 import { tap, filter, switchMap, map } from 'rxjs/operators';
 
 import { ActorFormService } from './actor-form.service';
@@ -15,7 +15,6 @@ export class AcquisitionFrameworkFormService {
   // Custom additional fields
   public additionalFieldsForm: Array<any> = [];
   public queryParamsSource: BehaviorSubject<Object> = new BehaviorSubject({});
-  private queryParams$: Observable<Object> = this.queryParamsSource.asObservable();
 
   constructor(
     private fb: UntypedFormBuilder,
@@ -106,9 +105,8 @@ export class AcquisitionFrameworkFormService {
    **/
   private setObservables() {
     //Observable de this.dataset pour adapter le formulaire selon la donnée
-    this.acquisition_framework
-      .asObservable()
-      .pipe(
+    combineLatest([
+      this.acquisition_framework.asObservable().pipe(
         tap(() => this.reset()),
         tap(() => {
           this.additionalFieldsForm = [];
@@ -156,8 +154,12 @@ export class AcquisitionFrameworkFormService {
         }),
         // Map to return acquisition framework data only
         map(([acquisition_framework, additional_data]) => acquisition_framework)
-      )
-      .subscribe((value: any) => this.form.patchValue(value));
+      ),
+      this.queryParamsSource,
+    ]).subscribe(([value, params]) => {
+      setTimeout(() => this.form.patchValue(value), 0);
+      this.setFromParams(params);
+    });
 
     //gère lactivation/désactivation de la zone de saisie du framework Parent
     this.form.get('is_parent').valueChanges.subscribe((value: boolean) => {
@@ -167,11 +169,6 @@ export class AcquisitionFrameworkFormService {
         this.form.get('acquisition_framework_parent_id').enable();
       }
     });
-
-    // Update form from query params on every update of the latter
-    this.queryParams$.subscribe((params) => {
-      this.setFromParams(params);
-    });
   }
 
   setFromParams(params: any) {
@@ -211,17 +208,19 @@ export class AcquisitionFrameworkFormService {
         (field) => field.paramName == key
       );
       if (keyAsAdditionalField) {
-        this.form.get('additional_data').get(keyAsAdditionalField.fieldName).setValue(params[key]);
+        setTimeout(() => {
+          this.form
+            .get('additional_data')
+            .patchValue({ [keyAsAdditionalField.fieldName]: params[key] });
+        }, 0);
       }
       if (key == 'id_role') {
-        this.form.get('cor_af_actor').patchValue([{ id_role: params[key] }]);
+        this.form.get('cor_af_actor').patchValue([{ id_role: +params[key] }]);
       }
       if (key == 'id_organism') {
-        this.form.get('cor_af_actor').patchValue([{ id_organism: params[key] }]);
+        this.form.get('cor_af_actor').patchValue([{ id_organism: +params[key] }]);
       }
     });
-    
-    console.log(this.form.value);
   }
 
   get actors(): UntypedFormArray {