This is a difficult issue... Maybe we can get the Atlassian guys to talk about what they've been doing in Confluence?
The reason doing database lookups in type converters is bad is because you're not in control of when it happens... It could be that it happens 1000's of times. Jason > -----Original Message----- > From: Dag Liodden [mailto:[EMAIL PROTECTED] > > Even if this example had only been about editing primitive > (or primitive > wrappers) properties of the Employee instance there would have been > trouble. The simplest approach is to instanciate a temporary > instance of > the Employee class in the action and let the interceptors set > the values > to that instance, but then you'll have to copy the properties > manually > when the action executed later. (a beanutils library will > help in some > cases, but what about nulls and transient instances?). > > The PetSoar example does not provide any pointers - it does manual > copying and avoids this issue. Some debate on this subject > would be very > welcome! If we get a good and simple pattern for this, it would be a > killer feature. > > One last point BTW. The type conversion doco states that it's > a bad idea > to do database lookups. Can someone elaborate, please? :) > > Cheers, > > Dag > ------------------------------------------------------- The SF.Net email is sponsored by EclipseCon 2004 Premiere Conference on Open Tools Development and Integration See the breadth of Eclipse activity. February 3-5 in Anaheim, CA. http://www.eclipsecon.org/osdn _______________________________________________ Opensymphony-webwork mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/opensymphony-webwork
