Changeset 4965 in subversion

Timestamp:

Jul 25, 2011 7:48:50 AM (22 months ago)

Author:

alec

Message:

• Fixed display name on contact lists

Location:

trunk/roundcubemail/program

Files:

• include/rcube_addressbook.php (modified) (2 diffs)
• steps/addressbook/func.inc (modified) (1 diff)
• steps/addressbook/save.inc (modified) (1 diff)

trunk/roundcubemail/program/include/rcube_addressbook.php

@@ -435 +435 @@
      *
      * @param array  Hash array with contact data as key-value pairs
+     * @param bool   The name will be used on the list
+     *
      * @return string Display name
      */
-    public static function compose_display_name($contact)
+    public static function compose_display_name($contact, $list_mode = false)
     {
         $contact = rcmail::get_instance()->plugins->exec_hook('contact_displayname', $contact);
@@ -447 +449 @@
         // use email address part for name
         $email = is_array($contact['email']) ? $contact['email'][0] : $contact['email'];
+
         if ($email && (empty($fn) || $fn == $email)) {
+            // Use full email address on contacts list
+            if ($list_mode)
+                return $email;
+
             list($emailname) = explode('@', $email);
             if (preg_match('/(.*)[\.\-\_](.*)/', $emailname, $match))

trunk/roundcubemail/program/steps/addressbook/func.inc

@@ -295 +295 @@
 
         // format each col
-        foreach ($a_show_cols as $col)
-            $a_row_cols[$col] = Q($row[$col]);
+        foreach ($a_show_cols as $col) {
+            $val = $row[$col];
+            if ($val == '' && $col == 'name') {
+                $val = rcube_addressbook::compose_display_name($row, true);
+            }
+            $a_row_cols[$col] = Q($val);
+        }
 
         $OUTPUT->command($prefix.'add_contact_row', $row['ID'], $a_row_cols);

trunk/roundcubemail/program/steps/addressbook/save.inc

@@ -189 +189 @@
     $record = $CONTACTS->get_record($newcid ? $newcid : $cid, true);
     $record['email'] = reset($CONTACTS->get_col_values('email', $record, true));
-    if (!$record['name'])
-      $record['name'] = $record['email'];
+    if (empty($record['name']))
+      $record['name']  = rcube_addressbook::compose_display_name($record, true);
 
     foreach (array('name', 'email') as $col)