From: pdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Fri, 25 Nov 2005 09:55:15 +0000 (+0000)
Subject: Fixed first message losing correct URI
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=84bb1d31ebd2334ce82086c65964c55055df33da;hp=8da3b983467ddfedb45991fc5fdede1f1baaa5f6;p=squirrelmail.git

Fixed first message losing correct URI


git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@10406 7612ce4b-ef26-0410-bec9-ea0150e637f0
---

diff --git a/functions/mailbox_display.php b/functions/mailbox_display.php
index 43b98f75..f2474c73 100644
--- a/functions/mailbox_display.php
+++ b/functions/mailbox_display.php
@@ -426,6 +426,10 @@ function fetchMessageHeaders($imapConnection, &$aMailbox) {
  * @author Marc Groot Koerkamp
  */
 function prepareMessageList(&$aMailbox, $aProps) {
+
+    /* Globalize link attributes so plugins can share in modifying them */
+    global $link, $title, $target, $onclick, $link_extra;
+
     /* retrieve the properties */
     $my_email_address = (isset($aProps['email'])) ? $aProps['email'] : false;
     $highlight_list   = (isset($aProps['config']['highlight_list'])) ? $aProps['config']['highlight_list'] : false;
@@ -598,10 +602,11 @@ function prepareMessageList(&$aMailbox, $aProps) {
                         // TODO, $sTargetModule should be a query parameter so that we can use a single entrypoint
                         $link = $sTargetModule.'.php?' . implode('&amp;',$aQuery);
 
-                        // globalize link attributes so plugins can share in 
-                        // modifying them; plugins are responsible for sharing 
-                        // nicely (such as for setting the target, etc)
-                        global $link, $title, $target, $onclick, $link_extra;
+                        // see top of this function for which attributes are available
+                        // in the global scope for plugin use (like $link, $target, 
+                        // $onclick, $link_extra, $title, and so forth)
+                        // plugins are responsible for sharing nicely (such as for 
+                        // setting the target, etc)
                         do_hook('subject_link', array($iPageOffset, $sSearch, $aSearch));
                     }
                     $value = (trim($value)) ? $value : _("(no subject)");