npm - @rsuci/shared-form-components - Versions diffs - 1.0.17 → 1.0.18 - Mend

@rsuci/shared-form-components 1.0.17 → 1.0.18

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (4) hide show

package/dist/lib/roster-condition-engine.d.ts +19 -1
package/dist/lib/roster-condition-engine.d.ts.map +1 -1
package/dist/lib/roster-condition-engine.js +81 -20
package/package.json +1 -1

package/dist/lib/roster-condition-engine.d.ts CHANGED Viewed

@@ -41,14 +41,32 @@ export declare class RosterConditionEngine {
      */
     static extractVariableReferences(condition: string): string[];
     /**
-     * Extrait la cible d'un jump s'il existe dans la condition
+     * Extrait la cible d'un jump s'il existe dans la condition (premier match uniquement)
      * Format: jump(condition, ${TARGET})
+     * @deprecated Utiliser extractAllJumps pour supporter les jumps multiples
      */
     static extractJumpTarget(condition: string): string | null;
+    /**
+     * Extrait TOUS les jumps d'une condition
+     * Supporte: jump(cond1, ${VAR1}) || jump(cond2, ${VAR2})
+     * Retourne un tableau de { fullExpression, innerCondition, target }
+     */
+    static extractAllJumps(condition: string): Array<{
+        fullExpression: string;
+        innerCondition: string;
+        target: string;
+    }>;
     /**
      * Calcule les variables qui doivent être masquées à cause des jumps actifs
+     * Supporte les jumps multiples avec court-circuit:
+     * - jump(cond1, ${VAR1}) || jump(cond2, ${VAR2})
+     * - Si cond1 est vraie, on saute à VAR1 et cond2 n'est pas évaluée
      */
     static computeJumpedVariables(sortedVariables: RosterVariableRef[], lineValues: Record<string, any>): Set<string>;
+    /**
+     * Évalue une condition interne (sans le wrapper jump())
+     */
+    private static evaluateInnerCondition;
     /**
      * Valide la syntaxe de base d'une condition
      */

package/dist/lib/roster-condition-engine.d.ts.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"roster-condition-engine.d.ts","sourceRoot":"","sources":["../../src/lib/roster-condition-engine.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,MAAM,MAAM,wBAAwB,GAChC,uBAAuB,GACvB,mBAAmB,GACnB,eAAe,GACf,gBAAgB,GAChB,cAAc,CAAC;AAEnB,MAAM,WAAW,oBAAoB;IACnC,IAAI,EAAE,wBAAwB,CAAC;IAC/B,YAAY,EAAE,MAAM,CAAC;IACrB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,+BAA+B;IAC9C,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,oBAAoB,EAAE,CAAC;CAChC;AAED,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;CACf;AAED;;;GAGG;AACH,qBAAa,qBAAqB;IAEhC;;;OAGG;IACH,MAAM,CAAC,iBAAiB,CACtB,SAAS,EAAE,MAAM,EACjB,kBAAkB,EAAE,iBAAiB,EAAE,EACvC,mBAAmB,EAAE,MAAM,EAC3B,oBAAoB,EAAE,MAAM,GAC3B,+BAA+B;IAgElC;;;OAGG;IACH,MAAM,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAAG,OAAO;IAyB5E;;;OAGG;IACH,MAAM,CAAC,yBAAyB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE;IAO7D~~;;;OAGG~~;IACH,MAAM,CAAC,iBAAiB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAO1D~~;;OAEG~~;IACH,MAAM,CAAC,sBAAsB,CAC3B,eAAe,EAAE,iBAAiB,EAAE,EACpC,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAC9B,GAAG,CAAC,MAAM,CAAC;~~IA2Bd~~;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;CA0B9B"}
1	+ {"version":3,"file":"roster-condition-engine.d.ts","sourceRoot":"","sources":["../../src/lib/roster-condition-engine.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,MAAM,MAAM,wBAAwB,GAChC,uBAAuB,GACvB,mBAAmB,GACnB,eAAe,GACf,gBAAgB,GAChB,cAAc,CAAC;AAEnB,MAAM,WAAW,oBAAoB;IACnC,IAAI,EAAE,wBAAwB,CAAC;IAC/B,YAAY,EAAE,MAAM,CAAC;IACrB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,+BAA+B;IAC9C,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,oBAAoB,EAAE,CAAC;CAChC;AAED,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;CACf;AAED;;;GAGG;AACH,qBAAa,qBAAqB;IAEhC;;;OAGG;IACH,MAAM,CAAC,iBAAiB,CACtB,SAAS,EAAE,MAAM,EACjB,kBAAkB,EAAE,iBAAiB,EAAE,EACvC,mBAAmB,EAAE,MAAM,EAC3B,oBAAoB,EAAE,MAAM,GAC3B,+BAA+B;IAgElC;;;OAGG;IACH,MAAM,CAAC,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAAG,OAAO;IAyB5E;;;OAGG;IACH,MAAM,CAAC,yBAAyB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,EAAE;IAO7D;;;;OAIG;IACH,MAAM,CAAC,iBAAiB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAO1D;;;;OAIG;IACH,MAAM,CAAC,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,KAAK,CAAC;QAC/C,cAAc,EAAE,MAAM,CAAC;QACvB,cAAc,EAAE,MAAM,CAAC;QACvB,MAAM,EAAE,MAAM,CAAC;KAChB,CAAC;IAoBF;;;;;OAKG;IACH,MAAM,CAAC,sBAAsB,CAC3B,eAAe,EAAE,iBAAiB,EAAE,EACpC,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAC9B,GAAG,CAAC,MAAM,CAAC;IAmCd;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,sBAAsB;IA6BrC;;OAEG;IACH,OAAO,CAAC,MAAM,CAAC,cAAc;CA0B9B"}

package/dist/lib/roster-condition-engine.js CHANGED Viewed

@@ -40,23 +40,23 @@ export class RosterConditionEngine {
                 });
             }
         }
-        // 2. Vérifier le jump s'il existe
-        const jumpTarget = this.extractJumpTarget(condition);
-        if (jumpTarget) {
-            if (!availableCodes.has(jumpTarget)) {
+        // 2. Vérifier TOUS les jumps (supporte jump() || jump() || ...)
+        const allJumps = this.extractAllJumps(condition);
+        for (const jump of allJumps) {
+            if (!availableCodes.has(jump.target)) {
                 errors.push({
                     type: 'jump_out_of_scope',
-                    variableCode: jumpTarget,
-                    message: `La variable cible \${${jumpTarget}} n'existe pas dans ce Roster`
+                    variableCode: jump.target,
+                    message: `La variable cible \${${jump.target}} n'existe pas dans ce Roster`
                 });
             }
             else {
-                const targetVar = availableVariables.find(v => v.code === jumpTarget);
+                const targetVar = availableVariables.find(v => v.code === jump.target);
                 if (targetVar && targetVar.ordre <= currentVariableOrdre) {
                     errors.push({
                         type: 'backward_jump',
-                        variableCode: jumpTarget,
-                        message: 'Le jump ne peut pas cibler une variable précédente'
+                        variableCode: jump.target,
+                        message: `Le jump vers \${${jump.target}} ne peut pas cibler une variable précédente`
                     });
                 }
             }
@@ -113,8 +113,9 @@ export class RosterConditionEngine {
         return matches.map(m => m.slice(2, -1));
     }
     /**
-     * Extrait la cible d'un jump s'il existe dans la condition
+     * Extrait la cible d'un jump s'il existe dans la condition (premier match uniquement)
      * Format: jump(condition, ${TARGET})
+     * @deprecated Utiliser extractAllJumps pour supporter les jumps multiples
      */
     static extractJumpTarget(condition) {
         if (!condition)
@@ -122,8 +123,32 @@ export class RosterConditionEngine {
         const match = condition.match(/jump\s*\([^,]+,\s*\$\{([A-Z_][A-Z0-9_]*)\}\s*\)/);
         return match ? match[1] : null;
     }
+    /**
+     * Extrait TOUS les jumps d'une condition
+     * Supporte: jump(cond1, ${VAR1}) || jump(cond2, ${VAR2})
+     * Retourne un tableau de { fullExpression, innerCondition, target }
+     */
+    static extractAllJumps(condition) {
+        if (!condition)
+            return [];
+        const jumps = [];
+        // Regex pour capturer chaque jump avec sa condition interne et sa cible
+        const jumpRegex = /jump\s*\(\s*([^,]+?)\s*,\s*\$\{([A-Z_][A-Z0-9_]*)\}\s*\)/g;
+        let match;
+        while ((match = jumpRegex.exec(condition)) !== null) {
+            jumps.push({
+                fullExpression: match[0],
+                innerCondition: match[1].trim(),
+                target: match[2]
+            });
+        }
+        return jumps;
+    }
     /**
      * Calcule les variables qui doivent être masquées à cause des jumps actifs
+     * Supporte les jumps multiples avec court-circuit:
+     * - jump(cond1, ${VAR1}) || jump(cond2, ${VAR2})
+     * - Si cond1 est vraie, on saute à VAR1 et cond2 n'est pas évaluée
      */
     static computeJumpedVariables(sortedVariables, lineValues) {
         const jumped = new Set();
@@ -131,24 +156,60 @@ export class RosterConditionEngine {
             const condition = rosterVar.conditionsAffichage;
             if (!condition)
                 continue;
-            const jumpTarget = this.extractJumpTarget(condition);
-            if (!jumpTarget)
+            // Extraire tous les jumps de cette condition
+            const allJumps = this.extractAllJumps(condition);
+            if (allJumps.length === 0)
                 continue;
-            // Évaluer si le jump doit s'activer
-            if (this.evaluate(condition, lineValues)) {
-                const targetVar = sortedVariables.find(v => v.code === jumpTarget);
-                if (targetVar) {
-                    // Masquer toutes les variables entre la source et la cible
-                    for (const v of sortedVariables) {
-                        if (v.ordre > rosterVar.ordre && v.ordre < targetVar.ordre) {
-                            jumped.add(v.code);
+            // Évaluer les jumps dans l'ordre avec court-circuit
+            for (const jump of allJumps) {
+                // Évaluer la condition interne du jump
+                const innerConditionResult = this.evaluateInnerCondition(jump.innerCondition, lineValues);
+                if (innerConditionResult) {
+                    // Ce jump s'active - masquer les variables entre source et cible
+                    const targetVar = sortedVariables.find(v => v.code === jump.target);
+                    if (targetVar) {
+                        for (const v of sortedVariables) {
+                            if (v.ordre > rosterVar.ordre && v.ordre < targetVar.ordre) {
+                                jumped.add(v.code);
+                            }
                         }
                     }
+                    // Court-circuit: on ne vérifie pas les jumps suivants
+                    break;
                 }
             }
         }
         return jumped;
     }
+    /**
+     * Évalue une condition interne (sans le wrapper jump())
+     */
+    static evaluateInnerCondition(innerCondition, lineValues) {
+        if (!innerCondition?.trim())
+            return false;
+        try {
+            // Construire un contexte isolé pour cette ligne du roster
+            const responses = {};
+            for (const [code, value] of Object.entries(lineValues)) {
+                responses[code] = {
+                    variableCode: code,
+                    valeur: value,
+                    dateModification: new Date()
+                };
+            }
+            const engine = new ConditionEngine(responses);
+            // Si c'est déjà un showMe(), l'évaluer directement
+            if (innerCondition.trim().startsWith('showMe(')) {
+                return engine.evaluate(innerCondition);
+            }
+            // Sinon, wrapper dans showMe pour évaluation
+            return engine.evaluate(`showMe(${innerCondition})`);
+        }
+        catch (error) {
+            console.warn('[RosterConditionEngine] Erreur évaluation condition interne:', innerCondition, error);
+            return false;
+        }
+    }
     /**
      * Valide la syntaxe de base d'une condition
      */

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@rsuci/shared-form-components",
-  "version": "1.0.17",
+  "version": "1.0.18",
   "description": "Composants partagés de rendu de formulaires RSU v2 - Package local pour frontend Admin et Public",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",