[bugs] [Bug 1068] New -

bugs@bugs.horde.org bugs@bugs.horde.org
Wed Oct 9 04:31:39 2002


http://bugs.horde.org/show_bug.cgi?id=1068

*** shadow/1068	Wed Oct  9 00:31:39 2002
--- shadow/1068.tmp.32619	Wed Oct  9 00:31:39 2002
***************
*** 0 ****
--- 1,47 ----
+ Bug#: 1068
+ Product: Horde
+ Version: 2.2 Stable
+ Platform: All Browsers
+ OS/Version: All
+ Status: NEW   
+ Resolution: 
+ Severity: normal
+ Priority: P2
+ Component: IMP
+ Area: BUILD
+ AssignedTo: chuck@horde.org                            
+ ReportedBy: link@mcs.anl.gov               
+ URL: 
+ Summary: 
+ 
+ 
+ The 'Add to Adressbook' link that appears in IMP's message view never contains 
+ the 'Name' of the person, just the email.  This cause the addressbook entry to 
+ only contain the email address for both, the Name and email.  Here is the very 
+ simple patch applied to imp/message.php (line 86 from cvs TRELENG_3 as of 10/8, 
+ the first arg to addParameter should be $curr_link, not $add_link)
+ 
+ Joe
+ 
+ Index: message.php
+ ===================================================================
+ RCS file: /repository/imp/message.php,v
+ retrieving revision 2.257.2.33
+ diff -u -r2.257.2.33 message.php
+ --- message.php 15 Sep 2002 22:32:05 -0000      2.257.2.33
+ +++ message.php 9 Oct 2002 03:17:16 -0000
+ @@ -83,7 +83,7 @@
+                     manager is available. */
+                  if (isset($add_link)) {
+                      $curr_link = Horde::addParameter($add_link, 'name=' . 
+ urlencode($ob->personal));
+ -                    $curr_link = Horde::addParameter($add_link, 'address=' . 
+ urlencode($inner));
+ +                    $curr_link = Horde::addParameter($curr_link, 'address=' . 
+ urlencode($inner));
+                      $ret .= Horde::link($curr_link, sprintf(_("Add to 
+ Addressbook (%s)"), $inner));
+                      $ret .= Horde::img('addressbook-
+ blue.gif', 'hspace="2"') . '</a>';
+                  }
+