[
https://issues.apache.org/jira/browse/WICKET-3315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12980460#action_12980460
]
Hudson commented on WICKET-3315:
--------------------------------
Integrated in Apache Wicket 1.4.x #383 (See
[https://hudson.apache.org/hudson/job/Apache%20Wicket%201.4.x/383/])
> 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
> Assignee: Peter Ertl
> Priority: Minor
> Fix For: 1.4.16, 1.5-M4
>
> Attachments: WICKET-3315.patch
>
>
> 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.
