Remove it and add a line in the migration guide. On Sat, Sep 11, 2010 at 11:44 AM, Juergen Donnerstag < juergen.donners...@gmail.com> wrote:
> I do > > -Juergen > > On Sat, Sep 11, 2010 at 11:34 AM, Peter Ertl <pe...@gmx.org> wrote: > > Hi wicket devs, > > > > in 1.5... > > > > I would to remove AbstractResource#isCacheable() and give > AbstractResource#getCacheDuration() the meaning > > > > 0 = no caching > > > 0 = duration in milliseconds for caching > > > > This would give the developer more control as he could not only set the > default cache duration via > > > > Application.get().getResourceSettings().getDefaultCacheDuration() > > > > but also if resources in general should be cached at all (> 0) or never > (=0) > > > > Otherwise we would need another flag > > > > Application.get().getResourceSettings().isCachingEnabledByDefault() > > > > to gain the same effect. > > > > Do you agree with this change? >
