#98 Global custom converters missing in default ClassMap

closed
None
5
2007-07-15
2007-05-30
No

Hi, I have some serious problems with my global custom converters since Dozer 3.4.

I found out that Dozer now creates a default ClassMap in MappingProcessor.checkForSuperTypeMapping() due to bug #1486105. That might be ok, but ClassMapBuilder.createDefaultClassMap() does not copy the custom converters from global configuration into the default ClassMap. (And it seems that copy-by-references are missing, too.)

I'm not able to reproduce this with a unit test yet, but perhaps you simply forgot to copy these values?

Greetings,
Christoph

Discussion

  • Franz Garsombke

    Franz Garsombke - 2007-05-30

    Logged In: YES
    user_id=550744
    Originator: NO

    Nice find. This fix will be in the next release.

     
  • Franz Garsombke

    Franz Garsombke - 2007-05-30
    • assigned_to: nobody --> fgarsombke
     
  • Matt Tierney

    Matt Tierney - 2007-05-31

    Logged In: YES
    user_id=1236069
    Originator: NO

    Good find indeed. I was really surprised that this issue hasn't been raised before so I created a unit test for it to see for myself. It was easy to reproduce. I think it has been an issue prior to the 3.4 release for simple implicit mapping, but with the changes to checkForSuperTypeMapping(), you may just be seeing it for the first time in the latest release. Thanks for submitting the bug

     
  • Matt Tierney

    Matt Tierney - 2007-05-31

    Logged In: YES
    user_id=1236069
    Originator: NO

    Forgot to mention I checked in the unit test in KnownFailures.java

     
  • Franz Garsombke

    Franz Garsombke - 2007-06-02
    • status: open --> pending
     
  • Franz Garsombke

    Franz Garsombke - 2007-06-02

    Logged In: YES
    user_id=550744
    Originator: NO

    fixed in 3.x branch. i will fix the copy-by-reference next...which correlates to the previous bug.

     
  • Franz Garsombke

    Franz Garsombke - 2007-06-03

    Logged In: YES
    user_id=550744
    Originator: NO

    CopyByReference is fixed as well.

     
  • Matt Tierney

    Matt Tierney - 2007-07-15
    • status: pending --> closed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks