Fix bug in Nub.resultHasMultipleValues()

bed3acde · Mathias Chouet · b5a74d22 · bed3acde
Commit bed3acde authored 5 years ago by Mathias Chouet
Hide whitespace changes
Inline Side-by-side

Showing

with 4 additions and 3 deletions
+4 -3
--- a/src/nub.ts
+++ b/src/nub.ts
@@ -706,15 +706,16 @@ export abstract class Nub extends ComputeNode implements IObservable {
     * linked or not, is variated
     */
    public resultHasMultipleValues(): boolean {
+        let hmv = false;
        for (const p of this.parameterIterator) {
            if (p.valueMode === ParamValueMode.MINMAX || p.valueMode === ParamValueMode.LISTE) {
-                return true;
+                hmv = true;
            } else if (p.valueMode === ParamValueMode.LINK && p.isReferenceDefined()) {
                // indirect recursivity
-                return p.referencedValue.hasMultipleValues();
+                hmv = hmv || p.referencedValue.hasMultipleValues();
            }
        }
-        return false;
+        return hmv;
    }

    /**