Changeset 4269 in subversion

Timestamp:

Nov 25, 2010 1:53:14 PM (2 years ago)

Author:

thomasb

Message:

Complete implementation of rcube_kolab_contacts; add localization texts for proprietary contact fields

Location:

trunk/plugins/kolab_addressbook

Files:

• kolab_addressbook.php (modified) (2 diffs)
• localization (added)
• localization/en_US.inc (added)
• rcube_kolab_contacts.php (modified) (16 diffs)

trunk/plugins/kolab_addressbook/kolab_addressbook.php

@@ -24 +24 @@
         // load required plugin
         $this->require_plugin('kolab_core');
+        
+        $this->add_texts('localization');
         
         // register hooks
@@ -122 +124 @@
             // TODO: add more Kolab-specific fields
             
-            // TODO: re-order fields
+            // re-order fields according to the coltypes list
+            $block = array();
+            $contacts = reset($this->sources);
+            foreach ($contacts->coltypes as $col => $prop) {
+                if (isset($p['form']['info']['content'][$col]))
+                    $block[$col] = $p['form']['info']['content'][$col];
+            }
+            
+            $p['form']['info']['content'] = $block;
         }
         

trunk/plugins/kolab_addressbook/rcube_kolab_contacts.php

@@ -28 +28 @@
       'department'   => array('limit' => 1),
       'gender'       => array('limit' => 1),
-      'birthday'     => array('limit' => 1),
+      'initials'     => array('type' => 'text', 'size' => 6, 'limit' => 1, 'label' => 'kolab_addressbook.initials'),
       'email'        => array('subtypes' => null),
       'phone'        => array(),
@@ -34 +34 @@
       'website'      => array('limit' => 1, 'subtypes' => null),
       'address'      => array('limit' => 2, 'subtypes' => array('home','business')),
+      'birthday'     => array('limit' => 1),
+      'anniversary'  => array('type' => 'date', 'size' => 12, 'limit' => 1, 'label' => 'kolab_addressbook.anniversary'),
+      // TODO: define more Kolab-specific fields such as: office-location, profession, manager-name, assistant, spouse-name, children, language, latitude, longitude, pgp-publickey, free-busy-url
       'notes'        => array(),
-      // define additional coltypes
-      'initials'     => array('type' => 'text', 'size' => 6, 'limit' => 1),
-      'anniversary'  => array('type' => 'date', 'size' => 12, 'limit' => 1),
-      // TODO: define more Kolab-specific fields such as: office-location, profession, manager-name, assistant, spouse-name, children, language, latitude, longitude, pgp-publickey, free-busy-url
     );
     
@@ -49 +48 @@
     private $contacts;
     private $distlists;
+    private $groupmembers;
     private $id2uid;
     private $filter;
@@ -86 +86 @@
         $this->coltypes['phone']['subtypes'] = $format->_phone_types;
         $this->coltypes['address']['subtypes'] = $format->_address_types;
-        $this->coltypes['anniversary']['label'] = rcube_label('anniversary');
+        
+        // set localized labels for proprietary cols
+        foreach ($this->coltypes as $col => $prop) {
+            if (is_string($prop['label']))
+                $this->coltypes[$col]['label'] = rcube_label($prop['label']);
+        }
         
         // fetch objects from the given IMAP folder
@@ -209 +214 @@
     public function search($fields, $value, $strict=false, $select=true)
     {
+        // search by ID
+        if ($fields == $this->primary_key) {
+            return $this->get_record($value);
+        }
+        
         // TODO: currently not implemented
         return new rcube_result_set(0, ($this->list_page-1) * $this->page_size);
@@ -269 +279 @@
         $this->_fetch_groups();
         
-        foreach ($this->distlists as $gid => $group) {
-            foreach ($group['member'] as $member) {
-                if ($member['ID'] == $id)
-                    $out[$gid] = $group['last-name'];
-            }
+        foreach ((array)$this->groupmembers[$id] as $gid) {
+            if ($group = $this->distlists[$gid])
+                $out[$gid] = $group['last-name'];
         }
         
@@ -375 +383 @@
     {
         $this->_fetch_contacts();
+        $this->_fetch_groups();
         
         if (!is_array($ids))
@@ -392 +401 @@
                 }
                 else {
-                    // TODO: remove from distribution lists
-                    unset($this->contacts[$id], $this->id2uid[$id]);
+                    // remove from distribution lists
+                    foreach ((array)$this->groupmembers[$id] as $gid)
+                        $this->remove_from_group($gid, $id);
+                    
+                    // clear internal cache
+                    unset($this->contacts[$id], $this->id2uid[$id], $this->groupmembers[$id]);
                     $count++;
                 }
@@ -433 +446 @@
     function create_group($name)
     {
-        // TODO: implement this
-        return false;
-    }
-
-    /**
-     * Delete the given group and all linked group members
-     *
-     * @param string Group identifier
-     * @return boolean True on success, false if no data was changed
-     */
-    function delete_group($gid)
-    {
-        // TODO: implement this
-        return false;
-    }
-
-    /**
-     * Rename a specific contact group
-     *
-     * @param string Group identifier
-     * @param string New name to set for this group
-     * @return boolean New name on success, false if no data was changed
-     */
-    function rename_group($gid, $newname)
-    {
-        $this->_fetch_groups();
-        $list = $this->distlists[$gid];
-        
-        if ($newname != $list['last-name']) {
-            $list['last-name'] = $newname;
-            $saved = $this->liststorage->save($list, $list['uid']);
-        }
+        $this->_fetch_groups();
+        $result = false;
+        
+        $list = array(
+            'uid' => $this->liststorage->generateUID(),
+            'last-name' => $name,
+            'member' => array(),
+        );
+        $saved = $this->liststorage->save($list);
 
         if (PEAR::isError($saved)) {
@@ -474 +464 @@
             return false;
         }
+        else {
+            $id = md5($list['uid']);
+            $this->distlists[$record['ID']] = $list;
+            $result = array('id' => $id, 'name' => $name);
+        }
+
+        return $result;
+    }
+
+    /**
+     * Delete the given group and all linked group members
+     *
+     * @param string Group identifier
+     * @return boolean True on success, false if no data was changed
+     */
+    function delete_group($gid)
+    {
+        $this->_fetch_groups();
+        $result = false;
+        
+        if ($list = $this->distlists[$gid])
+            $deleted = $this->liststorage->delete($list['uid']);
+
+        if (PEAR::isError($deleted)) {
+            raise_error(array(
+              'code' => 600, 'type' => 'php',
+              'file' => __FILE__, 'line' => __LINE__,
+              'message' => "Error deleting distribution-list object from the Kolab server:" . $deleted->getMessage()),
+            true, false);
+        }
+        else
+            $result = true;
+        
+        return $result;
+    }
+
+    /**
+     * Rename a specific contact group
+     *
+     * @param string Group identifier
+     * @param string New name to set for this group
+     * @return boolean New name on success, false if no data was changed
+     */
+    function rename_group($gid, $newname)
+    {
+        $this->_fetch_groups();
+        $list = $this->distlists[$gid];
+        
+        if ($newname != $list['last-name']) {
+            $list['last-name'] = $newname;
+            $saved = $this->liststorage->save($list, $list['uid']);
+        }
+
+        if (PEAR::isError($saved)) {
+            raise_error(array(
+              'code' => 600, 'type' => 'php',
+              'file' => __FILE__, 'line' => __LINE__,
+              'message' => "Error saving distribution-list object to Kolab server:" . $saved->getMessage()),
+            true, false);
+            return false;
+        }
 
         return $newname;
@@ -497 +548 @@
         $list = $this->distlists[$gid];
 
-        foreach ($list['member'] as $i => $member)
+        foreach ((array)$list['member'] as $i => $member)
             $exists[] = $member['ID'];
-            
+        
         // substract existing assignments from list
         $ids = array_diff($ids, $exists);
-        
+
         foreach ($ids as $contact_id) {
             if ($uid = $this->id2uid[$contact_id]) {
@@ -513 +564 @@
                     );
                 }
+                $this->groupmembers[$contact_id][] = $gid;
                 $added++;
             }
@@ -569 +621 @@
         }
         else {
+            // remove group assigments in local cache
+            foreach ($ids as $id) {
+                $j = array_search($gid, $this->groupmembers[$id]);
+                unset($this->groupmembers[$id][$j]);
+            }
             $this->distlists[$gid] = $list;
             return true;
@@ -603 +660 @@
     {
         if (!isset($this->distlists)) {
-            $this->distlists = array();
+            $this->distlists = $this->groupmembers = array();
             foreach ((array)$this->liststorage->getObjects() as $record) {
                 // FIXME: folders without any distribution-list objects return contacts instead ?!
@@ -609 +666 @@
                     continue;
                 $record['ID'] = md5($record['uid']);
-                foreach ($record['member'] as $i => $member)
-                    $record['member'][$i]['ID'] = md5($member['uid']);
+                foreach ((array)$record['member'] as $i => $member) {
+                    $mid = md5($member['uid']);
+                    $record['member'][$i]['ID'] = $mid;
+                    $this->groupmembers[$mid][] = $record['ID'];
+                }
                 $this->distlists[$record['ID']] = $record;
             }
@@ -638 +698 @@
         foreach ((array)$record['email'] as $i => $email)
             $out['email'][] = $email['smtp-address'];
+            
+        if (!$record['email'] && $record['emails'])
+            $out['email'] = preg_split('/,\s*/', $record['emails']);
 
         foreach ((array)$record['phone'] as $i => $phone)