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

CollectionUtil: a function to merge maps #2238

Closed
01es opened this issue Apr 18, 2024 · 0 comments · Fixed by #2239
Closed

CollectionUtil: a function to merge maps #2238

01es opened this issue Apr 18, 2024 · 0 comments · Fixed by #2239
Assignees
@01es
Labels
Developer empowerment
Milestone
1.4.6

Comments

@01es
Copy link
Member

01es commented Apr 18, 2024

Description

The Map interface in Java has method putAll, which returns void. This is very inconvenient when several maps needs to be merged as an expression. In order to support this, CollectionUtil needs to be enhanced with method Map<K,V> merge(Map<K,V> map1, Map<K,V>... otherMaps) that would create a new map of the same type as its first argument and put all elements from all of the non-null maps in to the result.

Expected outcome

Ability to merge maps as expression.
@01es 01es added this to the v1.4.6 M2 milestone Apr 18, 2024
@01es 01es self-assigned this Apr 18, 2024
01es added a commit that referenced this issue Apr 18, 2024
@01es
#2238 Implemented function merge.
e516d2a
@01es 01es mentioned this issue Apr 18, 2024
Merged
01es added a commit that referenced this issue Apr 24, 2024
@01es
#2238 Enabled merging of maps with compatible types for keys and values.
94cb32c
homedirectory added a commit that referenced this issue May 7, 2024
@homedirectory
#2238 Add missing Test annotation and fix a failing assertion
e19065f 
sorted() was causing a runtime error due to StringBuffer and String not
being mutually comparable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@01es 01es

Labels
Developer empowerment
Projects
None yet
Milestone
1.4.6
Development

Successfully merging a pull request may close this issue.

CollectionUtil: a function to merge maps fieldenms/tg
1 participant
@01es