@open3cl/engine 1.8.63 → 1.8.65

package/11_stockage_ecs_volume_chauffe.js

@@ -0,0 +1,31 @@
+/**
+ * Le stockage ECS d'un générateur est-il en volume chauffé ?
+ *
+ * Tribu porte l'information au niveau de l'installation (`installation_ECS.Is_volume_chauffee`,
+ * Calcul_batiment.cs l.375-377). Dans l'export ADEME, elle se lit, par ordre de priorité :
+ * 1. `position_volume_chauffe_stockage` du générateur, s'il est renseigné ;
+ * 2. `tv_rendement_distribution_ecs_id` de l'installation : 1 ou 2 (production en volume
+ *    habitable) → oui, 3 (production hors volume habitable) → non ;
+ * 3. `position_volume_chauffe` du générateur.
+ *
+ * @see Methode_de_calcul_3CL_DPE_2021-338.pdf - §9.1.1 et §11.3 (rendement de distribution)
+ * @param deGenerateur {object} donnee_entree du générateur ECS
+ * @param deInstallation {object} donnee_entree de l'installation ECS
+ * @returns {boolean}
+ */
+export function isStockageEcsVolumeChauffe(deGenerateur, deInstallation) {
+  const positionStockage = Number.parseInt(deGenerateur.position_volume_chauffe_stockage);
+  if (positionStockage === 0 || positionStockage === 1) {
+    return positionStockage === 1;
+  }
+
+  const rdId = Number.parseInt(deInstallation?.tv_rendement_distribution_ecs_id);
+  if (rdId === 1 || rdId === 2) {
+    return true;
+  }
+  if (rdId === 3) {
+    return false;
+  }
+
+  return Number.parseInt(deGenerateur.position_volume_chauffe) !== 0;
+}

package/11_stockage_ecs_volume_chauffe.spec.js

@@ -0,0 +1,35 @@
+import { describe, expect, test } from 'vitest';
+import { isStockageEcsVolumeChauffe } from './11_stockage_ecs_volume_chauffe.js';
+
+describe('isStockageEcsVolumeChauffe - stockage ECS en volume chauffé (#293)', () => {
+  /**
+   * Tribu : installation_ECS.Is_volume_chauffee (Calcul_batiment.cs l.375-377), lue dans l'export
+   * par position_volume_chauffe_stockage, puis tv_rendement_distribution_ecs_id, puis
+   * position_volume_chauffe.
+   */
+  test.each([
+    // [position stockage, rendement distribution, position générateur, attendu]
+    [1, '3', 0, true], // stockage renseigné prioritaire (oui)
+    [0, '1', 1, false], // stockage renseigné prioritaire (non)
+    ['1', undefined, 0, true], // valeur texte acceptée
+    [undefined, '1', 0, true], // production en volume habitable
+    [undefined, 2, 0, true], // production en volume habitable, pièces non contiguës
+    [undefined, '3', 1, false], // production hors volume habitable
+    [undefined, '4', 0, false], // ligne collective : repli sur la position du générateur
+    [undefined, undefined, 1, true], // repli : générateur en volume chauffé
+    [undefined, undefined, 0, false], // repli : générateur hors volume chauffé
+    [undefined, undefined, undefined, true], // aucune information : récupération conservée
+    [null, null, 0, false] // champs nuls traités comme absents
+  ])('stockage %s, rd %s, générateur %s => %s', (pvcs, rd, pvc, attendu) => {
+    expect(
+      isStockageEcsVolumeChauffe(
+        { position_volume_chauffe_stockage: pvcs, position_volume_chauffe: pvc },
+        { tv_rendement_distribution_ecs_id: rd }
+      )
+    ).toBe(attendu);
+  });
+
+  test('installation absente : repli sur la position du générateur', () => {
+    expect(isStockageEcsVolumeChauffe({ position_volume_chauffe: 0 }, undefined)).toBe(false);
+  });
+});

package/4_ventilation.js

@@ -17,6 +17,18 @@ function tv_debits_ventilation(di, de, du) {
   }
 }
 
+/**
+ * Vrai si le DPE comporte au moins une menuiserie (baie ou porte) et qu'aucune n'exporte
+ * le champ `presence_joint` (issue #294).
+ */
+export function presenceJointNonRenseignee(porte_list, bv_list) {
+  const menuiseries = (porte_list || []).concat(bv_list || []);
+  return (
+    menuiseries.length > 0 &&
+    menuiseries.every((m) => [undefined, null, ''].includes(m.donnee_entree?.presence_joint))
+  );
+}
+
 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) => {
@@ -82,14 +94,23 @@ function tv_q4pa_conv(di, de, cg, mur_list, ph_list, porte_list, bv_list) {
       if (rowQ4paConv.isolation_surfaces !== isolation_surfaces) {
         console.error(
           `Calcul de hperm pour la ventilation ${de.description}. Le DPE considère isolation_surfaces = ${rowQ4paConv.isolation_surfaces} 
-          alors que ce devrait être isolation_surfaces = ${isolation_surfaces}. La valeur du DPE isolation_surfaces = ${rowQ4paConv.isolation_surfaces} est utilisée.`
+          alors que ce devrait être isolation_surfaces = ${isolation_surfaces}. La valeur recalculée isolation_surfaces = ${isolation_surfaces} est utilisée.`
         );
       }
 
-      if (rowQ4paConv.presence_joints_menuiserie !== pjt) {
+      /**
+       * Issue #294 : aucune menuiserie n'exporte `presence_joint`. Aucune donnée ne contredit alors
+       * la ligne q4pa_conv retenue par le logiciel (ex. ligne 12 « avant 1948 avec joints », 2,5) :
+       * on reprend sa valeur `presence_joints_menuiserie` au lieu de considérer les menuiseries
+       * sans joint.
+       * @see : Methode_de_calcul_3CL_DPE_2021-338.pdf - §4
+       */
+      if (presenceJointNonRenseignee(porte_list, bv_list)) {
+        pjt = rowQ4paConv.presence_joints_menuiserie === '1' ? '1' : '0';
+      } else if (rowQ4paConv.presence_joints_menuiserie !== pjt) {
         console.error(
           `Calcul de hperm pour la ventilation ${de.description}. Le DPE considère presence_joints_menuiserie = ${rowQ4paConv.presence_joints_menuiserie} 
-          alors que ce devrait être presence_joints_menuiserie = ${pjt}. La valeur du DPE presence_joints_menuiserie = ${rowQ4paConv.presence_joints_menuiserie} est utilisée.`
+          alors que ce devrait être presence_joints_menuiserie = ${pjt}. La valeur recalculée presence_joints_menuiserie = ${pjt} est utilisée.`
         );
       }
     }

package/4_ventilation.spec.js

@@ -19,7 +19,11 @@ vi.mock('./5_conso_ventilation.js', () => ({
   default: vi.fn()
 }));
 
-const { default: calc_ventilation, calc_hperm } = await import('./4_ventilation.js');
+const {
+  default: calc_ventilation,
+  calc_hperm,
+  presenceJointNonRenseignee
+} = await import('./4_ventilation.js');
 const { tv } = await import('./utils.js');
 const { default: calc_pvent } = await import('./5_conso_ventilation.js');
 
@@ -404,4 +408,101 @@ describe('calc_ventilation - compatibilité bug_for_bug_compat activée', () =>
     expect(errorSpy).not.toHaveBeenCalled();
     errorSpy.mockRestore();
   });
+
+  /**
+   * Issue #294 : presence_joint absent sur toutes les menuiseries => la valeur
+   * presence_joints_menuiserie de la ligne q4pa_conv du DPE est reprise.
+   */
+  describe('presence_joint non exporté (issue #294)', () => {
+    /** Avant 1948, surfaces non isolées : seul le critère des joints départage les lignes 1 et 12. */
+    const murNonIsole = [
+      {
+        donnee_intermediaire: { b: 1 },
+        donnee_entree: { enum_type_isolation_id: '2', surface_paroi_opaque: 100 }
+      }
+    ];
+    const vtLigne12 = () => ({
+      donnee_entree: { enum_type_ventilation_id: '1', tv_q4pa_conv_id: 12, description: 'v' }
+    });
+    const ROW_12 = {
+      q4pa_conv: '2.5',
+      tv_q4pa_conv_id: '12',
+      isolation_surfaces: '0',
+      presence_joints_menuiserie: '1'
+    };
+
+    test('aucune menuiserie ne renseigne le champ : joints du DPE repris, sans message', async () => {
+      const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+      const { calc_ventilation_bug, tvBug } = await chargerAvecBug(ROW_12);
+      const baies = [{ donnee_entree: { surface_totale_baie: 10 } }];
+      const portes = [{ donnee_entree: { surface_porte: 2, presence_joint: null } }];
+
+      calc_ventilation_bug(vtLigne12(), CG, 'th', 80, 100, murNonIsole, [], portes, baies);
+
+      expect(tvBug).toHaveBeenLastCalledWith(
+        'q4pa_conv',
+        expect.objectContaining({ isolation_surfaces: '0', presence_joints_menuiserie: '1' })
+      );
+      expect(errorSpy).not.toHaveBeenCalled();
+      errorSpy.mockRestore();
+    });
+
+    test('ligne du DPE sans joints : la valeur "0" est conservée', async () => {
+      const { calc_ventilation_bug, tvBug } = await chargerAvecBug({
+        ...ROW_12,
+        tv_q4pa_conv_id: '1',
+        presence_joints_menuiserie: undefined
+      });
+      const baies = [{ donnee_entree: { surface_totale_baie: 10 } }];
+
+      calc_ventilation_bug(vtLigne12(), CG, 'th', 80, 100, murNonIsole, [], [], baies);
+
+      expect(tvBug).toHaveBeenLastCalledWith(
+        'q4pa_conv',
+        expect.objectContaining({ presence_joints_menuiserie: '0' })
+      );
+    });
+
+    test('presence_joint renseigné à 0 : la valeur recalculée prime et l’écart est signalé', async () => {
+      const errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+      const { calc_ventilation_bug, tvBug } = await chargerAvecBug(ROW_12);
+      const baies = [{ donnee_entree: { surface_totale_baie: 10, presence_joint: 0 } }];
+
+      calc_ventilation_bug(vtLigne12(), CG, 'th', 80, 100, murNonIsole, [], [], baies);
+
+      expect(tvBug).toHaveBeenLastCalledWith(
+        'q4pa_conv',
+        expect.objectContaining({ presence_joints_menuiserie: '0' })
+      );
+      expect(errorSpy).toHaveBeenCalledWith(
+        expect.stringContaining('La valeur recalculée presence_joints_menuiserie = 0 est utilisée')
+      );
+      errorSpy.mockRestore();
+    });
+  });
+});
+
+describe('presenceJointNonRenseignee', () => {
+  test('aucune menuiserie : faux', () => {
+    expect(presenceJointNonRenseignee([], [])).toBe(false);
+    expect(presenceJointNonRenseignee(undefined, undefined)).toBe(false);
+  });
+
+  test('champ absent, null ou vide sur toutes les menuiseries : vrai', () => {
+    expect(
+      presenceJointNonRenseignee(
+        [{ donnee_entree: { presence_joint: '' } }],
+        [{ donnee_entree: {} }, { donnee_entree: { presence_joint: null } }]
+      )
+    ).toBe(true);
+  });
+
+  test('au moins une menuiserie renseignée (même à 0) : faux', () => {
+    expect(
+      presenceJointNonRenseignee(
+        [{ donnee_entree: { presence_joint: 0 } }],
+        [{ donnee_entree: {} }]
+      )
+    ).toBe(false);
+  });
 });

package/9_besoin_ch.js

@@ -9,6 +9,7 @@ import {
 import { calc_ai_j, calc_as_j } from './6.1_apport_gratuit.js';
 import { calc_sse_j } from './6.2_surface_sud_equivalente.js';
 import { bug_for_bug_compat, mois_liste } from './utils.js';
+import { isStockageEcsVolumeChauffe } from './11_stockage_ecs_volume_chauffe.js';
 
 export default function calc_besoin_ch(
   ilpa,
@@ -67,10 +68,7 @@ export default function calc_besoin_ch(
     } else {
       Qgw = gen_ecs.reduce((acc, gen_ecs) => {
         // Pas de récupération de stockage si le ballon est hors volume chauffé
-        if (
-          gen_ecs.donnee_entree.position_volume_chauffe_stockage === 0 ||
-          gen_ecs.donnee_entree.position_volume_chauffe === 0
-        ) {
+        if (!isStockageEcsVolumeChauffe(gen_ecs.donnee_entree, instal_ecs.donnee_entree)) {
           return acc;
         }
 

package/9_besoin_ch.spec.js

@@ -313,7 +313,7 @@ describe('calc_besoin_ch - branches complémentaires', () => {
               donnee_intermediaire: { Qgw: 100 }
             },
             {
-              // générateur hors volume chauffé => ignoré
+              // générateur hors volume chauffé mais stockage en volume chauffé => compté (#293)
               donnee_entree: { position_volume_chauffe_stockage: 1, position_volume_chauffe: 0 },
               donnee_intermediaire: { Qgw: 100 }
             },
@@ -660,3 +660,60 @@ describe('calc_besoin_ch - immeuble à installations ECS individuelles et collec
     expect(ret.pertes_distribution_ecs_recup).toBeCloseTo((0.48 * 100 * total) / 8760, 9);
   });
 });
+
+describe('calc_besoin_ch - stockage ECS en volume chauffé (#293)', () => {
+  /**
+   * Cas des exemples de l'issue (2301E2484046J, 2322E2765016S) : ballon électrique LICIEL
+   * exporté avec position_volume_chauffe = 0 mais production en volume habitable (rd 1).
+   */
+  test('ballon en volume habitable exporté avec position_volume_chauffe = 0 : pertes récupérées', () => {
+    utilState.bug = false;
+    vi.mocked(calc_ai_j).mockReset().mockReturnValue(5000);
+    vi.mocked(calc_as_j).mockReset().mockReturnValue(2000);
+    vi.mocked(calc_sse_j).mockReset().mockReturnValue(10);
+    vi.mocked(calc_besoin_ecs_j).mockReset().mockReturnValue(10);
+    vi.mocked(calc_Qrec_gen_j).mockReset().mockReturnValue(0);
+    const instal = (rd) => [
+      {
+        donnee_entree: { enum_type_installation_id: '1', tv_rendement_distribution_ecs_id: rd },
+        generateur_ecs_collection: {
+          generateur_ecs: [
+            { donnee_entree: { position_volume_chauffe: 0 }, donnee_intermediaire: { Qgw: 8760 } }
+          ]
+        }
+      }
+    ];
+    const enVolume = calc_besoin_ch(
+      0,
+      0,
+      0,
+      0,
+      100,
+      100,
+      3,
+      instal('1'),
+      [],
+      [],
+      null,
+      'maison',
+      1
+    );
+    const horsVolume = calc_besoin_ch(
+      0,
+      0,
+      0,
+      0,
+      100,
+      100,
+      3,
+      instal('3'),
+      [],
+      [],
+      null,
+      'maison',
+      1
+    );
+    expect(enVolume.pertes_stockage_ecs_recup).toBeGreaterThan(0);
+    expect(horsVolume.pertes_stockage_ecs_recup).toBe(0);
+  });
+});

package/README.md

@@ -375,57 +375,57 @@ Ces résultats sont **générés automatiquement** à la fin de `npm run test:co
 
 <!-- CORPUS:START -->
 
-> **Version `1.8.62`** · branche `main` · généré le 2026-10-15
+> **Version `1.8.64`** · branche `main` · généré le 2026-10-15
 > Seuil de tolérance **5%**
 
 <table>
 <tr>
 <td align="center"><strong>9</strong><br/><sub>corpus</sub></td>
 <td align="center"><strong>89 418</strong><br/><sub>DPE analysés</sub></td>
-<td align="center"><strong>71 082</strong><br/><sub>DPE conformes</sub></td>
-<td align="center"><strong>79,49 %</strong><br/><sub>réussite globale</sub></td>
+<td align="center"><strong>71 309</strong><br/><sub>DPE conformes</sub></td>
+<td align="center"><strong>79,75 %</strong><br/><sub>réussite globale</sub></td>
 </tr>
 </table>
 
 |     | Corpus                                                                                                                  |    Réussite |                        |  DPE conformes |
 | :-: | :---------------------------------------------------------------------------------------------------------------------- | ----------: | :--------------------- | -------------: |
-| 🔴  | **Généraliste**<br/><sub>`corpus_dpe.csv`</sub>                                                                         | **47,82 %** | `██████████░░░░░░░░░░` |  4 782 / 9 999 |
-| 🟢  | **Appartement · chauffage individuel (2025)**<br/><sub>`dpe_appartement_individuel_chauffage_individuel_2025.csv`</sub> | **93,94 %** | `███████████████████░` | 9 394 / 10 000 |
-| 🟢  | **Appartement · chauffage collectif (2025)**<br/><sub>`dpe_appartement_individuel_chauffage_collectif_2025.csv`</sub>   | **93,64 %** | `███████████████████░` | 9 364 / 10 000 |
-| 🟢  | **Logement individuel (2025)**<br/><sub>`dpe_logement_individuel_2025.csv`</sub>                                        | **91,54 %** | `██████████████████░░` |  9 147 / 9 992 |
-| 🟢  | **Maison individuelle (2025)**<br/><sub>`dpe_maison_individuelle_2025.csv`</sub>                                        | **89,44 %** | `██████████████████░░` |  8 943 / 9 999 |
-| 🟢  | **Immeuble · chauffage collectif**<br/><sub>`dpe_immeuble_chauffage_collectif.csv`</sub>                                | **88,20 %** | `██████████████████░░` |  8 656 / 9 814 |
-| 🟢  | **Immeuble · chauffage individuel**<br/><sub>`dpe_immeuble_chauffage_individuel.csv`</sub>                              | **85,14 %** | `█████████████████░░░` |  8 512 / 9 998 |
-| 🟡  | **Immeuble · chauffage mixte**<br/><sub>`dpe_immeuble_chauffage_mixte.csv`</sub>                                        | **69,39 %** | `██████████████░░░░░░` |  6 673 / 9 616 |
-| 🔴  | **Individuel généré depuis l'immeuble (2026)**<br/><sub>`dpe_individuel_a_partir_dpe_immeuble_2026.csv`</sub>           | **56,11 %** | `███████████░░░░░░░░░` | 5 611 / 10 000 |
+| 🔴  | **Généraliste**<br/><sub>`corpus_dpe.csv`</sub>                                                                         | **47,91 %** | `██████████░░░░░░░░░░` |  4 791 / 9 999 |
+| 🟢  | **Appartement · chauffage individuel (2025)**<br/><sub>`dpe_appartement_individuel_chauffage_individuel_2025.csv`</sub> | **94,37 %** | `███████████████████░` | 9 437 / 10 000 |
+| 🟢  | **Appartement · chauffage collectif (2025)**<br/><sub>`dpe_appartement_individuel_chauffage_collectif_2025.csv`</sub>   | **93,84 %** | `███████████████████░` | 9 384 / 10 000 |
+| 🟢  | **Logement individuel (2025)**<br/><sub>`dpe_logement_individuel_2025.csv`</sub>                                        | **91,80 %** | `██████████████████░░` |  9 173 / 9 992 |
+| 🟢  | **Maison individuelle (2025)**<br/><sub>`dpe_maison_individuelle_2025.csv`</sub>                                        | **89,93 %** | `██████████████████░░` |  8 992 / 9 999 |
+| 🟢  | **Immeuble · chauffage collectif**<br/><sub>`dpe_immeuble_chauffage_collectif.csv`</sub>                                | **88,39 %** | `██████████████████░░` |  8 675 / 9 814 |
+| 🟢  | **Immeuble · chauffage individuel**<br/><sub>`dpe_immeuble_chauffage_individuel.csv`</sub>                              | **85,35 %** | `█████████████████░░░` |  8 533 / 9 998 |
+| 🟡  | **Immeuble · chauffage mixte**<br/><sub>`dpe_immeuble_chauffage_mixte.csv`</sub>                                        | **69,57 %** | `██████████████░░░░░░` |  6 690 / 9 616 |
+| 🔴  | **Individuel généré depuis l'immeuble (2026)**<br/><sub>`dpe_individuel_a_partir_dpe_immeuble_2026.csv`</sub>           | **56,34 %** | `███████████░░░░░░░░░` | 5 634 / 10 000 |
 
 <sub>🟢 ≥ 85 % · 🟡 ≥ 60 % · 🔴 < 60 %</sub>
 
 #### Temps d’exécution
 
-> Durée de l’appel à `calcul_3cl` par DPE, sur 89 384 calculs.
+> Durée de l’appel à `calcul_3cl` par DPE, sur 89 383 calculs.
 > La copie défensive de l’entrée et la lecture du fichier sont exclues de la mesure.
 
 <table>
 <tr>
-<td align="center"><strong>5,85 ms</strong><br/><sub>moyenne</sub></td>
-<td align="center"><strong>5,12 ms</strong><br/><sub>médiane</sub></td>
-<td align="center"><strong>0,10 ms</strong><br/><sub>min</sub></td>
-<td align="center"><strong>1,05 s</strong><br/><sub>max</sub></td>
+<td align="center"><strong>6,03 ms</strong><br/><sub>moyenne</sub></td>
+<td align="center"><strong>5,41 ms</strong><br/><sub>médiane</sub></td>
+<td align="center"><strong>0,19 ms</strong><br/><sub>min</sub></td>
+<td align="center"><strong>763,7 ms</strong><br/><sub>max</sub></td>
 </tr>
 </table>
 
 | Corpus                                         | Moyenne | Médiane |     Min |      Max |     p95 |     p99 |
 | :--------------------------------------------- | ------: | ------: | ------: | -------: | ------: | ------: |
-| **Généraliste**                                | 6,93 ms | 5,75 ms | 0,40 ms |  73,5 ms | 15,6 ms | 24,8 ms |
-| **Appartement · chauffage individuel (2025)**  | 5,03 ms | 4,64 ms | 1,66 ms |  38,8 ms | 8,73 ms | 12,6 ms |
-| **Appartement · chauffage collectif (2025)**   | 6,12 ms | 5,80 ms | 1,86 ms |  62,0 ms | 9,63 ms | 15,0 ms |
-| **Logement individuel (2025)**                 | 4,76 ms | 4,26 ms | 1,72 ms |  33,5 ms | 8,74 ms | 11,8 ms |
-| **Maison individuelle (2025)**                 | 5,58 ms | 5,02 ms | 1,86 ms |  47,1 ms | 9,86 ms | 13,1 ms |
-| **Immeuble · chauffage collectif**             | 5,90 ms | 5,14 ms | 0,17 ms | 139,6 ms | 11,0 ms | 17,6 ms |
-| **Immeuble · chauffage individuel**            | 6,51 ms | 5,59 ms | 0,10 ms |  88,8 ms | 13,0 ms | 20,7 ms |
-| **Immeuble · chauffage mixte**                 | 5,71 ms | 4,62 ms | 0,19 ms |   1,05 s | 11,9 ms | 18,8 ms |
-| **Individuel généré depuis l'immeuble (2026)** | 6,12 ms | 5,29 ms | 1,69 ms |  64,4 ms | 11,4 ms | 18,8 ms |
+| **Généraliste**                                | 5,69 ms | 5,26 ms | 0,31 ms |  43,3 ms | 9,94 ms | 14,5 ms |
+| **Appartement · chauffage individuel (2025)**  | 4,38 ms | 4,08 ms | 1,56 ms |  34,5 ms | 7,30 ms | 9,34 ms |
+| **Appartement · chauffage collectif (2025)**   | 5,28 ms | 5,15 ms | 1,71 ms |  45,6 ms | 8,16 ms | 10,4 ms |
+| **Logement individuel (2025)**                 | 5,57 ms | 5,06 ms | 1,69 ms |  39,2 ms | 10,1 ms | 13,6 ms |
+| **Maison individuelle (2025)**                 | 6,58 ms | 5,99 ms | 1,96 ms |  45,7 ms | 11,6 ms | 15,3 ms |
+| **Immeuble · chauffage collectif**             | 6,51 ms | 5,85 ms | 0,23 ms | 163,5 ms | 11,7 ms | 19,1 ms |
+| **Immeuble · chauffage individuel**            | 7,69 ms | 6,64 ms | 0,19 ms | 121,8 ms | 15,4 ms | 24,7 ms |
+| **Immeuble · chauffage mixte**                 | 5,92 ms | 4,86 ms | 0,19 ms | 763,7 ms | 12,1 ms | 20,2 ms |
+| **Individuel généré depuis l'immeuble (2026)** | 6,65 ms | 5,78 ms | 1,75 ms |  60,9 ms | 12,6 ms | 20,5 ms |
 
 <sub>La moyenne est tirée vers le haut par les DPE collectifs, dont le coût atteint plusieurs
 dizaines de fois la médiane : c’est la médiane qui décrit le cas courant, et p95/p99 le cas

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.8.63';
+const LIB_VERSION = '1.8.65';
 
 const dpeSanitizerService = new DpeSanitizerService();
 

package/features/dpe/infrastructure/tv.store.js

@@ -340,6 +340,16 @@ export class TvStore {
     return debitsVentilation;
   }
 
+  /**
+   * Ligne de la table q4pa_conv par son identifiant
+   * @param tvQ4paConvId {number|string|undefined}
+   * @return {object|undefined}
+   */
+  getQ4paConvById(tvQ4paConvId) {
+    if (tvQ4paConvId === undefined || tvQ4paConvId === null) return;
+    return tv['q4pa_conv'].find((v) => v.tv_q4pa_conv_id === String(tvQ4paConvId));
+  }
+
   /**
    * Valeur conventionnelle de la perméabilité sous 4Pa
    * @see Chapitre 4 - Calcul des déperditions par renouvellement d’air

package/features/dpe/infrastructure/tv.store.spec.js

@@ -607,6 +607,17 @@ describe('Lecture des tables de valeurs', () => {
       const ug = tvStore.getQ4paConv('2', TypeHabitation.MAISON, '0', '1');
       expect(ug).toBeUndefined();
     });
+
+    test('lecture par identifiant (issue #294)', () => {
+      expect(tvStore.getQ4paConvById(12)).toMatchObject({
+        q4pa_conv: '2.5',
+        presence_joints_menuiserie: '1'
+      });
+      expect(tvStore.getQ4paConvById('1')).toMatchObject({ q4pa_conv: '4.6' });
+      expect(tvStore.getQ4paConvById(undefined)).toBeUndefined();
+      expect(tvStore.getQ4paConvById(null)).toBeUndefined();
+      expect(tvStore.getQ4paConvById(999)).toBeUndefined();
+    });
   });
 
   test('lecture des valeurs de tempBase', () => {

package/features/engine/domain/ecs/installation-ecs.service.js

@@ -1,5 +1,6 @@
 import { GenerateurEcsService } from './generateur-ecs.service.js';
 import { inject } from 'dioma';
+import { isStockageEcsVolumeChauffe } from '../../../../11_stockage_ecs_volume_chauffe.js';
 
 /**
  * Calcul des besoins et pertes des installations ECS
@@ -142,11 +143,8 @@ export class InstallationEcsService {
     if (installationIndividuelle) {
       installationEcs.donnee_utilisateur.QgwRecuperable =
         installationEcs.generateur_ecs_collection?.generateur_ecs
-          .filter(
-            (generateurEcs) =>
-              parseInt(generateurEcs.donnee_entree.position_volume_chauffe_stockage) === 1 ||
-              (generateurEcs.donnee_entree.position_volume_chauffe_stockage === undefined &&
-                parseInt(generateurEcs.donnee_entree.position_volume_chauffe) === 1)
+          .filter((generateurEcs) =>
+            isStockageEcsVolumeChauffe(generateurEcs.donnee_entree, installationEcsDE)
           )
           .reduce((acc, generateurEcs) => {
             return acc + generateurEcs.donnee_utilisateur.Qgw;

package/features/engine/domain/enveloppe/deperdition-enveloppe.service.js

@@ -305,7 +305,8 @@ export class DeperditionEnveloppeService {
         this.#surfaceIsolee,
         this.#surfaceNonIsolee,
         this.#surfaceMenuiserieAvecJoint,
-        this.#surfaceMenuiserieSansJoint
+        this.#surfaceMenuiserieSansJoint,
+        this.#presenceJointNonRenseignee(logement.enveloppe)
       );
       deperditions.hvent += ventilation.donnee_intermediaire.hvent;
       deperditions.hperm += ventilation.donnee_intermediaire.hperm;
@@ -313,4 +314,22 @@ export class DeperditionEnveloppeService {
 
     return deperditions;
   }
+
+  /**
+   * Vrai si le logement comporte au moins une menuiserie (baie ou porte) et qu'aucune n'exporte
+   * le champ presence_joint (issue #294).
+   *
+   * @param enveloppe {Enveloppe}
+   * @return {boolean}
+   */
+  #presenceJointNonRenseignee(enveloppe) {
+    const menuiseries = [
+      ...(enveloppe?.porte_collection?.porte || []),
+      ...(enveloppe?.baie_vitree_collection?.baie_vitree || [])
+    ];
+    return (
+      menuiseries.length > 0 &&
+      menuiseries.every((m) => [undefined, null, ''].includes(m.donnee_entree?.presence_joint))
+    );
+  }
 }

package/features/engine/domain/enveloppe/deperdition-enveloppe.service.spec.js

@@ -173,7 +173,17 @@ describe('DeperditionEnveloppeService (§3 déperditions de l’enveloppe)', ()
     // déperditive = mur 10 + porte 2 + plancher haut 30 + baie 4 = 46
     // isolée = mur 5 + plancher haut 7 = 12 ; non isolée = mur 10 + plancher haut 30 = 40
     // menuiseries avec joint = porte 2 + baie 4 = 6 ; sans joint = porte 3 + baie 6 = 9
-    expect(ventilationService.execute).toHaveBeenCalledWith(ctx, { id: 'v' }, 46, 12, 40, 6, 9);
+    // presence_joint renseigné sur au moins une menuiserie => indicateur #294 à false
+    expect(ventilationService.execute).toHaveBeenCalledWith(
+      ctx,
+      { id: 'v' },
+      46,
+      12,
+      40,
+      6,
+      9,
+      false
+    );
   });
 
   test('les surfaces sont réinitialisées à chaque calcul', () => {
@@ -186,7 +196,18 @@ describe('DeperditionEnveloppeService (§3 déperditions de l’enveloppe)', ()
     service.deperditions(ctx, logement);
     service.deperditions(ctx, logement);
 
-    expect(ventilationService.execute).toHaveBeenLastCalledWith(ctx, {}, 2, 0, 0, 0, 2);
+    // aucune menuiserie ne renseigne presence_joint => indicateur #294 à true
+    expect(ventilationService.execute).toHaveBeenLastCalledWith(ctx, {}, 2, 0, 0, 0, 2, true);
+  });
+
+  test('logement sans menuiserie : indicateur presence_joint non renseigné à false (issue #294)', () => {
+    const logement = logementVide();
+    logement.ventilation_collection.ventilation = [{ donnee_entree: {} }];
+    ventilationService.execute.mockReturnValue({ hvent: 0, hperm: 0 });
+
+    service.deperditions(ctx, logement);
+
+    expect(ventilationService.execute).toHaveBeenLastCalledWith(ctx, {}, 0, 0, 0, 0, 0, false);
   });
 
   describe('espaces tampons solarisés', () => {

package/features/engine/domain/enveloppe/ventilation/deperdition-ventilation.service.js

@@ -1,6 +1,7 @@
 import { DeperditionService } from '../deperdition.service.js';
 
 import { logger } from '../../../../../core/util/logger/log-service.js';
+import { bug_for_bug_compat } from '../../../../../utils.js';
 import { TypeHabitation } from '../../../../dpe/domain/models/type-habitation.model.js';
 import { TypeVentilation } from '../../../../dpe/domain/models/type-ventilation.model.js';
 
@@ -22,6 +23,7 @@ export class DeperditionVentilationService extends DeperditionService {
    * @param surfaceNonIsolee {number}
    * @param surfaceMenuiserieAvecJoint {number}
    * @param surfaceMenuiserieSansJoint {number}
+   * @param presenceJointNonRenseignee {boolean} aucune menuiserie n'exporte presence_joint (issue #294)
    * @return {VentilationDI}
    */
   execute(
@@ -31,7 +33,8 @@ export class DeperditionVentilationService extends DeperditionService {
     surfaceIsolee,
     surfaceNonIsolee,
     surfaceMenuiserieAvecJoint,
-    surfaceMenuiserieSansJoint
+    surfaceMenuiserieSansJoint,
+    presenceJointNonRenseignee = false
   ) {
     const q4paConv = this.q4paConv(
       ventilationDE,
@@ -39,7 +42,8 @@ export class DeperditionVentilationService extends DeperditionService {
       surfaceIsolee,
       surfaceNonIsolee,
       surfaceMenuiserieAvecJoint,
-      surfaceMenuiserieSansJoint
+      surfaceMenuiserieSansJoint,
+      presenceJointNonRenseignee
     );
     const debitsVentilation = this.#debitsVentilation(ventilationDE.enum_type_ventilation_id);
 
@@ -62,6 +66,7 @@ export class DeperditionVentilationService extends DeperditionService {
    * @param surfaceNonIsolee {number}
    * @param surfaceMenuiserieAvecJoint {number}
    * @param surfaceMenuiserieSansJoint {number}
+   * @param presenceJointNonRenseignee {boolean}
    * @return {number}
    */
   q4paConv(
@@ -70,7 +75,8 @@ export class DeperditionVentilationService extends DeperditionService {
     surfaceIsolee,
     surfaceNonIsolee,
     surfaceMenuiserieAvecJoint,
-    surfaceMenuiserieSansJoint
+    surfaceMenuiserieSansJoint,
+    presenceJointNonRenseignee = false
   ) {
     // Pour les bâtiments qui ont fait l’objet d’une mesure d’étanchéité à l’air moins de deux ans avant le diagnostic, la valeur
     // mesurée de Q4Paconv/m² peut être saisie.
@@ -92,6 +98,15 @@ export class DeperditionVentilationService extends DeperditionService {
         // (si les menuiseries représentant plus de 50% de la surface totale possèdent des joints) => valeur forfaitaire.
         if (surfaceMenuiserieAvecJoint > surfaceMenuiserieSansJoint) {
           presenceJointsMenuiserie = '1';
+        } else if (
+          bug_for_bug_compat &&
+          presenceJointNonRenseignee &&
+          this.tvStore.getQ4paConvById(ventilationDE.tv_q4pa_conv_id)
+            ?.presence_joints_menuiserie === '1'
+        ) {
+          // Issue #294 : aucune menuiserie n'exporte presence_joint, rien ne contredit la ligne
+          // q4pa_conv « avant 1948 avec joints » retenue par le DPE => on la reprend.
+          presenceJointsMenuiserie = '1';
         }
       }
     } else if (enumPeriodeConstructionId === '2') {

package/features/engine/domain/enveloppe/ventilation/deperdition-ventilation.service.spec.js

@@ -0,0 +1,100 @@
+import { beforeEach, describe, expect, test, vi } from 'vitest';
+
+/** `bug_for_bug_compat` (utils.js) piloté par le test. */
+const state = vi.hoisted(() => ({ bug: false }));
+vi.mock('../../../../../utils.js', async (importOriginal) => ({
+  ...(await importOriginal()),
+  get bug_for_bug_compat() {
+    return state.bug;
+  }
+}));
+
+const { DeperditionVentilationService } = await import('./deperdition-ventilation.service.js');
+
+/**
+ * Tests unitaires isolés du calcul de Q4Paconv (`DeperditionVentilationService.q4paConv`).
+ * Le store des tables de valeurs est remplacé par un double de test.
+ *
+ * @see : Methode_de_calcul_3CL_DPE_2021-338.pdf - §4
+ */
+
+/** @type {DeperditionVentilationService} */
+let service;
+let tvStore;
+
+const ctx = { enumPeriodeConstructionId: '1', typeHabitation: 'APPARTEMENT' };
+
+describe('DeperditionVentilationService.q4paConv', () => {
+  beforeEach(() => {
+    tvStore = {
+      getQ4paConv: vi.fn().mockReturnValue({ q4pa_conv: '4.6' }),
+      getQ4paConvById: vi.fn()
+    };
+    service = new DeperditionVentilationService(tvStore);
+    state.bug = false;
+  });
+
+  test('Q4Pa mesuré saisi : utilisé directement', () => {
+    expect(service.q4paConv({ q4pa_conv_saisi: 1.7 }, ctx, 0, 100, 0, 10, true)).toBe(1.7);
+    expect(tvStore.getQ4paConv).not.toHaveBeenCalled();
+  });
+
+  test('avant 1948, menuiseries majoritairement avec joints : ligne « avec joints »', () => {
+    service.q4paConv({}, ctx, 0, 100, 8, 2, false);
+    expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', '1');
+  });
+
+  test('avant 1948, menuiseries sans joint : ligne sans critère de joints', () => {
+    service.q4paConv({ tv_q4pa_conv_id: 12 }, ctx, 0, 100, 0, 10, false);
+    expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', undefined);
+  });
+
+  describe('presence_joint non exporté (issue #294)', () => {
+    test('hors bug_for_bug_compat : la ligne du DPE est ignorée', () => {
+      state.bug = false;
+      tvStore.getQ4paConvById.mockReturnValue({ presence_joints_menuiserie: '1' });
+
+      service.q4paConv({ tv_q4pa_conv_id: 12 }, ctx, 0, 100, 0, 10, true);
+
+      expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', undefined);
+    });
+
+    test('bug_for_bug_compat et ligne DPE « avec joints » : joints repris', () => {
+      state.bug = true;
+      tvStore.getQ4paConvById.mockReturnValue({ presence_joints_menuiserie: '1' });
+
+      service.q4paConv({ tv_q4pa_conv_id: 12 }, ctx, 0, 100, 0, 10, true);
+
+      expect(tvStore.getQ4paConvById).toHaveBeenCalledWith(12);
+      expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', '1');
+    });
+
+    test('bug_for_bug_compat mais presence_joint renseigné : pas de reprise', () => {
+      state.bug = true;
+      tvStore.getQ4paConvById.mockReturnValue({ presence_joints_menuiserie: '1' });
+
+      service.q4paConv({ tv_q4pa_conv_id: 12 }, ctx, 0, 100, 0, 10, false);
+
+      expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', undefined);
+    });
+
+    test('bug_for_bug_compat et ligne DPE sans joints : pas de reprise', () => {
+      state.bug = true;
+      tvStore.getQ4paConvById.mockReturnValue({ presence_joints_menuiserie: '' });
+
+      service.q4paConv({ tv_q4pa_conv_id: 1 }, ctx, 0, 100, 0, 10, true);
+
+      expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '0', undefined);
+    });
+
+    test('surfaces majoritairement isolées : le critère des joints ne s’applique pas', () => {
+      state.bug = true;
+      tvStore.getQ4paConvById.mockReturnValue({ presence_joints_menuiserie: '1' });
+
+      service.q4paConv({ tv_q4pa_conv_id: 12 }, ctx, 100, 0, 0, 10, true);
+
+      expect(tvStore.getQ4paConvById).not.toHaveBeenCalled();
+      expect(tvStore.getQ4paConv).toHaveBeenCalledWith('1', 'APPARTEMENT', '1', undefined);
+    });
+  });
+});

package/package.json

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