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

User.merge puts array attribute in array #550

Open
dmironowicz opened this issue Apr 12, 2022 · 0 comments · May be fixed by #551
Open

User.merge puts array attribute in array #550

dmironowicz opened this issue Apr 12, 2022 · 0 comments · May be fixed by #551
Labels
bug

Comments

@dmironowicz
Copy link

Version

4.2.6

Context

Merging two User objects, such that only one of them has an array attribute with a given name, results in the merged user having this attribute as array of arrays.

Steps to reproduce

  1. Create 1st User without any attributes
  2. Create 2nd User with a single JsonArray attribute, e.g. roles = ["write"]
  3. Call merge on the 1st user passing 2nd as parameter
  4. Merged user has roles = [ ["write"] ]
dmironowicz added a commit to dmironowicz/vertx-auth that referenced this issue Apr 12, 2022
@dmironowicz
Fix eclipse-vertx#550: Fixed merging array-type attributes when only …
3ea6603 
…one of the two User objects has the attribute
dmironowicz added a commit to dmironowicz/vertx-auth that referenced this issue Apr 12, 2022
@dmironowicz
Revert "Fix eclipse-vertx#550: Fixed merging array-type attributes wh…
f43b981 
…en only one of the two User objects has the attribute"

This reverts commit 3ea6603.
dmironowicz added a commit to dmironowicz/vertx-auth that referenced this issue Apr 12, 2022
@dmironowicz
Fix eclipse-vertx#550: Fixed merging array-type attributes when only …
be1eb16 
…one of the two User objects has the attribute
@dmironowicz dmironowicz linked a pull request Apr 12, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #550: Fixed merging array-type attributes when only one of the tw… dmironowicz/vertx-auth
1 participant
@dmironowicz