Mailing-List: contact commits-help@roller.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@roller.apache.org
Date: Wed, 26 Feb 2014 09:25:20 +0000 (UTC)
From: "Glen Mazza (JIRA)" <jira@apache.org>
To: commits@roller.apache.org
Message-ID: <JIRA.12421403.1155126322851.107344.1393406720514@arcas>
In-Reply-To: <JIRA.12421403.1155126322851@arcas>
References: <JIRA.12421403.1155126322851@arcas>
Subject: [jira] [Closed] (ROL-1203) Popup comment pages cached the same as
 permalink pages
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/ROL-1203?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Glen Mazza closed ROL-1203.
---------------------------

    Resolution: Cannot Reproduce

Unsure what is meant by popup comments -- I don't think we have today.  Two of the three classes mentioned no longer exist.

> Popup comment pages cached the same as permalink pages
> ------------------------------------------------------
>
>                 Key: ROL-1203
>                 URL: https://issues.apache.org/jira/browse/ROL-1203
>             Project: Apache Roller
>          Issue Type: Bug
>          Components: Comments
>    Affects Versions: 2.3
>            Reporter: Richard Jones
>            Assignee: Roller Unassigned
>
> classes effected:
> org.apache.roller.presentation.WeblogPageRequest.java
> org.apache.roller.presentation.filters.WeblogPageCacheFilter.java
> org.apache.roller.presentation.filters.IfModifiedWeblogPageCacheFilter.java
> The page caching does not take into account the '&popup=true' parameter.  This means the following pages will be cached as the same
> /roller/page/handle?entry=new_entry_1
> and
> /roller/page/handle?entry=new_entry_1&popup=true
> This causes roller to return the permalink pages when the popup comments form was requested and viceversa.
> Resolution:
> Add 'private string weblogPopup' to WeblogPageRequest.java and change the 'generateKey()' methods in WeblogPageCacheFilter.java and IfModifiedWeblogPageCacheFilter.java to check for the popup parameter (I can provide the code if requested).


--
This message was sent by Atlassian JIRA
(v6.1.5#6160)