Changeset 5010 in subversion

Timestamp:

Aug 3, 2011 5:20:52 AM (22 months ago)

Author:

alec

Message:

• Added possibility to extend rcube_vcard's fieldmap
• Skip empty values in vCard export

Location:

trunk/roundcubemail/program

Files:

• include/rcube_contacts.php (modified) (1 diff)
• include/rcube_vcard.php (modified) (10 diffs)
• steps/addressbook/export.inc (modified) (1 diff)
• steps/addressbook/import.inc (modified) (1 diff)

trunk/roundcubemail/program/include/rcube_contacts.php

@@ -582 +582 @@
         $write_sql = array();
         $record = $this->get_record($id, true);
+console($save_cols);
         $save_cols = $this->convert_save_data($save_cols, $record);
-
+console($save_cols);
         foreach ($save_cols as $col => $value) {
             $write_sql[] = sprintf("%s=%s", $this->db->quoteIdentifier($col), $this->db->quote($value));

trunk/roundcubemail/program/include/rcube_vcard.php

@@ -34 +34 @@
     'N' => array(array('','','','','')),
   );
-  private $fieldmap = array(
+  static private $fieldmap = array(
     'phone'    => 'TEL',
     'birthday' => 'BDAY',
@@ -70 +70 @@
    * Constructor
    */
-  public function __construct($vcard = null, $charset = RCMAIL_CHARSET, $detect = false)
-  {
+  public function __construct($vcard = null, $charset = RCMAIL_CHARSET, $detect = false, $fieldmap = array())
+  {
+    if (!empty($fielmap))
+      $this->extend_fieldmap($fieldmap);
+
     if (!empty($vcard))
       $this->load($vcard, $charset, $detect);
@@ -147 +150 @@
 
     // convert from raw vcard data into associative data for Roundcube
-    foreach (array_flip($this->fieldmap) as $tag => $col) {
+    foreach (array_flip(self::$fieldmap) as $tag => $col) {
       foreach ((array)$this->raw[$tag] as $i => $raw) {
         if (is_array($raw)) {
           $k = -1;
           $key = $col;
-
-          $subtype = $typemap[$raw['type'][++$k]] ? $typemap[$raw['type'][$k]] : strtolower($raw['type'][$k]);
-          while ($k < count($raw['type']) && ($subtype == 'internet' || $subtype == 'pref'))
+          $subtype = '';
+
+          if (!empty($raw['type'])) {
             $subtype = $typemap[$raw['type'][++$k]] ? $typemap[$raw['type'][$k]] : strtolower($raw['type'][$k]);
+            while ($k < count($raw['type']) && ($subtype == 'internet' || $subtype == 'pref'))
+              $subtype = $typemap[$raw['type'][++$k]] ? $typemap[$raw['type'][$k]] : strtolower($raw['type'][$k]);
+          }
 
           // read vcard 2.1 subtype
@@ -168 +174 @@
 
           // force subtype if none set
-          if (preg_match('/^(email|phone|address|website)/', $key) && !$subtype)
+          if (!$subtype && preg_match('/^(email|phone|address|website)/', $key))
             $subtype = 'other';
 
@@ -221 +227 @@
   {
     if (!$fields)
-      $fields = array_merge(array_values($this->fieldmap), array_keys($this->immap), array('FN','N','ORG','NICKNAME','EMAIL','ADR','BDAY'));
+      $fields = array_merge(array_values(self::$fieldmap), array_keys($this->immap), array('FN','N','ORG','NICKNAME','EMAIL','ADR','BDAY'));
 
     foreach ($fields as $f)
@@ -324 +330 @@
           $type = $this->phonetypemap[$type];
 
-        if (($tag = $this->fieldmap[$field]) && (is_array($value) || strlen($value))) {
+        if (($tag = self::$fieldmap[$field]) && (is_array($value) || strlen($value))) {
           $index = count($this->raw[$tag]);
           $this->raw[$tag][$index] = (array)$value;
@@ -391 +397 @@
 
   /**
+   * Extends fieldmap definition
+   */
+  public function extend_fieldmap($map)
+  {
+    if (is_array($map))
+      self::$fieldmap = array_merge($map, self::$fieldmap);
+  }
+
+
+  /**
    * Factory method to import a vcard file
    *
@@ -421 +437 @@
       if (preg_match('/^END:VCARD$/i', $line)) {
         // parse vcard
-        $obj = new rcube_vcard(self::cleanup($vcard_block), $charset, true);
+        $obj = new rcube_vcard(self::cleanup($vcard_block), $charset, true, self::$fieldmap);
         if (!empty($obj->displayname) || !empty($obj->email))
           $out[] = $obj;
@@ -621 +637 @@
         }
 
+        // skip empty entries
+        if (self::is_empty($value))
+          continue;
+
         $vcard .= self::vcard_quote($type) . $attr . ':' . self::vcard_quote($value) . "\n";
       }
@@ -669 +689 @@
       return strtr($s, array("\r" => '', '\\\\' => '\\', '\n' => "\n", '\N' => "\n", '\,' => ',', '\;' => ';', '\:' => ':'));
     }
+  }
+
+
+  /**
+   * Check if vCard entry is empty: empty string or an array with
+   * all entries empty.
+   *
+   * @param mixed $value Attribute value (string or array)
+   *
+   * @return bool True if the value is empty, False otherwise
+   */
+  private static function is_empty($value)
+  {
+    foreach ((array)$value as $v) {
+      if (((string)$v) !== '') {
+        return false;
+      }
+    }
+
+    return true;
   }
 

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

@@ -78 +78 @@
     // copy values into vcard object
     else {
-        $vcard = new rcube_vcard($row['vcard']);
+        $vcard = new rcube_vcard();
+        $vcard->extend_fieldmap($CONTACTS->vcard_map);
+        $vcard->load($row['vcard']);
         $vcard->reset();
 

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

@@ -139 +139 @@
 
   // let rcube_vcard do the hard work :-)
-  $vcards = rcube_vcard::import(file_get_contents($_FILES['_file']['tmp_name']));
+  $vcard_o = new rcube_vcard();
+  $vcard_o->extend_fieldmap($CONTACTS->vcard_map);
+
+  $vcards = $vcard_o->import(file_get_contents($_FILES['_file']['tmp_name']));
 
   // no vcards detected