[MAGNOLIA-352] Fix for access ctrl on public site breaks preview mode Created: 27/Apr/05  Updated: 05/Nov/12  Resolved: 27/Apr/05

Status: Closed
Project: Magnolia
Component/s: core
Affects Version/s: 2.01, 2.1 Final
Fix Version/s: 2.1.5

Type: Bug Priority: Minor
Reporter: Andreas Weder Assignee: Unassigned
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

any


Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

The fix applied on both the trunk and the 2.02 branch implemented for the public site access problem causes the showPreview() in info.magnolia.cms.util.Resource to longer work as expected. As an effect, preview mode is no longer persistent: if you switch into preview mode, then surf to another page, that page shows up in edit mode again.

The problem probably is with authenticate() in info.magnolia.cms.servlets.EntryServlet: the session of a particular user seems to be invalidated there, which causes the "mgnlPreview" attribute to be removed - preview mode is then no longer active. I've compared a previous working copy of magnolia with the current one of the 2.02 branch and found this to be the only significant change.

This bug is also related to a recent patch I submitted to fix showPreview of the info.magnolia.cms.util.Resource class. However, I think, the patch is not the problem.



 Comments   
Comment by Sameer Charles [ 27/Apr/05 ]

its invalid for 2.1 trunk
I fixed it for 2.02 branch http://svn.magnolia.info/svn/magnolia/branches/magnolia2.02/

Generated at Mon Feb 12 03:16:34 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.