FLUID-6196: Collect Input paths implementation faulty when combining standard and complex transforms

Metadata

Source
FLUID-6196
Type
Bug
Priority
Major
Status
Closed
Resolution
Fixed
Assignee
Kasper Galschiot Markus
Reporter
Kasper Galschiot Markus
Created
2017-09-19T04:33:58.029-0400
Updated
2024-07-22T09:02:56.940-0400
Versions
N/A
Fixed Versions
N/A
Component
  1. Model Transformation System

Description

When having a nested complex transform (such as the valueMapper) it destroys the array holding the inputPaths.

In most cases we just pass around the array of input paths and expect that reference to hold (see: https://github.com/fluid-project/infusion/blob/136c58613cb218ceaafc0528b24f2175a780009f/src/framework/core/js/ModelTransformation.js#L404 ) when modifying the content which is all fine and dandy until one of these https://github.com/fluid-project/infusion/blob/136c58613cb218ceaafc0528b24f2175a780009f/src/framework/core/js/ModelTransformation.js#L413 come along. Apparently Array.concat returns a new array, which means that our previous inputPath array reference suddenly is no longer linked to the transformer object