Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

#2553 Support source property paths for maps#2653

Merged
filiphr merged 2 commits intomapstruct:mastermapstruct/mapstruct:masterfrom
filiphr:2553filiphr/mapstruct:2553Copy head branch name to clipboard
Nov 20, 2021
Merged

#2553 Support source property paths for maps#2653
filiphr merged 2 commits intomapstruct:mastermapstruct/mapstruct:masterfrom
filiphr:2553filiphr/mapstruct:2553Copy head branch name to clipboard

Conversation

@filiphr
Copy link
Member

@filiphr filiphr commented Nov 13, 2021

Fixes #2553

This PR does some small changes in the way we acquire read accessors. The refactorings in the PR, especially the new methods in the Type can open the door for better support for things like Jackson JsonNode, ResultSet, etc.

Copy link
Contributor

@sjaakd sjaakd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a tiny remark.. If all testcases pass: LGTM

@filiphr filiphr merged commit 13bc0c0 into mapstruct:master Nov 20, 2021
@filiphr filiphr deleted the 2553 branch November 20, 2021 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New mapping feature not support property paths

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.