Changeset 4231 in subversion

Timestamp:

Nov 17, 2010 8:35:58 AM (3 years ago)

Author:

thomasb

Message:

Improve vcard field name assignments

Location:

branches/devel-addressbook

Files:

• program/include/rcube_contacts.php (modified) (1 diff)
• program/include/rcube_vcard.php (modified) (6 diffs)
• program/steps/addressbook/func.inc (modified) (1 diff)
• program/steps/addressbook/show.inc (modified) (3 diffs)
• skins/default/addressbook.css (modified) (2 diffs)

branches/devel-addressbook/program/include/rcube_contacts.php

@@ -483 +483 @@
         if ($sql_arr['vcard']) {
           $vcard = new rcube_vcard($sql_arr['vcard']);
-          $record += $vcard->get_assoc();
+          $record += $vcard->get_assoc() + $sql_arr;
         }
         else

branches/devel-addressbook/program/include/rcube_vcard.php

@@ -34 +34 @@
   );
   private $fieldmap = array('phone' => 'TEL', 'birthday' => 'BDAY', 'website' => 'URL', 'notes' => 'NOTE', 'email' => 'EMAIL', 'address' => 'ADR');
+  private $typemap = array('iPhone' => 'mobile', 'CELL' => 'mobile');
 
   public $business = false;
@@ -106 +107 @@
   {
     $out = array('name' => $this->displayname);
-    
+    $typemap = $this->typemap;
+    
+    // copy name fields to output array
     foreach (array('firstname','surname','middlename','nickname','organization') as $col)
       $out[$col] = $this->$col;
     
+    // convert from raw vcard data into associative data for Roundcube
     foreach (array_flip($this->fieldmap) as $tag => $col) {
       foreach ((array)$this->raw[$tag] as $i => $raw) {
         if (is_array($raw)) {
-          $k = 0;
+          $k = -1;
           $key = $col;
-          $subtype = strtolower($raw['type'][$k++]);
+          $subtype = $typemap[$raw['type'][++$k]] ?: strtolower($raw['type'][$k]);
           while ($k < count($raw['type']) && ($subtype == 'internet' || $subtype == 'pref'))
-            $subtype = strtolower($raw['type'][$k++]);
+            $subtype = $typemap[$raw['type'][++$k]] ?: strtolower($raw['type'][$k]);
           if ($subtype)
             $key .= ':' . $subtype;
-            
+
+          // split ADR values into assoc array
           if ($tag == 'ADR') {
             list(,, $value['street'], $value['locality'], $value['region'], $value['zipcode'], $value['country']) = $raw;
@@ -134 +139 @@
     }
     
+    // handle special IM fields as used by Apple
+    foreach (array('X-JABBER' => 'jabber', 'X-ICQ' => 'icq', 'X-MSN' => 'msn', 'X-AIM' => 'aim', 'X-YAHOO' => 'yahoo') as $tag => $type) {
+      foreach ((array)$this->raw[$tag] as $i => $raw) {
+        $out['im:'.$type][] = $raw[0];
+      }
+    }
+    
     return $out;
   }
@@ -152 +164 @@
    * @param string Field name
    * @param string Field value
-   * @param string Section name
-   */
-  public function set($field, $value, $section = 'HOME')
+   * @param string Type/section name
+   */
+  public function set($field, $value, $type = 'HOME')
   {
     static $touched = array();
+    $typemap = array_flip($this->typemap);
     
     switch ($field) {
@@ -184 +197 @@
           $this->raw['EMAIL'] = array();
         $index = count($this->raw['EMAIL']);
-        $this->raw['EMAIL'][$index] = array(0 => $value, 'type' => array_filter(array('INTERNET', $section)));
+        $this->raw['EMAIL'][$index] = array(0 => $value, 'type' => array_filter(array('INTERNET', $type)));
         break;
 
@@ -197 +210 @@
           $index = count($this->raw[$tag]);
           $this->raw[$tag][$index] = (array)$value;
-          if ($section)
-            $this->raw[$tag][$index]['type'] = array($section);
+          if ($type)
+            $this->raw[$tag][$index]['type'] = array(($typemap[$type] ?: $type));
         }
         break;

branches/devel-addressbook/program/steps/addressbook/func.inc

@@ -311 +311 @@
                     if ($RCMAIL->action == 'show') {
                         if (!empty($record[$col]))
-                            $fields .= html::span('namefield', Q($record[$col])) . " ";
+                            $fields .= html::span('namefield ' . $col, Q($record[$col])) . " ";
                     }
                     else {

branches/devel-addressbook/program/steps/addressbook/show.inc

@@ -74 +74 @@
               'address' => array(),
               'birthday' => array('size' => $i_size),
-              'website' => array('size' => $i_size),
+              'website' => array('size' => $i_size, 'value' => array()),
               'im' => array('size' => $i_size),
             ),
@@ -89 +89 @@
         ),
     );
-    
+/* FIXME: this destroys subtype assignments
     foreach ((array)$record as $field => $values) {
         if (strpos($field, 'email') === 0) {
@@ -101 +101 @@
             }
         }
+        else if (strpos($field, 'website') === 0) {
+            foreach ((array)$values as $url) {
+                $prefix = preg_match('![htfps]+://!', $url) ? '' : 'http://';
+                $form['info']['content']['website']['value'][] = html::a(array(
+                    'href' => $prefix . $url,
+                    'target' => '_blank',
+                    'class' => 'url',
+                ), Q($url));
+            }
+        }
     }
-
+*/
     return rcmail_contact_form($form, $record);
 }

branches/devel-addressbook/skins/default/addressbook.css

@@ -239 +239 @@
 }
 
-#contacthead span.namefield,
+#contacthead .names span.namefield,
 #contacthead input.ff_firstname,
 #contacthead input.ff_surname,
@@ -245 +245 @@
 {
         font-size: 140%;
+}
+
+#contacthead span.nickname:before,
+#contacthead span.nickname:after
+{
+        content: '"';
 }