Skip to content

Commit a489a3b

Browse files
committed
[KNOWAGE-9898] reset dependency arrays to avoid duplicates in loadParameters
1 parent ad68bc2 commit a489a3b

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

src/components/UI/KnParameterSidebar/KnParameterSidebar.vue

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -372,6 +372,16 @@ export default defineComponent({
372372
this.parameters.filterStatus.push(el)
373373
}
374374
})
375+
// Reset dependency arrays before recomputing them to avoid duplicates when loadParameters() is called multiple times (e.g. both propDocument and filtersData watchers trigger).
376+
// Without this reset, set*Dependency functions would push the same parameter references again into already-populated arrays, causing admissibleValues to be called N times per change.
377+
this.parameters.filterStatus.forEach((el: any) => {
378+
el.dataDependentParameters = undefined
379+
el.dataDependsOnParameters = undefined
380+
el.lovDependentParameters = undefined
381+
el.lovDependsOnParameters = undefined
382+
el.dependentParameters = undefined
383+
el.dependsOnParameters = undefined
384+
})
375385
this.parameters?.filterStatus.forEach((el: any) => setVisualDependency(this.parameters, el))
376386
this.parameters?.filterStatus.forEach((el: any) => setDataDependency(this.parameters, el))
377387
this.parameters?.filterStatus.forEach((el: any) => setLovsDependency(this.parameters, el))

0 commit comments

Comments
 (0)