From: Matt B. <gud...@ya...> - 2007-01-15 17:19:10
|
--- Ben Alex <ben...@ac...> wrote: > Ben Alex wrote: > > I've attached the patch (described in my last > post). It has been working > > fine here. > > Hi guys > > Any progress on this? I really need this > functionality in core. I've > been using the patch successfully for a few days > now. A further reason > we've found for supporting a modified > InstantiatingReflector is to > retrieve existing objects from a repository that we > want to decorate > with changes from an input DTO. There are therefore > several usage > scenarios which benefit from the "Object parameters" > additional > parameter on the interface method. > Hi Ben, I have just committed your patch (with IIRC one spelling correction) to HEAD. Sorry for the delay; I wanted to think about this first, because I was having a little trouble deciding whether this was the responsibility of the InstantiatingReflector, or that of a particular Transformer implementation. Finally I realized that (1) the Reflector exists at least partially (originally wholly I suppose) to facilitate conversions, and (2) if Constructor.newInstance() required the parameter (at least before varargs) then it is sensible for Morph to follow suit. Finally, as I mentioned in an earlier email, Morph is not as constrained by BC issues at present as are other software packages. So how are you finding Morph compared to Dozer so far? br, Matt > Cheers > Ben > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get > the chance to share your > opinions on IT & business topics through brief > surveys - and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > morph-developer mailing list > mor...@li... > https://lists.sourceforge.net/lists/listinfo/morph-developer > ____________________________________________________________________________________ The fish are biting. Get more visitors on your site using Yahoo! Search Marketing. http://searchmarketing.yahoo.com/arp/sponsoredsearch_v2.php |