Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Catmandu-Path-Wildcard-Support: $prepend #93

Closed
TobiasNx opened this issue Dec 16, 2021 · 4 comments · Fixed by #368
Closed

Catmandu-Path-Wildcard-Support: $prepend #93

TobiasNx opened this issue Dec 16, 2021 · 4 comments · Fixed by #368
Labels
Conform To Catmandu Revisit After #102

Comments

@TobiasNx
Copy link
Collaborator

TobiasNx commented Dec 16, 2021 •
edited
Loading

Test for prepend:
For array of strings:
Here

Output should be:

{
	"animals":[
                        "earthworm".
                        "unicorn",
                        "human",
			"dog",
			"cat"
	],
	"others": "human"
}

For array of objects:
Here

Output should be:

{
	"animals":[
                        {"animal": "earthworm"}
                        {"animal": "unicorn"},
                        {"animal": "human"},
			{"animal": "dog"},
			{"animal": "cat"}
	],
	"others": {"animal": "human"}
}
TobiasNx added a commit to TobiasNx/fix-FunctionalReview-Testing that referenced this issue Feb 1, 2022
@TobiasNx
Add test for prepend string array
d3ebe83 
metafacture/metafacture-fix#93
@TobiasNx
Copy link
Collaborator Author

TobiasNx commented Feb 1, 2022

Added a test scenario: https://github.com/TobiasNx/fix-FunctionalReview-Testing/tree/master/data/testing/prependArrays_strings

@blackwinter
Copy link
Member

See also #92.

@fsteeg
Copy link
Member

fsteeg commented Feb 3, 2022 •
edited
Loading

Will revisit after #102.

@fsteeg fsteeg moved this to Backlog in Metafacture Feb 3, 2022
@TobiasNx TobiasNx mentioned this issue Feb 3, 2022
Closed
@TobiasNx TobiasNx changed the title Catmandu-Path-Wildcard-Support: $prepend Catmandu-Path-Wildcard-Support: $prepend Feb 8, 2022
TobiasNx added a commit that referenced this issue Feb 28, 2022
@TobiasNx
Add tests to generate new arrayOfStrings with move_field
b5cc5b8 
Related to #160,  #127, #93
TobiasNx added a commit that referenced this issue Feb 28, 2022
@TobiasNx
Add tests to generate new arrayOfStrings with copy_field
fe51543 
Related to #160,  #127, #93
TobiasNx added a commit that referenced this issue Feb 28, 2022
@TobiasNx
Add tests to generate new arrayOfStrings with add_field
48c2f75 
Related to #160,  #127, #93
@blackwinter
Copy link
Member

Integration tests:

fsteeg added a commit that referenced this issue Sep 25, 2024
@fsteeg
Support $prepend for inserting & array creation (#92, #93, #127)
fffb6d9
@fsteeg fsteeg mentioned this issue Sep 25, 2024
Merged
@github-project-automation github-project-automation bot moved this from Backlog to Done in Metafacture Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Conform To Catmandu Revisit After #102
Projects
Metafacture
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implicit array creation with $append, general $prepend support metafacture/metafacture-fix
3 participants
@blackwinter @fsteeg @TobiasNx