Browse Source

Fix forwarding from the Mailbox list. I posted to the Devel list about this on June 6, but got no response. We need to forward to the compose page, but instead it forwards back to itself ($PHP_SELF) which just makes it refresh the page. I am working on fixing one last issue with it now. When forwarding, unread messages appear to be marked as read in the cache, even when they aren't in message list (refreshing both frames still leaves them out of synch).

Jimmy Conner 21 years ago
parent
commit
fef09db6dc
1 changed files with 2 additions and 7 deletions
  1. 2 7
      src/right_main.php

+ 2 - 7
src/right_main.php

@@ -184,13 +184,8 @@ if (isset($aMailbox['FORWARD_SESSION'])) {
         sqsession_register($aMailbox,'aLastSelectedMailbox');
         sqsession_register($aMailbox,'aLastSelectedMailbox');
         session_write_close();
         session_write_close();
         // we have to redirect to the compose page
         // we have to redirect to the compose page
-        global $PHP_SELF;
-        if (!strpos($PHP_SELF,'?')) {
-            $location = $PHP_SELF.'?mailbox=INBOX&startMessage=1';
-        } else {
-            $location = $PHP_SELF;
-        }
-        $location = set_url_var($location, 'session',$aMailbox['FORWARD_SESSION'], false);
+        $location = SM_PATH . 'src/compose.php?mailbox='. urlencode($mailbox).
+                    '&session='.$aMailbox['FORWARD_SESSION'];
         header("Location: $location");
         header("Location: $location");
         exit;
         exit;
     }
     }