[#7748] setting $RECOMPOSE to true not possible when logged out
Summary setting $RECOMPOSE to true not possible when logged out
Queue IMP
Queue Version 4.3
Type Bug
State Resolved
Priority 2. Medium
Owners Michael Slusarz <slusarz (at) horde (dot) org>
Requester coleman (at) boulder (dot) nist (dot) gov
Created 12/02/2008 (214 days ago)
Due
Updated 12/14/2008 (202 days ago)
Assigned 12/02/2008 (214 days ago)
Resolved 12/14/2008 (202 days ago)
Attachments
Milestone
Patch No

History
12/14/2008 Chuck Hagenbuch State ⇒ Resolved
 
12/02/2008 Michael Slusarz Comment #4
State ⇒ Feedback
Reply to this comment
Yes - it has already been fixed in IMP 5 by using VFS instead of the 
old method.

The IMP 4.x way of doing things breaks down in multiple ways - if 
using the non-standard login page; if maintenance is performed on 
login; if not using IMP auth.... It is pretty much useless.  But I've 
tried to make it work at least for the situation where IMP is 
authenticating through the login page with no maintenance.  Try the 
patch (link is in the ticket history) and see if it helps.

12/02/2008 CVS Commit Comment #3 Reply to this comment
12/02/2008 Chuck Hagenbuch Comment #2
Assigned to Michael Slusarz
State ⇒ Assigned
Reply to this comment
I'm guessing this is moot because of the rewrite to use the VFS?
12/02/2008 coleman (at) boulder (dot) nist (dot) gov Comment #1
Patch ⇒
Milestone ⇒
Summary ⇒ setting $RECOMPOSE to true not possible when logged out
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Queue ⇒ IMP
Reply to this comment
User will lose content in the compose window when logged out because 
the section of code in imp/lib/base.php where $RECOMPOSE is set to 
true can never be executed because $compose_page will always be false 
when logged out.

On line 53 $compose_page will be set to false because $_SESSION['imp'] 
will be null after being logged out.

$compose_page = (isset($_SESSION['imp']['viewmode']) && 
$_SESSION['imp']['viewmode'] == 'imp') && strstr($_SERVER['PHP_SELF'], 
'compose.php');

Line 102 and 122  will never be executed since lines 101 and 121 will 
be false due to the results of line 53