Pedro Santos created WICKET-7128:
------------------------------------
Summary: Improvement suggestions to IStaticCacheableResource's API
Key: WICKET-7128
URL: https://issues.apache.org/jira/browse/WICKET-7128
Project: Wicket
Issue Type: Sub-task
Components: wicket-core
Affects Versions: 10.3.0
Reporter: Pedro Santos
Context: currently Wicket uses an IResourceCachingStrategy to control how the
app resources will be cached in the browser by decorating the URL, and a
CachingResourceStreamLocator to control how the resource is cached in the
server. However, in the resource interface, it's not clear that the flag
IStaticCacheableResource.isCachingEnabled() is used only to inform how the
resource should be cached in the client.
Improvement:
- to deprecate IStaticCacheableResource.isCachingEnabled() in the current
versions and rename it to
IStaticCacheableResource.isClientCachingStrategyEnabled() in the next major
one, plus to improve its Javadoc.
Context: after working on the ticket WICKET-7024, I stumbled on the need for
IStaticCacheableResource implementations to be able to know if the work to
resolve an IResourceStream should update or not the server resources cache in
the CachingResourceStreamLocator.
Improvement:
- to introduce a new method:
IStaticCacheableResource.isServerCachingUpdateEnabled() or to change
IStaticCacheableResource.getResourceStream() to
IStaticCacheableResource.getResourceStream(boolean updateCache)
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
