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

uniq mixes arrays when used with asterisk #261

Open
TobiasNx opened this issue Oct 7, 2022 · 2 comments
Open

uniq mixes arrays when used with asterisk #261

TobiasNx opened this issue Oct 7, 2022 · 2 comments
Assignees
@blackwinter @fsteeg
Labels
bug Something isn't working

Comments

@TobiasNx
Copy link
Collaborator

TobiasNx commented Oct 7, 2022

e.g. uniq("publication[].*.location[]") or uniq("publication[].*.publishedBy[]")
Instead of only uniqing a single array it mixes the different arrays and outputs uniq versions of the mixed arrays.
See: https://github.com/hbz/lobid-resources/pull/1463/files
@TobiasNx TobiasNx added the bug Something isn't working label Oct 7, 2022
@TobiasNx TobiasNx changed the title uniq mixes arrays when used with asterics uniq mixes arrays when used with asteriks Oct 7, 2022
@TobiasNx TobiasNx changed the title uniq mixes arrays when used with asteriks uniq mixes arrays when used with asterisk Oct 7, 2022
TobiasNx added a commit that referenced this issue Oct 7, 2022
@TobiasNx
Introduce integrationtest for #261
5799bb2
@TobiasNx
Copy link
Collaborator Author

TobiasNx commented Oct 7, 2022

#262 I introduced an integrationtest for this bug.

TobiasNx added a commit that referenced this issue Oct 7, 2022
@TobiasNx
Add todo.txt #261
330d6e2
fsteeg added a commit that referenced this issue Oct 7, 2022
@fsteeg
Merge pull request #262 from metafacture/261-uniqErrorIntegrationTest
ce9157f 
Introduce integrationtest for #261
@TobiasNx TobiasNx mentioned this issue Feb 21, 2023
Closed
@blackwinter
Copy link
Member

Integration test:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blackwinter blackwinter

@fsteeg fsteeg

Labels
bug Something isn't working
Projects
Metafacture
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@blackwinter @fsteeg @TobiasNx