@rsuci/shared-form-components 1.0.17 → 1.0.19

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

@@ -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

@@ -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"}
+{"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;CAsD9B"}

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

@@ -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
      */
@@ -173,6 +234,31 @@ export class RosterConditionEngine {
             if (!functionMatch && !condition.includes('${')) {
                 return 'Condition invalide: utilisez showMe(), hideMe() ou jump()';
             }
+            // Vérifier que les jumps multiples utilisent uniquement ||
+            const allJumps = this.extractAllJumps(condition);
+            if (allJumps.length > 1) {
+                // Retirer tous les jumps et vérifier ce qui reste entre eux
+                let remaining = condition;
+                for (const jump of allJumps) {
+                    remaining = remaining.replace(jump.fullExpression, '###JUMP###');
+                }
+                // Entre les ###JUMP###, seul || est autorisé (avec espaces optionnels)
+                const betweenJumps = remaining.split('###JUMP###').slice(1, -1);
+                for (const between of betweenJumps) {
+                    const trimmed = between.trim();
+                    if (trimmed !== '||') {
+                        return `Seul l'opérateur || est autorisé entre les jumps multiples (trouvé: "${trimmed || 'rien'}")`;
+                    }
+                }
+                // Vérifier aussi avant le premier et après le dernier jump
+                const parts = remaining.split('###JUMP###');
+                if (parts[0].trim() !== '') {
+                    return `Syntaxe invalide avant le premier jump: "${parts[0].trim()}"`;
+                }
+                if (parts[parts.length - 1].trim() !== '') {
+                    return `Syntaxe invalide après le dernier jump: "${parts[parts.length - 1].trim()}"`;
+                }
+            }
             return null;
         }
         catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rsuci/shared-form-components",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "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",