@open3cl/engine 1.9.0 → 1.9.1

package/4_ventilation.js

@@ -38,6 +38,16 @@ export function isLigneQ4paMemePeriode(rowQ4paConv, cg) {
     .includes(String(cg.enum_periode_construction_id));
 }
 
+/**
+ * Vrai si la ligne de la table `q4pa_conv` considère les surfaces isolées (`isolation_surfaces` = 1)
+ * alors que le recalcul à partir des parois les donne non isolées (`isolation_surfaces` = 0).
+ */
+export function isLigneQ4paSurfacesIsolees(rowQ4paConv, isolationSurfacesRecalculee) {
+  return (
+    String(rowQ4paConv.isolation_surfaces) === '1' && String(isolationSurfacesRecalculee) === '0'
+  );
+}
+
 function tv_q4pa_conv(di, de, cg, mur_list, ph_list, porte_list, bv_list) {
   const surfaces = mur_list.concat(ph_list);
   const surface_isolee = surfaces.reduce((acc, s) => {
@@ -108,10 +118,18 @@ function tv_q4pa_conv(di, de, cg, mur_list, ph_list, porte_list, bv_list) {
        * 1948, surfaces isolées », 2, alors que toutes les parois sont non isolées → ligne 1, 4,6).
        * Si la ligne saisie correspond à la période de construction du DPE, on la reprend en entier
        * (isolation et joints) pour reproduire le calcul du logiciel.
+       *
+       * Sens unique : la reprise ne s'applique que si la ligne indique des surfaces isolées alors que
+       * le recalcul les donne non isolées. Dans le sens inverse (ligne « non isolées », recalcul
+       * « isolées »), le DPE a pu être modifié par l'appelant (scénario d'isolation des parois) :
+       * la valeur recalculée est utilisée.
        * @see : Methode_de_calcul_3CL_DPE_2021-338.pdf - §4
        */
       if (rowQ4paConv.isolation_surfaces !== isolation_surfaces) {
-        if (isLigneQ4paMemePeriode(rowQ4paConv, cg)) {
+        if (
+          isLigneQ4paSurfacesIsolees(rowQ4paConv, isolation_surfaces) &&
+          isLigneQ4paMemePeriode(rowQ4paConv, cg)
+        ) {
           console.error(
             `Calcul de hperm pour la ventilation ${de.description}. Le DPE considère isolation_surfaces = ${rowQ4paConv.isolation_surfaces} \n          alors que ce devrait être isolation_surfaces = ${isolation_surfaces}. La ligne q4pa_conv ${de.tv_q4pa_conv_id} du DPE est conservée.`
           );

package/4_ventilation.spec.js

@@ -23,7 +23,8 @@ const {
   default: calc_ventilation,
   calc_hperm,
   presenceJointNonRenseignee,
-  isLigneQ4paMemePeriode
+  isLigneQ4paMemePeriode,
+  isLigneQ4paSurfacesIsolees
 } = await import('./4_ventilation.js');
 const { tv } = await import('./utils.js');
 const { default: calc_pvent } = await import('./5_conso_ventilation.js');
@@ -522,25 +523,88 @@ describe('calc_ventilation - compatibilité bug_for_bug_compat activée', () =>
       errorSpy.mockRestore();
     });
 
-    test('ligne 12 (non isolé, avec joints) reprise en entier, malgré des baies sans joint', async () => {
+    test('ligne isolée de la même période, joints absents de la ligne : joints « 0 » repris', async () => {
+      const { calc_ventilation_bug, tvBug } = await chargerAvecBug(ROW_10);
+      vi.spyOn(console, 'error').mockImplementation(() => {});
+      const baieAvecJoint = [{ donnee_entree: { surface_totale_baie: 10, presence_joint: 1 } }];
+
+      calc_ventilation_bug(vtLigne(10), CG, 'th', 80, 100, murNonIsole, [], [], baieAvecJoint);
+
+      // la ligne reprise prime sur les joints recalculés ('1')
+      expect(tvBug).toHaveBeenLastCalledWith(
+        'q4pa_conv',
+        expect.objectContaining({ isolation_surfaces: '1', presence_joints_menuiserie: '0' })
+      );
+    });
+
+    test('ligne isolée de la même période avec joints : joints « 1 » repris malgré des baies sans joint', async () => {
       const { calc_ventilation_bug, tvBug } = await chargerAvecBug({
-        q4pa_conv: '2.5',
-        tv_q4pa_conv_id: '12',
-        enum_periode_construction_id: '1',
-        isolation_surfaces: '0',
+        ...ROW_10,
+        tv_q4pa_conv_id: '11',
         presence_joints_menuiserie: '1'
       });
       vi.spyOn(console, 'error').mockImplementation(() => {});
-      const { murs } = surfacesIsolantes();
 
-      calc_ventilation_bug(vtLigne(12), CG, 'th', 80, 100, murs, [], [], baieSansJoint);
+      calc_ventilation_bug(vtLigne(11), CG, 'th', 80, 100, murNonIsole, [], [], baieSansJoint);
 
       expect(tvBug).toHaveBeenLastCalledWith(
         'q4pa_conv',
-        expect.objectContaining({ isolation_surfaces: '0', presence_joints_menuiserie: '1' })
+        expect.objectContaining({ isolation_surfaces: '1', presence_joints_menuiserie: '1' })
       );
     });
 
+    /**
+     * Sens unique : ligne « non isolées » alors que les parois sont isolées (cas d'un scénario de
+     * rénovation où l'appelant isole les murs du DPE). La ligne n'est pas reprise : l'isolation et
+     * les joints sont recalculés comme avant #312.
+     */
+    describe('ligne non isolée alors que les parois sont isolées (sens inverse)', () => {
+      const ROW_12 = {
+        q4pa_conv: '2.5',
+        tv_q4pa_conv_id: '12',
+        enum_periode_construction_id: '1',
+        isolation_surfaces: '0',
+        presence_joints_menuiserie: '1'
+      };
+
+      test('isolation recalculée utilisée et écart des joints signalé', async () => {
+        const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+        const { calc_ventilation_bug, tvBug } = await chargerAvecBug(ROW_12);
+        const { murs } = surfacesIsolantes();
+
+        calc_ventilation_bug(vtLigne(12), CG, 'th', 80, 100, murs, [], [], baieSansJoint);
+
+        expect(tvBug).toHaveBeenLastCalledWith(
+          'q4pa_conv',
+          expect.objectContaining({ isolation_surfaces: '1', presence_joints_menuiserie: '0' })
+        );
+        expect(errorSpy).toHaveBeenCalledWith(
+          expect.stringContaining('La valeur recalculée isolation_surfaces = 1 est utilisée')
+        );
+        expect(errorSpy).toHaveBeenCalledWith(
+          expect.stringContaining(
+            'La valeur recalculée presence_joints_menuiserie = 0 est utilisée'
+          )
+        );
+        expect(errorSpy).not.toHaveBeenCalledWith(expect.stringContaining('est conservée'));
+        errorSpy.mockRestore();
+      });
+
+      test('presence_joint non exporté : seuls les joints de la ligne du DPE sont repris', async () => {
+        vi.spyOn(console, 'error').mockImplementation(() => {});
+        const { calc_ventilation_bug, tvBug } = await chargerAvecBug(ROW_12);
+        const { murs } = surfacesIsolantes();
+        const baies = [{ donnee_entree: { surface_totale_baie: 10 } }];
+
+        calc_ventilation_bug(vtLigne(12), CG, 'th', 80, 100, murs, [], [], baies);
+
+        expect(tvBug).toHaveBeenLastCalledWith(
+          'q4pa_conv',
+          expect.objectContaining({ isolation_surfaces: '1', presence_joints_menuiserie: '1' })
+        );
+      });
+    });
+
     test('ligne d’une autre période : la valeur recalculée est conservée', async () => {
       const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
       const { calc_ventilation_bug, tvBug } = await chargerAvecBug({
@@ -612,3 +676,19 @@ describe('isLigneQ4paMemePeriode', () => {
     expect(isLigneQ4paMemePeriode(null, { enum_periode_construction_id: '1' })).toBe(false);
   });
 });
+
+describe('isLigneQ4paSurfacesIsolees', () => {
+  test('ligne « surfaces isolées » et recalcul « non isolées » : vrai', () => {
+    expect(isLigneQ4paSurfacesIsolees({ isolation_surfaces: '1' }, '0')).toBe(true);
+    expect(isLigneQ4paSurfacesIsolees({ isolation_surfaces: 1 }, 0)).toBe(true);
+  });
+
+  test('ligne « non isolées » et recalcul « isolées » : faux (sens inverse)', () => {
+    expect(isLigneQ4paSurfacesIsolees({ isolation_surfaces: '0' }, '1')).toBe(false);
+  });
+
+  test('valeurs concordantes ou ligne sans critère : faux', () => {
+    expect(isLigneQ4paSurfacesIsolees({ isolation_surfaces: '1' }, '1')).toBe(false);
+    expect(isLigneQ4paSurfacesIsolees({}, '0')).toBe(false);
+  });
+});

package/engine.js

@@ -35,7 +35,7 @@ import DpeSanitizerService from './dpe-sanitizer.service.js';
 import { reconstituerRdimDpeGenereImmeuble } from './17.2_rdim_dpe_genere_immeuble.js';
 import { reconstituerRdimEcsIndividuelleUnique } from './11_rdim_ecs_individuelle_unique.js';
 
-const LIB_VERSION = '1.9.0';
+const LIB_VERSION = '1.9.1';
 
 const dpeSanitizerService = new DpeSanitizerService();
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open3cl/engine",
-  "version": "1.9.0",
+  "version": "1.9.1",
   "description": "Open Source 3CL-DPE engine",
   "main": "index.js",
   "directories": {