+1 -----Original Message----- From: [email protected] [mailto:[email protected]] Sent: Friday, January 16, 2009 7:47 AM To: [email protected] Subject: Re: Why you should not override isVisible
Ok, IMHO it's a bug that wicket calls isVisible() after detachment. Thus caching isVisible() is not needed. Sven ----- Ursprüngliche Nachricht ----- Von: Michael Sparer Gesendet: 16.01.09 11:20 Uhr An: [email protected] Betreff: Re: Re: Why you should not override isVisible Nope, the problem is that the model object *possibly* gets reloaded if isVisible is called after the cached object got detached - and that's what started the whole bunch of messages Michael svenmeier wrote: > > What's taking so long in your isVisible() method? > > > The model object should be cached, and is isPositive() so expensive? > > Sven > > ----- Ursprüngliche Nachricht ----- > Von: Scott Swank > Gesendet: 16.01.09 02:06 Uhr > An: [email protected] > Betreff: Re: Why you should not override isVisible > > We have implemented this, perhaps a dozen times or more across our > application. For example, there are several payment options whose > relevance is determined by whether the customer owes any money on > their purchase (e.g. as opposed to using a gift card). These "total > the order and determine visibility" methods were particular hot spots. > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
