[ 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.