PackagedTextTemplate should set lastModifiedTime
------------------------------------------------
Key: WICKET-3315
URL: https://issues.apache.org/jira/browse/WICKET-3315
Project: Wicket
Issue Type: Bug
Components: wicket
Affects Versions: 1.4.15
Reporter: Guillaume Smet
Priority: Minor
Hi,
When using TextTemplateResourceReference in a
setAddLastModifiedTimeToResourceReferenceUrl(true) configuration, the w:lm
parameter isn't added to the URL of the resource as
PackagedTextTemplate.lastModifiedTime() (called in
TextTemplateResourceReference.lastModifiedTime()) always returns null.
IMHO, PackagedTextTemplate should set lastModifiedTime when accessing the
resource.
AFAICS, the cache code at the top of PackagedTextTemplate isn't used at all so
I think we can simply set it in the constructor when we access the stream. This
is what the attached patch does.
Any comment?
--
Guillaume
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
