Skip to content
This repository has been archived by the owner on Jan 27, 2025. It is now read-only.

Strange behaviour moving generated arrays and successively generating arrays into nested structure #113

Closed
TobiasNx opened this issue Jan 26, 2022 · 3 comments
Assignees
Labels
Bug Something isn't working Revisit After #102

Comments

@TobiasNx
Copy link
Collaborator

TobiasNx commented Jan 26, 2022

move_field and copy_field seem to have problems moving generated arrays and generating arrays into nested structures.

Moving

  • moving a generated array with a single value is transformed into an simple string-array
  • moving a generated array with a multiple values is working

Generating

  • generating an array with a multiple values into an nested object is transformed into an array with an object indexing all values as own elements

  • generating an array with a single value into an nested object is working

  • generating or moving an array into an object of an array is breaking or not working

--- was originally:----
When moving an array in an object of arrays the behavior seems strange.
If you move an array into an object of array the behavior seems okay.

See testcase here:

https://github.com/TobiasNx/fix-FunctionalReview-Testing/tree/master/data/testing/move_field_in_object_array
https://github.com/TobiasNx/fix-FunctionalReview-Testing/tree/master/data/testing/copy_array

@TobiasNx TobiasNx added the Bug Something isn't working label Jan 26, 2022
TobiasNx added a commit to TobiasNx/fix-FunctionalReview-Testing that referenced this issue Jan 27, 2022
@TobiasNx TobiasNx changed the title Strange behaviour when moving an array in an object of an array. Strange behaviour moving generated arrays and successively generating arrays into nested structure Jan 27, 2022
TobiasNx added a commit to TobiasNx/fix-FunctionalReview-Testing that referenced this issue Jan 27, 2022
@fsteeg
Copy link
Member

fsteeg commented Feb 3, 2022

To avoid conflict with #102, should be started after the planned first PR for #102.

Functional review: @TobiasNx
Code review: @blackwinter / @fsteeg

@TobiasNx
Copy link
Collaborator Author

TobiasNx commented Mar 2, 2022

Some behaviour seems to be solved.

TobiasNx added a commit that referenced this issue Mar 3, 2022
TobiasNx added a commit that referenced this issue Mar 8, 2022
TobiasNx added a commit that referenced this issue Mar 8, 2022
@fsteeg fsteeg closed this as completed May 15, 2023
@github-project-automation github-project-automation bot moved this from Ready to Done in Metafacture May 15, 2023
@blackwinter
Copy link
Member

Pending integration test:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working Revisit After #102
Projects
Archived in project
Development

No branches or pull requests

3 participants