diff --git a/action/ajax.php b/action/ajax.php
--- a/action/ajax.php
+++ b/action/ajax.php
@@ -1,272 +1,274 @@
 <?php
 
 /**
  * DokuWiki DAVCard PlugIn - Ajax component
+ * @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
+ * @author  Andreas Böhler <dev@aboehler.at>
  */
 
 if(!defined('DOKU_INC')) die();
 
 class action_plugin_davcard_ajax extends DokuWiki_Action_Plugin {
 
     /**
      * @var helper_plugin_davcard
      */
     private $hlp = null;
 
     function __construct() {
         $this->hlp =& plugin_load('helper','davcard');
     }
 
     function register(Doku_Event_Handler $controller) {
         $controller->register_hook('AJAX_CALL_UNKNOWN', 'BEFORE', $this, 'handle_ajax_call_unknown');
     }
 
     function handle_ajax_call_unknown(&$event, $param) {
       if($event->data != 'plugin_davcard') return;
       
       $event->preventDefault();
       $event->stopPropagation();
       global $INPUT;
       
       $action = trim($INPUT->post->str('action'));
       $id = trim($INPUT->post->str('id'));
       $page = trim($INPUT->post->str('page'));
       $params = $INPUT->post->arr('params');
       if(isset($_SERVER['REMOTE_USER']) && !is_null($_SERVER['REMOTE_USER']))
         $user = $_SERVER['REMOTE_USER'];
       else
         $user = null;
       
       if(!checkSecurityToken())
       {
           echo "CSRF Attack.";
           return;
       }
       
       $data = array();
       
       $data['result'] = false;
       $data['html'] = $this->getLang('unknown_error');
       
       $acl = auth_quickaclcheck($id);
       if($acl > AUTH_READ)
       {
           $write = true;
       }
       elseif($acl < AUTH_READ)
       {
         $data['result'] = false;
         $data['html'] = $this->getLang('no_permission');
         // Overwrite $action to bypass switch statement below
         $action = 'invalid';
       }
       else 
       {
           $write = false;
       }
       
       // Parse the requested action
       switch($action)
       {
           // Add a new Contact
           case 'newContact':
               if($write && ($this->hlp->addContactEntryToAddressbookForPage($id, $user, $params) === true))
               {
                   $data['result'] = true;
               }
               else
               {
                   $data['result'] = false;
                   if(!$write)
                     $data['html'] = $this->getLang('no_permission');
                   else
                     $data['html'] = $this->getLang('error_adding');
               }
           break;
           
           // Retrieve contact details
           case 'getContactDetails':
               $contactdata = $this->hlp->getContactByUri($id, $params['uri']);
               if($contactdata['result'] === true)
               {
                   // When we support pictures for editing contacts,
                   // we need to use the following line: 
                   // $contactdata['photo'] = base64_encode($contactdata['photo']);
                   // For now, we just save bandwidth :)
                   unset($contactdata['photo']);
                   $data['result'] = true;
                   $data['contactdata'] = $contactdata;
               }
               else
               {
                   $data['result'] = false;
                   $data['html'] = sprintf($this->getLang('contact_not_found'), 'ID='.$id.' URI='.$params['uri']);
               }
           break;
 
           // Edit a contact
           case 'editContact':
               if($write && ($this->hlp->editContactEntryToAddressbookForPage($id, $user, $params['uri'], $params) === true))
               {
                   $data['result'] = true;
               }
               else 
               {
                   $data['result'] = false;
                   if(!$write)
                     $data['html'] = $this->getLang('no_permission');
                   else
                     $data['html'] = $this->getLang('error_editing');
               }
           break;
           // Delete a Contact
           case 'deleteContact':
               if($write && ($this->hlp->deleteContactEntryToAddressbookForPage($id, $user, $params['uri']) === true))
               {
                   $data['result'] = true;
               }
               else
               {
                   $data['result'] = false;
                   if(!$write)
                     $data['html'] = $this->getLang('no_permission');
                   else
                     $data['html'] = $this->getLang('error_deleting');
               }
           break;
           // Get AJAX popup
           case 'getContactAjax':
               $contactdata = $this->hlp->getContactByUri($id, $params['uri']);
               $cardpattern = $this->getConf('popup_content');
               if($contactdata['result'] === false)
               {
                   echo hsc($contactdata['formattedname']);
                   return;
               }
               echo '<div class="plugin_davcard_popup_container">';
               foreach($contactdata['photo'] as $data)
               {
                   if(isset($data['type']))
                     $type = $data['type'];
                   else
                     $type = '';
                   echo '<div class="plugin_davcard_popup_image">';
                   $imgdata = base64_encode($data['photo']);
                   $pattern = '/^(?:[;\/?:@&=+$,]|(?:[^\W_]|[-_.!~*\()\[\] ])|(?:%[\da-fA-F]{2}))*$/';
 
                   // PNG images
                   if($type == 'png')
                   {
                       $imgdata = 'data:image/png;base64,'.$imgdata;
                       echo '<img src="'.hsc($imgdata).'" alt="contact image" />';
                   }
                   // JPEG images
                   elseif(($type == 'jpeg') || ($type == 'jpg'))
                   {
                       $imgdata = 'data:image/jpeg;base64,'.$imgdata;
                       echo '<img src="'.hsc($imgdata).'" alt="contact image" />';
                   }
                   // GIF images
                   elseif($type == 'gif')
                   {
                       $imgdata = 'data:image/gif;base64,'.$imgdata;
                       echo '<img src="'.hsc($imgdata).'" alt="contact image" />';
                   }
                   // URLs (no type given)
                   elseif(preg_match( $pattern, $string ) == 1)
                   {
                       echo '<img src="'.hsc($data['photo']).'" alt="contact image" />';
                   }
                   echo '</div>';
               }
               echo '<div class="plugin_davcard_popup_content">';
               $contactname = explode(';', $contactdata['structuredname']);
               if(count($contactname) > 1)
               {
                   $cardpattern = str_replace('<DCLASTNAME>', $contactname[0], $cardpattern);
                   $cardpattern = str_replace('<DCFIRSTNAME>', $contactname[1], $cardpattern);
               }
               if(count($contactdata['addr']) > 0)
               {
                   foreach($contactdata['addr'] as $data)
                   {
                       if(isset($data['type']) && ($data['type'] == 'work'))
                         $prefix = 'WORK';
                       else
                         $prefix = 'PRIVATE';
                       $cardpattern = str_replace('<DC'.$prefix.'STREET>', $data['address'][2], $cardpattern);
                       $cardpattern = str_replace('<DC'.$prefix.'CITY>', $data['address'][3], $cardpattern);
                       $cardpattern = str_replace('<DC'.$prefix.'ZIP>', $data['address'][5], $cardpattern);
                       $cardpattern = str_replace('<DC'.$prefix.'COUNTRY>', $data['address'][6], $cardpattern);
                   }
               }
               if(count($contactdata['tel']) > 0)
               {
                   $telArr = array();
                   foreach($contactdata['tel'] as $data)
                   {
                       if(isset($data['type']))
                         $type = $data['type'];
                       else
                         $type = 'other';
                       $type = $data['type'];
                       $telArr[] = $this->getLang('tel'.$type).': '.$data['number'];
                   }
                   $telStr = implode(' \\\\ ', $telArr);
                   $cardpattern = str_replace('<DCPHONE>', $telStr, $cardpattern);
               }
               if(count($contactdata['mail']) > 0)
               {
                   $mailArr = array();
                   foreach($contactdata['mail'] as $data)
                   {
                       $mailArr[] = '[['.$data['mail'].']]';
                   }
                   $mailStr = implode(' \\\\ ', $mailArr);
                   $cardpattern = str_replace('<DCEMAIL>', $mailStr, $cardpattern);
               }
               if($contactdata['birthday'] != '')
               {
                   $date = DateTime::createFromFormat('Ymd', $contactdata['birthday']);
                   $dateStr = $date->format($this->getConf('date_format'));
                   $cardpattern = str_replace('<DCBIRTHDAY>', $dateStr, $cardpattern);
               }
               if($contactdata['note'] != '')
               {
                   $notestr = str_replace('\n', ' \\ ', $contactdata['note']);
                   $cardpattern = str_replace('<DCNOTE>', $notestr, $cardpattern);
               }
               if($contactdata['title'] != '')
               {
                   $cardpattern = str_replace('<DCTITLE>', $contactdata['title'], $cardpattern);
               }
               if($contactdata['url'] != '')
               {
                   $url = $contactdata['url'];
                   if(strpos($url, '://') === false)
                     $url = 'http://'.$url;
                   $url = '[['.$url.']]';
                   $cardpattern = str_replace('<DCWEBSITE>', $url, $cardpattern);
               }
               
               $replace_match = '/^.*<DC.*>.*$(?:\r\n|\n)?/m';
               $cardpattern = preg_replace($replace_match, '', $cardpattern);
               echo $this->render_text($cardpattern);
               echo '</div>';
               echo '</div>';
               return;
           break;
       }
     
       // If we are still here, JSON output is requested
       
       //json library of DokuWiki
       require_once DOKU_INC . 'inc/JSON.php';
       $json = new JSON();
  
       //set content type
       header('Content-Type: application/json');
       echo $json->encode($data);            
     }
  
 }              
diff --git a/action/cache.php b/action/cache.php
--- a/action/cache.php
+++ b/action/cache.php
@@ -1,51 +1,53 @@
 <?php
 
 /**
- * DokuWiki DAVCal PlugIn - Ajax component
+ * DokuWiki DAVCal PlugIn - Cache component
+ * @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
+ * @author  Andreas Böhler <dev@aboehler.at>
  */
 
 if(!defined('DOKU_INC')) die();
 
 class action_plugin_davcard_cache extends DokuWiki_Action_Plugin {
 
     function __construct() {
 
     }
 
     function register(Doku_Event_Handler $controller) {
         $controller->register_hook('PARSER_CACHE_USE', 'BEFORE', $this, 'handle_parser_cache_use');
     }
 
   function handle_parser_cache_use(Doku_Event $event, $param)
   {
       $cache = &$event->data;
       if(!isset($cache->page)) return;
       //purge only xhtml cache
       if($cache->mode != "xhtml") return;
       
       $meta = p_get_metadata($cache->page, 'plugin_davcard');
       if($meta === null)
         return;
       // Force re-caching if the webdavclient has synced
       if(isset($meta['webdavclient']))
       {
           $wdc =& plugin_load('helper', 'webdavclient');
           if(is_null($wdc))
             return;
           foreach($meta['webdavclient'] as $connectionId)
           {
             $cache->depends['files'][] = $wdc->getLastSyncChangeFileForConnection($connectionId);
           }
       }
       // Disable caching if there is a tabular addressbook
       if(isset($meta['addressbooks']) && in_array($cache->page, $meta['addressbooks']['id']))
       {
             $event->preventDefault();
             $event->stopPropagation();
             $event->result = false;
       }    
   }
  
 }
   
   
diff --git a/action/jsinfo.php b/action/jsinfo.php
--- a/action/jsinfo.php
+++ b/action/jsinfo.php
@@ -1,23 +1,25 @@
 <?php
 
 /**
  * DokuWiki DAVCard PlugIn - JSINFO component
+ * @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
+ * @author  Andreas Böhler <dev@aboehler.at>
  */
  
 if(!defined('DOKU_INC')) die();
 
 class action_plugin_davcard_jsinfo extends DokuWiki_Action_Plugin {
 
     function register(Doku_Event_Handler $controller) {
         $controller->register_hook('DOKUWIKI_STARTED', 'AFTER', $this, 'add_jsinfo_information');
     }
 
     /**
      * Add the language variable to the JSINFO variable
      */
     function add_jsinfo_information(Doku_Event $event, $param) {
       global $JSINFO;
  
       $JSINFO['plugin']['davcard']['sectok'] = getSecurityToken();
     }  
 }
diff --git a/helper.php b/helper.php
--- a/helper.php
+++ b/helper.php
@@ -1,645 +1,788 @@
 <?php
 /** 
   * Helper Class for the DAVCard plugin
   * This helper does the actual work.
-  * 
+  * @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
+  * @author  Andreas Böhler <dev@aboehler.at>
   */
   
 // must be run within Dokuwiki
 if(!defined('DOKU_INC')) die();
 
 class helper_plugin_davcard extends DokuWiki_Plugin {
   
   protected $sqlite = null;
   
   
   /**
     * Constructor to load the configuration
     */
   public function helper_plugin_davcard() {
     $this->sqlite =& plugin_load('helper', 'sqlite');
     global $conf;
 
     if(!$this->sqlite)
     {
         if($conf['allowdebug'])
             dbglog('This plugin requires the sqlite plugin. Please install it.');
         msg('This plugin requires the sqlite plugin. Please install it.');
         return;
     }
     
     if(!$this->sqlite->init('davcard', DOKU_PLUGIN.'davcard/db/'))
     {
         if($conf['allowdebug'])
             dbglog('Error initialising the SQLite DB for DAVCard');
         return;
     }
   }
   
+  /**
+   * Retrieve a contact by specifying details like the name
+   * 
+   * @param int $id The address book ID
+   * @param string $type The type to look for
+   * @param array $params The parameter array
+   * 
+   * @return array An array containing the results
+   */
   private function getContactByDetails($id, $type, $params = array())
   {
     $write = false;
     if(strpos($id, 'webdav://') === 0)
     {
         $wdc =& plugin_load('helper', 'webdavclient');
         if(is_null($wdc))
             return $this->getLang('no_wdc');
         $connectionId = str_replace('webdav://', '', $id);
         $settings = $wdc->getConnection($connectionId);
         
         if($settings === false)
             return array('formattedname' => $this->getLang('settings_not_found'), 'result' => false);
         if($settings['type'] !== 'contacts')
             return array('formattedname' => $this->getLang('wrong_type'), 'result' => false);
         
         $entries = $wdc->getAddressbookEntries($connectionId);
         $write = $settings['write'];
     }
     else 
     {
         $acl = auth_quickaclcheck($id);
         if($acl > AUTH_READ)
         {
             $write = true;
         }
         elseif($acl < AUTH_READ)
         {
             return array('formattedname' => $this->getLang('no_permission'), 'result' => false);
         }
         else 
         {
             $write = false;
         }
         $addressbookid = $this->getAddressbookIdForPage($id);
         $entries = $this->getAddressbookEntries($addressbookid);
     }
     foreach($entries as $entry)
     {
         switch($type)
         {
             case 'structuredname':
                 $contactdata = explode(';', strtolower($entry['structuredname']));
                 if(count($contactdata) < 2) // We need at least first and last name
                     return array('formattedname' => sprintf($this->getLang('contact_not_found'), $params['firstname']. ' '.$params['lastname']), 'result' => false);
                 if(($params['lastname'] != '') && 
                     ($contactdata[0] === $params['lastname']) 
                     || $params['lastname'] === '')
                 {
                     // last name matched or no last name given
                     if(($params['firstname'] != '') && 
                        ($contactdata[1] === $params['firstname']) 
                        || $params['firstname'] === '')
                     {
                         // first name matched too or no first name given
                         $info = $this->parseVcard($entry['contactdata'], $entry['uri'], $write);
                         return $info;
                     }
                 }
             break;
             case 'formattedname':
                 if(trim(strtolower($entry['formattedname'])) == $params['formattedname'])
                 {
                     $info = $this->parseVcard($entry['contactdata'], $entry['uri'], $write);
                     return $info;
                 }
             break;
             case 'email':
                 $info = $this->parseVcard($entry['contactdata'], $entry['uri'], $write);
                 foreach($info['mail'] as $data)
                 {
                     if(trim(strtolower($data['mail'])) === $params['email'])
                         return $info;
                 }
             break;
         }
     }
     return array('formattedname' => sprintf($this->getLang('contact_not_found'), $this->getLang('invalid_options')), 'result' => false);
   }
 
+
+  /**
+   * Retreive all address book entries
+   * 
+   * @param int $id The addressbook ID to retrieve
+   * 
+   * @return array All address book entries
+   */
   public function getAddressbookEntries($id)
   {
       $query = "SELECT contactdata, uri, formattedname, structuredname FROM addressbookobjects WHERE addressbookid = ? ORDER BY formattedname ASC";
       $res = $this->sqlite->query($query, $id);
       return $this->sqlite->res2arr($res);
   }
  
+  /**
+   * Retrieve a contact by the structured name
+   * 
+   * @param string $id The addressbook ID to work with
+   * @param string $firstname The contact's first name
+   * @param string $lastname The contact's last name
+   * 
+   * @return array The contact's details
+   */
   public function getContactByStructuredName($id, $firstname = '', $lastname = '')
   {
     return $this->getContactByDetails($id, 'structuredname', 
         array('firstname' => strtolower($firstname), 'lastname' => strtolower($lastname)));
   }
 
+  /**
+   * Retrieve a contact by e-mail address
+   * 
+   * @param string $id The address book ID
+   * @param string $email The E-Mail address
+   * 
+   * @return array The contact's details
+   */
   public function getContactByEmail($id, $email)
   {
       // FIXME: Maybe it's a good idea to save the e-mail in the database as well!
       return $this->getContactByDetails($id, 'email', array('email' => strtolower($email)));
   }
    
+  /**
+   * Retrieve a contact by formatted name
+   * 
+   * @param string $id The address book  ID
+   * @param string $name The contact's formatted name
+   * 
+   * @return array The contact's details
+   */
   public function getContactByFormattedName($id, $name)
   {
       return $this->getContactByDetails($id, 'formattedname', array('formattedname' => strtolower($name)));
   }
   
+  /**
+   * Retrieve a contact object by its URI
+   * 
+   * @param string $ID The address book ID
+   * @param string $uri The object URI
+   * 
+   * @return array An array containing the result
+   */
   public function getContactByUri($id, $uri)
   {
     $write = false;
     if(strpos($id, 'webdav://') === 0)
     {
         $wdc =& plugin_load('helper', 'webdavclient');
         if(is_null($wdc))
             return $this->getLang('no_wdc');
         $connectionId = str_replace('webdav://', '', $id);
         $settings = $wdc->getConnection($connectionId);
         
         if($settings === false)
             return array('formattedname' => $this->getLang('settings_not_found'), 'result' => false);
         if($settings['type'] !== 'contacts')
             return array('formattedname' => $this->getLang('wrong_type'), 'result' => false);
         
         $row = $wdc->getAddressbookEntryByUri($connectionId, $uri);
         $write = $settings['write'];
     }
     else
     {
         $acl = auth_quickaclcheck($id);
         if($acl > AUTH_READ)
         {
             $write = true;
         }
         elseif($acl < AUTH_READ)
         {
             return array('formattedname' => $this->getLang('no_permission'), 'result' => false);
         }
         else 
         {
             $write = false;
         }
         $addressbookid = $this->getAddressbookIdForPage($id);
         $row = $this->getAddressbookEntryByUri($addressbookid, $uri);       
     }
     
     if($row === false)
         return array('formattedname' => sprintf($this->getLang('contact_not_found'), 'ID='.$id.' URI='.$uri), 'result' => false);
     $info = $this->parseVcard($row['contactdata'], $row['uri'], $write);
     $info['result'] = true;
     return $info;
   }
   
+  /**
+   * Retrieve an address book entry by URI (low-level version)
+   * 
+   * @param int $id The address book ID
+   * @param string $uri The object URI
+   * 
+   * @return array The contact's details
+   */
   private function getAddressbookEntryByUri($id, $uri)
   {
       $query = "SELECT contactdata, addressbookid, etag, uri, formattedname, structuredname FROM addressbookobjects WHERE addressbookid = ? AND uri = ?";
       $res = $this->sqlite->query($query, $id, $uri);
       return $this->sqlite->res2row($res);
   }
   
-  
+  /**
+   * Set the addressbook name for a given page
+   * 
+   * @param string $name The name to set
+   * @param string $description The address book description
+   * @param int $id (optional) The page ID
+   * @param string $userid (optional) The user's ID
+   * 
+   * @return boolean true on success, otherwise false 
+   */
   public function setAddressbookNameForPage($name, $description, $id = null, $userid = null)
   {
       if(is_null($id))
       {
           global $ID;
           $id = $ID;
       }
       if(is_null($userid))
       {
         if(isset($_SERVER['REMOTE_USER']) && !is_null($_SERVER['REMOTE_USER']))
         {
           $userid = $_SERVER['REMOTE_USER'];
         }
         else
         {
           $userid = uniqid('davcard-');
         }
       }
       $bookid = $this->getAddressbookIdForPage($id);
       if($bookid === false)
         return $this->createAddressbookForPage($name, $description, $id, $userid);
       
       $query = "UPDATE addressbooks SET displayname = ?, description = ? WHERE id = ?";
       $res = $this->sqlite->query($query, $name, $description, $bookid);
       if($res !== false)
         return true;
       return false;
   }
   
+  /**
+   * Get the address book ID associated with a given page
+   * 
+   * @param string $id (optional) The page ID
+   * 
+   * @return mixed The address book ID or false
+   */
   public function getAddressbookIdForPage($id = null)
   {
       if(is_null($id))
       {
           global $ID;
           $id = $ID;
       }
       
       $query = "SELECT addressbookid FROM pagetoaddressbookmapping WHERE page = ?";
       $res = $this->sqlite->query($query, $id);
       $row = $this->sqlite->res2row($res);
       if(isset($row['addressbookid']))
       {
-        $calid = $row['addressbookid'];
-        return $calid;
+        $addrbkid = $row['addressbookid'];
+        return $addrbkid;
       }
       return false;
   }
   
+  /**
+   * Create a new address book for a given page
+   * 
+   * @param string $name The name of the new address book
+   * @param string $description The address book's description
+   * @param string $id (optional) The page ID
+   * @param string $userid (optional) The user's ID
+   * 
+   * @return boolean True on success, otherwise false
+   */
   public function createAddressbookForPage($name, $description, $id = null, $userid = null)
   {
       if(is_null($id))
       {
           global $ID;
           $id = $ID;
       }
       if(is_null($userid))
       {
         if(isset($_SERVER['REMOTE_USER']) && !is_null($_SERVER['REMOTE_USER']))
         {
           $userid = $_SERVER['REMOTE_USER'];
         }
         else
         {
           $userid = uniqid('davcard-');
         }
       }
       $values = array('principals/'.$userid, 
                       $name,
                       str_replace(array('/', ' ', ':'), '_', $id), 
                       $description,
                       1);
       $query = "INSERT INTO addressbooks (principaluri, displayname, uri, description, synctoken) ".
                "VALUES (?, ?, ?, ?, ?)";
       $res = $this->sqlite->query($query, $values);
       if($res === false)
         return false;
       
       // Get the new addressbook ID
       $query = "SELECT id FROM addressbooks WHERE principaluri = ? AND displayname = ? AND ".
                "uri = ? AND description = ? AND synctoken = ?";
       $res = $this->sqlite->query($query, $values);
       $row = $this->sqlite->res2row($res);
       
       // Update the pagetocalendarmapping table with the new calendar ID
       if(isset($row['id']))
       {
           $query = "INSERT INTO pagetoaddressbookmapping (page, addressbookid) VALUES (?, ?)";
           $res = $this->sqlite->query($query, $id, $row['id']);
           return ($res !== false);
       }
       
       return false;
   }
 
+  /**
+   * Delete a contact entry from an address book by URI
+   * 
+   * @param string $id The address book ID
+   * @param string $user The user's ID
+   * @param string $uri The object URI to delete
+   * 
+   * @return boolean True on success, otherwise false
+   */
   public function deleteContactEntryToAddressbookForPage($id, $user, $uri)
   {
       if(strpos($id, 'webdav://') === 0)
       {
         $wdc =& plugin_load('helper', 'webdavclient');
         if(is_null($wdc))
             return $this->getLang('no_wdc');
         $connectionId = str_replace('webdav://', '', $id);
         $settings = $wdc->getConnection($connectionId);
         
         if($settings === false)
             return array('formattedname' => $this->getLang('settings_not_found'), 'result' => false);
         if($settings['type'] !== 'contacts')
             return array('formattedname' => $this->getLang('wrong_type'), 'result' => false);
         
         return $wdc->deleteAddressbookEntry($connectionId, $uri);
       }
       
       $addressbookid = $this->getAddressbookIdForPage($id);
-      dbglog($addressbookid);
-      dbglog($uri);
       $query = "DELETE FROM addressbookobjects WHERE uri = ? AND addressbookid = ?";
-            dbglog($query);
       
       $res = $this->sqlite->query($query, $uri, $addressbookid);
       if($res !== false)
       {
           $this->updateSyncTokenLog($addressbookid, $uri, 'deleted');
           return true;
       }
       return false;
   }
 
+  /**
+   * Edit a contact for a given address book
+   * 
+   * @param string $id The address book ID
+   * @param string $user The user name
+   * @param string $uri The object URI
+   * @param array $params The new address book parameters
+   * 
+   * @return boolean True on success, otherwise false
+   */
   public function editContactEntryToAddressbookForPage($id, $user, $uri, $params)
   {
       require_once(DOKU_PLUGIN.'davcard/vendor/autoload.php');
       
       if(strpos($id, 'webdav://') === 0)
       {
         $wdc =& plugin_load('helper', 'webdavclient');
         if(is_null($wdc))
             return $this->getLang('no_wdc');
         $connectionId = str_replace('webdav://', '', $id);
         $settings = $wdc->getConnection($connectionId);
         
         if($settings === false)
             return array('formattedname' => $this->getLang('settings_not_found'), 'result' => false);
         if($settings['type'] !== 'contacts')
             return array('formattedname' => $this->getLang('wrong_type'), 'result' => false);
         
         $row = $wdc->getAddressbookEntryByUri($connectionId, $uri);
       }
       else
       {
         $addressbookid = $this->getAddressbookIdForPage($id);
         $row = $this->getAddressbookEntryByUri($addressbookid, $uri);       
       }
       $vcard = \Sabre\VObject\Reader::read($row['contactdata']);
       
       $vcard->remove('ADR');
       $vcard->remove('TEL');
       $vcard->remove('EMAIL');
       
       if(isset($params['phones']))
       {
           foreach($params['phones'] as $data)
           {
               $vcard->add('TEL', $data['number'], array('type' => $data['type']));
           }
       }
 
       if(isset($params['email']))
       {
           foreach($params['email'] as $data)
           {
               $vcard->add('EMAIL', $data['mail'], array('type' => $data['type']));
           }
       }
 
       if(isset($params['addresses']))
       {
           foreach($params['addresses'] as $data)
           {
               $vcard->add('ADR', array('', '', $data['street'], $data['city'], '', $data['zipcode'], $data['country']), array('type' => $data['type']));
           }
       }
       
       $structuredname = explode(';', (string)$vcard->N);
       $structuredname[0] = $params['lastname'];
       $structuredname[1] = $params['firstname'];
       $formattedname = $params['firstname'].' '.$params['lastname']; // FIXME: Make this configurable?
       $vcard->N = $structuredname;
       $vcard->FN = $formattedname;
       
       $contactdata = $vcard->serialize();
       
       if(strpos($id, 'webdav://') === 0)
       {
           return $wdc->editAddressbookEntry($connectionId, $uri, $contactdata);
       }
       else
       {
           $now = new \DateTime();
           $query = "UPDATE addressbookobjects SET contactdata = ?, lastmodified = ?, etag = ?, size = ?, formattedname = ?, structuredname = ? WHERE addressbookid = ? AND uri = ?";
           $res = $this->sqlite->query($query,
                                       $contactdata,
                                       $now->getTimestamp(),
                                       md5($contactdata),
                                       strlen($contactdata),
                                       $formattedname,
                                       implode(';', $structuredname),
                                       $addressbookid,
                                       $uri
                                       );
           if($res !== false)
           {
               $this->updateSyncTokenLog($addressbookid, $uri, 'modified');
               return true;
           }
       }
       return false;
   }
   
+  /**
+   * Add a new contact entry to an address book page
+   * 
+   * @param string $id The page ID
+   * @param string $user The user ID
+   * @param array $params The entry's parameters
+   * 
+   * @return boolean True on success, otherwise false
+   */
   public function addContactEntryToAddressbookForPage($id, $user, $params)
   {
       require_once(DOKU_PLUGIN.'davcard/vendor/autoload.php');
       $vcard = new \Sabre\VObject\Component\VCard();
       $formattedname = $params['firstname'].' '.$params['lastname']; // FIXME: Make this configurable?
       $structuredname = array($params['lastname'], $params['firstname'], '', '', '');
       $vcard->FN =  $formattedname;
       $vcard->N = $structuredname;
 
       if(isset($params['phones']))
       {
         foreach($params['phones'] as $data)
         {
             $vcard->add('TEL', $data['number'], array('type' => $data['type']));
         }
       }
       if(isset($params['email']))
       {
           foreach($params['email'] as $data)
           {
               $vcard->add('EMAIL', $data['mail'], array('type' => $data['type']));
           }
       }
       if(isset($params['addresses']))
       {
           foreach($params['addresses'] as $data)
           {
               $vcard->add('ADR', array('', '', $data['street'], $data['city'], '', $data['zipcode'], $data['country']), array('type' => $data['type']));
           }
       }
 
       $contactdata = $vcard->serialize();
       
       if(strpos($id, 'webdav://') === 0)
       {
           $wdc =& plugin_load('helper', 'webdavclient');
           if(is_null($wdc))
             return false;
           $connectionId = str_replace('webdav://', '', $id);
           return $wdc->addAddressbookEntry($connectionId, $contactdata);
       }
       else
       {
           $addressbookid = $this->getAddressbookIdForPage($id);
           $uri = uniqid('dokuwiki-').'.vcf';
           $now = new \DateTime();
           $query = "INSERT INTO addressbookobjects (contactdata, uri, addressbookid, lastmodified, etag, size, formattedname, structuredname) VALUES (?, ?, ?, ?, ?, ?, ?, ?)";
           $res = $this->sqlite->query($query, 
                                       $contactdata, 
                                       $uri, 
                                       $addressbookid, 
                                       $now->getTimestamp(), 
                                       md5($contactdata), 
                                       strlen($contactdata), 
                                       $formattedname, 
                                       implode(';', $structuredname)
                                       );
           // If successfully, update the sync token database
           if($res !== false)
           {
               $this->updateSyncTokenLog($addressbookid, $uri, 'added');
               return true;
           }
       }
       return false;
   }
   
+  /**
+   * Parse a VCard and extract important contact information
+   * 
+   * @param string $card The VCard data
+   * @param string $uri The object URI
+   * @param boolean $write Writable
+   * 
+   * @return array An array with parsed data
+   */
   public function parseVcard($card, $uri, $write)
   {
     require_once(DOKU_PLUGIN.'davcard/vendor/autoload.php');
       
     $vObject = \Sabre\VObject\Reader::read($card);
       
     $formattedname = '';
     $structuredname = '';
     $tel = array();
     $addr = array();
     $mail = array();
     $photo = array();
     $birthday = '';
     $note = '';
     $title = '';
     $url = '';
     
     if(isset($vObject->FN))
       $formattedname = (string)$vObject->FN;
     
     if(isset($vObject->N))
       $structuredname = join(';', $vObject->N->getParts());
     
     if(isset($vObject->TEL))
     {
         foreach($vObject->TEL as $number)
         {
             if(isset($number['TYPE']))
                 $tel[] = array('type' => strtolower((string)$number['TYPE']), 'number' => (string)$number);
             else
                 $tel[] = array('number' => (string)$number);
         }
     }
     
     if(isset($vObject->ADR))
     {
         foreach($vObject->ADR as $adr)
         {
             if(isset($adr['TYPE']))
                 $addr[] = array('type' => strtolower((string)$adr['TYPE']), 'address' => $adr->getParts());
             else
                 $addr[] = array('address' => $adr->getParts());
         }
     }
     
     if(isset($vObject->EMAIL))
     {
         foreach($vObject->EMAIL as $email)
         {
             if(isset($email['TYPE']))
                 $mail[] = array('type' => strtolower((string)$email['TYPE']), 'mail' => (string)$email);
             else
                 $mail[] = array('mail' => (string)$email);
         }
     }
     
     if(isset($vObject->PHOTO))
     {
         if(isset($vObject->PHOTO['TYPE']))
         {
             $photo[] = array('type' => strtolower((string)$vObject->PHOTO['TYPE']), 'photo' => (string)$vObject->PHOTO);
         }
         else
             $photo[] = array('photo' => (string)$vObject->PHOTO);
     }
     
     if(isset($vObject->BDAY))
     {
         $birthday = (string)$vObject->BDAY;
         $birthday = str_replace('-', '', $birthday);
     }
     
     if(isset($vObject->NOTE))
     {
         $note = (string)$vObject->NOTE;
     }
     
     if(isset($vObject->TITLE))
     {
         $title = (string)$vObject->TITLE;
     }
     
     if(isset($vObject->URL))
     {
         $url = (string)$vObject->URL;
     }
     
     return array(
         'formattedname' => $formattedname,
         'structuredname' => $structuredname,
         'tel' => $tel,
         'mail' => $mail,
         'addr' => $addr,
         'uri' => $uri,
         'photo' => $photo,
         'birthday' => $birthday,
         'note' => $note,
         'title' => $title,
         'url' => $url,
         'result' => true,
         'write' => $write
     );
   }
 
+  /**
+   * Retrieve the settings of a given address book
+   * 
+   * @param int $addressbookid The addressbook's ID
+   * 
+   * @return array The settings
+   */
   public function getAddressbookSettings($addressbookid)
   {
       $query = "SELECT id, principaluri, displayname, uri, description, synctoken FROM addressbooks WHERE id= ? ";
       $res = $this->sqlite->query($query, $addressbookid);
       $row = $this->sqlite->res2row($res);
       return $row;
   }
 
+  /**
+   * Retrieve the current synctoken for an address book
+   * 
+   * @param int $addressbookid The addressbook's ID
+   * 
+   * @return string The current synctoken
+   */
   public function getSyncTokenForAddressbook($addressbookid)
   {
       $row = $this->getAddressbookSettings($addressbookid);
       if(isset($row['synctoken']))
           return $row['synctoken'];
       return false;
   }
   
   /**
    * Helper function to convert the operation name to 
    * an operation code as stored in the database
    * 
    * @param string $operationName The operation name
    * 
    * @return mixed The operation code or false
    */
   public function operationNameToOperation($operationName)
   {
       switch($operationName)
       {
           case 'added':
               return 1;
           break;
           case 'modified':
               return 2;
           break;
           case 'deleted':
               return 3;
           break;
       }
       return false;
   }
+  
+  /**
+   * Update the synctoken log for a given address book
+   * 
+   * @param string $addressbookid The addressbook ID to work with
+   * @param string $uri The object URI that was modified
+   * @param string $operation The operation that was performed
+   * 
+   * @return boolean True on success, otherwise false
+   */
   private function updateSyncTokenLog($addressbookid, $uri, $operation)
   {
       $currentToken = $this->getSyncTokenForAddressbook($addressbookid);
       $operationCode = $this->operationNameToOperation($operation);
       if(($operationCode === false) || ($currentToken === false))
           return false;
       $values = array($uri,
                       $currentToken,
                       $addressbookid,
                       $operationCode
       );
       $query = "INSERT INTO addressbookchanges (uri, synctoken, addressbookid, operation) VALUES(?, ?, ?, ?)";
       $res = $this->sqlite->query($query, $uri, $currentToken, $addressbookid, $operationCode);
       if($res === false)
         return false;
       $currentToken++;
       $query = "UPDATE addressbooks SET synctoken = ? WHERE id = ?";
       $res = $this->sqlite->query($query, $currentToken, $addressbookid);
       return ($res !== false);
   }
  
 }
diff --git a/plugin.info.txt b/plugin.info.txt
--- a/plugin.info.txt
+++ b/plugin.info.txt
@@ -1,7 +1,7 @@
 base	davcard
 author  Andreas Boehler
 email   dev@aboehler.at
-date    2016-05-28
+date    2016-07-01
 name    Addressbook PlugIn with CardDAV client support
 desc    Show contact information from a CardDAV address book (needs webdavclient)
 url     http://www.dokuwiki.org/plugin:davcard
diff --git a/syntax/book.php b/syntax/book.php
--- a/syntax/book.php
+++ b/syntax/book.php
@@ -1,305 +1,313 @@
 <?php
 /**
  * DokuWiki Plugin DAVCard (Book Syntax Component)
  *
  * @license GPL 2 http://www.gnu.org/licenses/gpl-2.0.html
  * @author  Andreas Böhler <dev@aboehler.at>
  */
 
 // must be run within Dokuwiki
 if(!defined('DOKU_INC')) die();
 
 if(!defined('DOKU_PLUGIN')) define('DOKU_PLUGIN',DOKU_INC.'lib/plugins/');
 require_once(DOKU_PLUGIN.'syntax.php');
 
 class syntax_plugin_davcard_book extends DokuWiki_Syntax_Plugin {
     
     protected $hlp = null;
     
     // Load the helper plugin
     public function syntax_plugin_davcard_book() {  
         $this->hlp =& plugin_load('helper', 'davcard');     
     }
     
     
     /**
      * What kind of syntax are we?
      */
     function getType(){
         return 'substition';
     }
 
     /**
      * What about paragraphs?
      */
     function getPType(){
         return 'normal';
     }
 
     /**
      * Where to sort in?
      */
     function getSort(){
         return 165;
     }
 
     /**
      * Connect pattern to lexer
      */
     function connectTo($mode) {
         $this->Lexer->addSpecialPattern('\{\{davcardbook>[^}]*\}\}',$mode,'plugin_davcard_book');
     }
 
     /**
      * Handle the match
      */
     function handle($match, $state, $pos, Doku_Handler $handler){
         global $ID;
         $options = trim(substr($match,14,-2));
         $options = explode(',', $options);
         $data = array('name' => $ID,
                       'description' => $this->getLang('created_by_davcard'),
                       'id' => array(),
                       'filter' => array(),
                       );
         foreach($options as $option)
         {
             list($key, $val) = explode('=', $option);
             $key = strtolower(trim($key));
             $val = trim($val);
             switch($key)
             {
                 case 'filter':
                     list($k, $v) = explode(':', strtolower($val), 2);
                     $data['filter'][$k] = $v;
                 break;
                 case 'id':
                     if(!in_array($val, $data['id']))
                         $data['id'][] = $val;
                 break;
                 default:
                     $data[$key] = $val;
             }
         }
         // Handle the default case when the user didn't enter a different ID
         if(empty($data['id']))
         {
             $data['id'] = array($ID);
         }
         // Only update the addressbook name/description if the ID matches the page ID.
         // Otherwise, the addressbook is included in another page and we don't want
         // to interfere with its data.
         if(in_array($ID, $data['id']))
         {
             if(isset($_SERVER['REMOTE_USER']) && !is_null($_SERVER['REMOTE_USER']))
                 $username = $_SERVER['REMOTE_USER'];
             else
                 $username = uniqid('davcard-');
             $this->hlp->setAddressbookNameForPage($data['name'], $data['description'], $ID, $username);
         }
 
         $meta = p_get_metadata($ID, 'plugin_davcard');
         if(is_null($meta))
             $meta = array();
         $meta['addressbooks'] = $data;
         // Add webdavclient information so that we can disable caching if need be
         foreach($data['id'] as $addrbkid)
         {
             if(strpos($addrbkid, 'webdav://') === 0)
             {
                 $connectionId = str_replace('webdav://', '', $addrbkid);
                 if(!is_array($meta['webdavclient']))
                     $meta['webdavclient'] = array();
                 if(!in_array($addrbkid, $meta['webdavclient']))
                     $meta['webdavclient'][] = $connectionId;
             }
         }
         p_set_metadata($ID, array('plugin_davcard' => $meta));
         return $data;
     }
     
     /**
      * Create output
      */
     function render($format, Doku_Renderer $R, $data) 
     {
         global $ID;
         if($format !== 'xhtml')
             return false;
         if(in_array($ID, $data['id']))
         {        
             $R->doc .= '<div class="davcardAddressbookAddNew"><a href="#" class="davcardAddressbookAddNew">'.$this->getLang('add_new').'</a></div>';
         }        
         // FIXME: Add new is not yet permission checked and does not support
         // included address books!
         
         $R->doc .= '<div id="davcardAddressbookList" data-addressbookpage="'.$ID.'">';
         $R->doc .= '<table class="davcardAddressbookTable">';
         $R->doc .= '<tr><th>'.$this->getLang('name').'</th><th>'.$this->getLang('address').'</th><th>'.$this->getLang('phone').'</th><th>'.$this->getLang('email').'</th></tr>';
         foreach($data['id'] as $id)
         {
             $write = false;
             if(strpos($id, 'webdav://') === 0)
             {
                 $wdc =& plugin_load('helper', 'webdavclient');
                 if(is_null($wdc))
                 {
                     echo $this->getLang('no_wdc');
                     continue;
                 }
                 $connectionId = str_replace('webdav://', '', $id);
                 $settings = $wdc->getConnection($connectionId);
         
                 if($settings === false)
                 {
                     echo $this->getLang('settings_not_found');
                     continue;
                 }
                 if($settings['type'] !== 'contacts')
                 {
                     echo $this->getLang('wrong_type');
                     continue;
                 }
                 $entries = $wdc->getAddressbookEntries($connectionId);
                 $write = $settings['write'];
             }
             else
             {
                 $acl = auth_quickaclcheck($id);
                 if($acl > AUTH_READ)
                 {
                     $write = true;
                 }
                 elseif($acl < AUTH_READ)
                 {
                     continue;
                 }
                 else 
                 {
                     $write = false;
                 }
                 $addressbookid = $this->hlp->getAddressbookIdForPage($id);
                 $entries = $this->hlp->getAddressbookEntries($addressbookid);
             }
             foreach($entries as $entry)
             {
-                $contactdata = $this->hlp->parseVcard($entry['contactdata'], $entry['uri']);
+                $contactdata = $this->hlp->parseVcard($entry['contactdata'], $entry['uri'], $write);
                 if(!$this->contactFilterMatch($data['filter'], $contactdata))
                     continue;
                 $R->doc .= '<tr><td><a href="#" class="plugin_davcard_edit_vcard" data-davcardid="'.$id.'" data-davcarduri="'.$entry['uri'].'" data-write="'.($write ? 'true' : 'false').'">'.$entry['formattedname'].'</a></td><td>';
                 if(count($contactdata['addr']) > 0)
                 {
                     $R->doc .= '<span class="adr">';
                     foreach($contactdata['addr'] as $dat)
                     {
                         if(isset($dat['type']))
                             $type = $dat['type'];
                         else
                             $type = 'other';
                         $R->doc .= '<span class="type">'.$this->getLang('adr'.strtolower($type)).'</span>';
                         if($dat['address'][2] != '')
                         {
                             $R->doc .= '<span class="street-address">'.$dat['address'][2].'</span><br>';
                         }
                         if($dat['address'][5] != '')
                         {
                             $R->doc .= '<span class="postal-code">'.$dat['address'][5].' </span>';
                         }
                         if($dat['address'][3] != '')
                         {
                             $R->doc .= '<span class="locality">'.$dat['address'][3].'</span><br>';
                         }
         
                         if($dat['address'][6] != '')
                         {
                             $R->doc .= '<span class="country-name">'.$dat['address'][6].'</span>';
                         }
                     }
                     $R->doc .= '</span>';
                 }
                 $R->doc .= '</td><td>';
                 if(count($contactdata['tel']) > 0)
                 {
                     $R->doc .= '<span class="tel">';
                     foreach($contactdata['tel'] as $dat)
                     {
                         if(isset($dat['type']))
                             $type = $dat['type'];
                         else
                             $type = 'other';
                         $R->doc .= '<span class="type">'.$this->getLang('tel'.strtolower($type)).' </span>';
                         $R->doc .= $dat['number'].'<br>';
                     }
                     $R->doc .= '</span>';
                 }
                 $R->doc .= '</td><td>';
                 if(count($contactdata['mail']) > 0)
                 {
                     foreach($contactdata['mail'] as $dat)
                     {
                         $R->doc .= '<span class="email">'.$dat['mail'].'</span><br>';
                     }
                 }
                 $R->doc .= '</td></tr>';
             }
         }
         $R->doc .= '</table>';
         $R->doc .= '</div>';
         
     }
 
+/**
+ * Check if a contact matches a given filter pattern
+ * 
+ * @param array $filter The filter array
+ * @param array $contactdata The contact's data to match
+ * 
+ * @return true on success, otherwise false
+ */
     private function contactFilterMatch($filter, $contactdata)
     {
         if(empty($filter))
             return true;
         
         foreach($filter as $type => $params)
         {
             $params = '/'.$params.'/i';
             switch($type)
             {
                 case 'name':
                     if(preg_match($params, $contactdata['formattedname']) !== 1)
                         return false;
                 break;
                 case 'mail':
                     $found = false;
                     foreach($contactdata['mail'] as $dat)
                     {
                         if(preg_match($params, $dat['mail']) === 1)
                             $found = true;
                     }
                     if(!$found)
                         return false;
                 break;
                 case 'address':
                     $found = false;
                     foreach($contactdata['addr'] as $dat)
                     {
                         foreach($dat['address'] as $da)
                         {
                             if(preg_match($params, $da) === 1)
                                 $found = true;
                         }
                     }
                     if(!$found)
                         return false;
                 break;
                 case 'tel':
                     $found = false;
                     foreach($contactdata['tel'] as $dat)
                     {
                         if(preg_match($params, $dat['number']) === 1)
                             $found = true;
                     }
                     if(!$found)
                         return false;
                 break;
             }
         }
         return true;
     }
    
 }
 
 // vim:ts=4:sw=4:et:enc=utf-8: