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.

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

#1164 Make sure that all import types of a Type are imported for the wrapper #1165

Merged
merged 1 commit into from Mar 31, 2017

Conversation

filiphr
Copy link
Member

@filiphr filiphr commented Mar 30, 2017

Fixes #1164

@agudian
Copy link
Member

agudian commented Mar 31, 2017

👍 Feel free to merge!

@filiphr
Copy link
Member Author

filiphr commented Mar 31, 2017

Thanks for the review @agudian. Once the build is done I will merge it. Can you also have a look at the issue. It looks like the code generated in 1.0.0.Final is not the same as in 1.1.0.Final, and I am not sure what is correct :). It looks like for Collection(s) and Map(s) we are not setting them to null if the source is null.

@agudian
Copy link
Member

agudian commented Mar 31, 2017

It looks like for Collection(s) and Map(s) we are not setting them to null if the source is null.

I think Sjaak fixed some stuff there and the change was intentional (didn't we even have something about that in the migration notes?)...

@filiphr filiphr merged commit bbff0c0 into mapstruct:master Mar 31, 2017
@filiphr filiphr deleted the 1164 branch March 31, 2017 19:41
@filiphr
Copy link
Member Author

filiphr commented Mar 31, 2017

@agudian Thanks again for the review. I have merged it to master

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.

None yet

2 participants