Merge branch '536-lois-d-ouvrages-les-champs-ne-sont-pas-vide-a-l-ajout-d-un-ouvrage' into 'devel'

Resolve "Lois d'ouvrages: les champs ne sont pas vide à l'ajout d'un ouvrage"

Closes #536

See merge request !136

e2e/ouvrages-empty-fields.e2e-spec.ts

@@ -163,4 +163,23 @@ describe("ngHyd - check that created/cloned structures have empty fields - ", ()
         const emptys = [true, true, true];
         await checkFields(inputIds, emptys);
     });
+
+    it("when a structure is modified (input) and then a structure is added", async () => {
+        await setup();
+
+        // fill 
+        const inp = calcPage.getInputById("0_ZDV");
+        await inp.clear();
+        await inp.sendKeys("1");
+
+        // copy structure
+        const addStruct = calcPage.getAddStructureButton();
+        await addStruct.click();
+        await browser.sleep(200);
+
+        // check empty fields
+        const inputIds = ["1_ZDV", "1_L", "1_W", "1_CdGR"];
+        const emptys = [true, true, true, false];
+        await checkFields(inputIds, emptys);
+    });
 });

src/app/components/fieldset-container/fieldset-container.component.ts

@@ -75,15 +75,15 @@ export class FieldsetContainerComponent implements DoCheck, AfterViewInit {
      * dans un nouveau fieldset
      */
     private addSubNub(after: FieldSet, clone: boolean = false) {
-        const prms = after.backupParameters();
+        const prms = clone ? after.backupParameters() : undefined;
         const newFs = this._container.addFromTemplate(0, after.indexAsKid());
         if (clone) {
             // replace in-place to change properties (overkill)
             // @WTF why only those two ?
             newFs.setPropValue("structureType", after.properties.getPropValue("structureType"));
             newFs.setPropValue("loiDebit", after.properties.getPropValue("loiDebit"));
+            newFs.restoreParameters(prms);
         }
-        newFs.restoreParameters(prms);
     }
 
     private onFieldsetListChange() {