search

jmapper-framework / jmapper-core

Elegance, high performance and robustness all in one java bean mapper
http://jmapper-framework.github.io/jmapper-core
Apache License 2.0
231 stars 41 forks source link

Support of immutable objects #37

Closed vraptorche closed 8 years ago

vraptorche commented 8 years ago

When using immutable objects - eg. immutables.org framework, default constructor is missing or private, which prevents instantiation of destination bean

avurro commented 8 years ago

Hi, you have need that the mapping is done through the constructor, or you just need to instantiate destination before mapping? if it is the second, you can use the getDestination method passing the target bean created. wiki page here.

Add a star if you like the project, thank you in advance!

vraptorche commented 8 years ago

Hi, Thank You for the prompt replay.

I can not use the getDestination() methods because I get exception when I try to create mapper as per:

JMapper<JobListingVO, JobListingEntity> mapper = new JMapper<>(JobListingVO.class, JobListingEntity.class);

JobListingVO does not have public default constructor and instances are created with JobListingVO.create()

Best

~Emil

Dipl. Eng. Emil Korladinov M.Sc. EE JurassicWare Ltd

http://www.automobilnik.com

On Mon, Jun 13, 2016 at 7:49 AM, Alessandro Vurro notifications@github.com wrote:

Hi, you have need that the mapping is done through the constructor, or you just need to instantiate destination before mapping? if it is the second, you can use the getDestination method passing the target bean created. wiki page here https://github.com/jmapper-framework/jmapper-core/wiki/getDestination-method#enrichment-signatures .

Add a star if you like the project, thank you in advance!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jmapper-framework/jmapper-core/issues/37#issuecomment-225494102, or mute the thread https://github.com/notifications/unsubscribe/ADpdTG8ukO2Xcv3eBXeMzoWsWOzEb2FFks5qLO9RgaJpZM4I0AnI .

avurro commented 8 years ago

ok I've seen, but answer my question, what is the desired behavior?

vraptorche commented 8 years ago

Hi, It will be enough to use pre-created instance through getDestination()

Best ~Emil On Jun 13, 2016 11:47 AM, "Alessandro Vurro" notifications@github.com wrote:

ok I've seen, but answer my question, what is the desired behavior?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jmapper-framework/jmapper-core/issues/37#issuecomment-225536007, or mute the thread https://github.com/notifications/unsubscribe/ADpdTCeH8KMiS1gYJrkB3GI3kTGfHGf5ks5qLSc7gaJpZM4I0AnI .

avurro commented 8 years ago

ok, in the next release we will also this feature.

remember to add the star :)

avurro commented 8 years ago

Implemented, i'm near to release this feature. stay tuned!