[prev in list] [next in list] [prev in thread] [next in thread] 

List:       imp
Subject:    [imp] Adding to addressbook fails
From:       "W. Tasin" <tasin () fhm ! edu>
Date:       2003-02-01 22:01:51
[Download RAW message or body]

Hello,

adding a person by clicking the "addressbook-icon" in a message view 
fails for me (CVS-HEAD).
The following patch correct this (I hope this is the right way to handle 
the problem).

Ciao

Walter


["diff.patch" (text/plain)]

--- /home/tasin/horde/CVS-HORDE/horde/imp/message.php	Sat Feb  1 02:57:58 2003
+++ message.php	Sat Feb  1 22:32:09 2003
@@ -302,10 +302,13 @@
 /* For the self URL link, we can't trust the index in the query string as it
    may have changed if we deleted/copied/moved messages. We may need other
    stuff in the query string, so we need to do an add/remove of 'index'. */
-$selfURL = Horde::removeParameter(Horde::selfUrl(true), array('index', 'actionID'));
+$selfURL = Horde::removeParameter(Horde::selfUrl(true), array('index', 'actionID', \
'name', 'address'));  $selfURL = Horde::addParameter($selfURL, 'index', $index);
 $headersURL = Horde::removeParameter($selfURL, array('show_all_headers', \
'show_list_headers'));  
+/* Generate the message link. */
+$message_link = Horde::removeParameter($selfURL, array('start', 'show_all_headers', \
'show_list_headers')); +
 /* Determine previous message index. */
 if (($prev_msg = $imp_mailbox->messageIndices(-1))) {
     $prev_link = IMP_Search::generateSearchUrl('message.php', \
$prev_msg['thismailbox']); @@ -389,18 +392,18 @@
 }
 
 /* Build From address links. */
-$imp_headers->buildAddressLinks('from', $view_link, true, $addrIcon);
+$imp_headers->buildAddressLinks('from', $message_link, true, $addrIcon);
 
 /* Build To/Cc/Bcc links. */
 $address_headers = array('to' => 'toaddress', 'cc' => 'ccaddress', 'bcc' => \
'bccaddress');  foreach ($address_headers as $key => $val) {
-    if ($imp_headers->buildAddressLinks($key, $view_link, true, $addrIcon)) {
+    if ($imp_headers->buildAddressLinks($key, $message_link, true, $addrIcon)) {
         $msgAddresses .= $imp_headers->getOb($val);
     }
 }
 
 /* Build Reply-To address links. */
-if (($reply_to = $imp_headers->buildAddressLinks('reply_to', $view_link, false, \
$addrIcon))) { +if (($reply_to = $imp_headers->buildAddressLinks('reply_to', \
                $message_link, false, $addrIcon))) {
     if (!($from = $imp_headers->getValue('from')) || ($from != $reply_to)) {
         $imp_headers->setValue('Reply-to', $reply_to);
     } else {



-- 
IMP mailing list
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: imp-unsubscribe@lists.horde.org


[prev in list] [next in list] [prev in thread] [next in thread]