Dolibarr / dolibarr

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Backend/BackendInterface.php

Indentation +163 added lines, -163 removed lines

@@ -19,176 +19,176 @@
  */
 interface BackendInterface
 {
-    /**
-     * Returns the list of addressbooks for a specific user.
-     *
-     * Every addressbook should have the following properties:
-     *   id - an arbitrary unique id
-     *   uri - the 'basename' part of the url
-     *   principaluri - Same as the passed parameter
-     *
-     * Any additional clark-notation property may be passed besides this. Some
-     * common ones are :
-     *   {DAV:}displayname
-     *   {urn:ietf:params:xml:ns:carddav}addressbook-description
-     *   {http://calendarserver.org/ns/}getctag
-     *
-     * @param string $principalUri
-     *
-     * @return array
-     */
-    public function getAddressBooksForUser($principalUri);
+	/**
+	 * Returns the list of addressbooks for a specific user.
+	 *
+	 * Every addressbook should have the following properties:
+	 *   id - an arbitrary unique id
+	 *   uri - the 'basename' part of the url
+	 *   principaluri - Same as the passed parameter
+	 *
+	 * Any additional clark-notation property may be passed besides this. Some
+	 * common ones are :
+	 *   {DAV:}displayname
+	 *   {urn:ietf:params:xml:ns:carddav}addressbook-description
+	 *   {http://calendarserver.org/ns/}getctag
+	 *
+	 * @param string $principalUri
+	 *
+	 * @return array
+	 */
+	public function getAddressBooksForUser($principalUri);
 
-    /**
-     * Updates properties for an address book.
-     *
-     * The list of mutations is stored in a Sabre\DAV\PropPatch object.
-     * To do the actual updates, you must tell this object which properties
-     * you're going to process with the handle() method.
-     *
-     * Calling the handle method is like telling the PropPatch object "I
-     * promise I can handle updating this property".
-     *
-     * Read the PropPatch documentation for more info and examples.
-     *
-     * @param string $addressBookId
-     */
-    public function updateAddressBook($addressBookId, \Sabre\DAV\PropPatch $propPatch);
+	/**
+	 * Updates properties for an address book.
+	 *
+	 * The list of mutations is stored in a Sabre\DAV\PropPatch object.
+	 * To do the actual updates, you must tell this object which properties
+	 * you're going to process with the handle() method.
+	 *
+	 * Calling the handle method is like telling the PropPatch object "I
+	 * promise I can handle updating this property".
+	 *
+	 * Read the PropPatch documentation for more info and examples.
+	 *
+	 * @param string $addressBookId
+	 */
+	public function updateAddressBook($addressBookId, \Sabre\DAV\PropPatch $propPatch);
 
-    /**
-     * Creates a new address book.
-     *
-     * This method should return the id of the new address book. The id can be
-     * in any format, including ints, strings, arrays or objects.
-     *
-     * @param string $principalUri
-     * @param string $url          just the 'basename' of the url
-     *
-     * @return mixed
-     */
-    public function createAddressBook($principalUri, $url, array $properties);
+	/**
+	 * Creates a new address book.
+	 *
+	 * This method should return the id of the new address book. The id can be
+	 * in any format, including ints, strings, arrays or objects.
+	 *
+	 * @param string $principalUri
+	 * @param string $url          just the 'basename' of the url
+	 *
+	 * @return mixed
+	 */
+	public function createAddressBook($principalUri, $url, array $properties);
 
-    /**
-     * Deletes an entire addressbook and all its contents.
-     *
-     * @param mixed $addressBookId
-     */
-    public function deleteAddressBook($addressBookId);
+	/**
+	 * Deletes an entire addressbook and all its contents.
+	 *
+	 * @param mixed $addressBookId
+	 */
+	public function deleteAddressBook($addressBookId);
 
-    /**
-     * Returns all cards for a specific addressbook id.
-     *
-     * This method should return the following properties for each card:
-     *   * carddata - raw vcard data
-     *   * uri - Some unique url
-     *   * lastmodified - A unix timestamp
-     *
-     * It's recommended to also return the following properties:
-     *   * etag - A unique etag. This must change every time the card changes.
-     *   * size - The size of the card in bytes.
-     *
-     * If these last two properties are provided, less time will be spent
-     * calculating them. If they are specified, you can also ommit carddata.
-     * This may speed up certain requests, especially with large cards.
-     *
-     * @param mixed $addressbookId
-     *
-     * @return array
-     */
-    public function getCards($addressbookId);
+	/**
+	 * Returns all cards for a specific addressbook id.
+	 *
+	 * This method should return the following properties for each card:
+	 *   * carddata - raw vcard data
+	 *   * uri - Some unique url
+	 *   * lastmodified - A unix timestamp
+	 *
+	 * It's recommended to also return the following properties:
+	 *   * etag - A unique etag. This must change every time the card changes.
+	 *   * size - The size of the card in bytes.
+	 *
+	 * If these last two properties are provided, less time will be spent
+	 * calculating them. If they are specified, you can also ommit carddata.
+	 * This may speed up certain requests, especially with large cards.
+	 *
+	 * @param mixed $addressbookId
+	 *
+	 * @return array
+	 */
+	public function getCards($addressbookId);
 
-    /**
-     * Returns a specfic card.
-     *
-     * The same set of properties must be returned as with getCards. The only
-     * exception is that 'carddata' is absolutely required.
-     *
-     * If the card does not exist, you must return false.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     *
-     * @return array
-     */
-    public function getCard($addressBookId, $cardUri);
+	/**
+	 * Returns a specfic card.
+	 *
+	 * The same set of properties must be returned as with getCards. The only
+	 * exception is that 'carddata' is absolutely required.
+	 *
+	 * If the card does not exist, you must return false.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 *
+	 * @return array
+	 */
+	public function getCard($addressBookId, $cardUri);
 
-    /**
-     * Returns a list of cards.
-     *
-     * This method should work identical to getCard, but instead return all the
-     * cards in the list as an array.
-     *
-     * If the backend supports this, it may allow for some speed-ups.
-     *
-     * @param mixed $addressBookId
-     *
-     * @return array
-     */
-    public function getMultipleCards($addressBookId, array $uris);
+	/**
+	 * Returns a list of cards.
+	 *
+	 * This method should work identical to getCard, but instead return all the
+	 * cards in the list as an array.
+	 *
+	 * If the backend supports this, it may allow for some speed-ups.
+	 *
+	 * @param mixed $addressBookId
+	 *
+	 * @return array
+	 */
+	public function getMultipleCards($addressBookId, array $uris);
 
-    /**
-     * Creates a new card.
-     *
-     * The addressbook id will be passed as the first argument. This is the
-     * same id as it is returned from the getAddressBooksForUser method.
-     *
-     * The cardUri is a base uri, and doesn't include the full path. The
-     * cardData argument is the vcard body, and is passed as a string.
-     *
-     * It is possible to return an ETag from this method. This ETag is for the
-     * newly created resource, and must be enclosed with double quotes (that
-     * is, the string itself must contain the double quotes).
-     *
-     * You should only return the ETag if you store the carddata as-is. If a
-     * subsequent GET request on the same card does not have the same body,
-     * byte-by-byte and you did return an ETag here, clients tend to get
-     * confused.
-     *
-     * If you don't return an ETag, you can just return null.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     * @param string $cardData
-     *
-     * @return string|null
-     */
-    public function createCard($addressBookId, $cardUri, $cardData);
+	/**
+	 * Creates a new card.
+	 *
+	 * The addressbook id will be passed as the first argument. This is the
+	 * same id as it is returned from the getAddressBooksForUser method.
+	 *
+	 * The cardUri is a base uri, and doesn't include the full path. The
+	 * cardData argument is the vcard body, and is passed as a string.
+	 *
+	 * It is possible to return an ETag from this method. This ETag is for the
+	 * newly created resource, and must be enclosed with double quotes (that
+	 * is, the string itself must contain the double quotes).
+	 *
+	 * You should only return the ETag if you store the carddata as-is. If a
+	 * subsequent GET request on the same card does not have the same body,
+	 * byte-by-byte and you did return an ETag here, clients tend to get
+	 * confused.
+	 *
+	 * If you don't return an ETag, you can just return null.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 * @param string $cardData
+	 *
+	 * @return string|null
+	 */
+	public function createCard($addressBookId, $cardUri, $cardData);
 
-    /**
-     * Updates a card.
-     *
-     * The addressbook id will be passed as the first argument. This is the
-     * same id as it is returned from the getAddressBooksForUser method.
-     *
-     * The cardUri is a base uri, and doesn't include the full path. The
-     * cardData argument is the vcard body, and is passed as a string.
-     *
-     * It is possible to return an ETag from this method. This ETag should
-     * match that of the updated resource, and must be enclosed with double
-     * quotes (that is: the string itself must contain the actual quotes).
-     *
-     * You should only return the ETag if you store the carddata as-is. If a
-     * subsequent GET request on the same card does not have the same body,
-     * byte-by-byte and you did return an ETag here, clients tend to get
-     * confused.
-     *
-     * If you don't return an ETag, you can just return null.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     * @param string $cardData
-     *
-     * @return string|null
-     */
-    public function updateCard($addressBookId, $cardUri, $cardData);
+	/**
+	 * Updates a card.
+	 *
+	 * The addressbook id will be passed as the first argument. This is the
+	 * same id as it is returned from the getAddressBooksForUser method.
+	 *
+	 * The cardUri is a base uri, and doesn't include the full path. The
+	 * cardData argument is the vcard body, and is passed as a string.
+	 *
+	 * It is possible to return an ETag from this method. This ETag should
+	 * match that of the updated resource, and must be enclosed with double
+	 * quotes (that is: the string itself must contain the actual quotes).
+	 *
+	 * You should only return the ETag if you store the carddata as-is. If a
+	 * subsequent GET request on the same card does not have the same body,
+	 * byte-by-byte and you did return an ETag here, clients tend to get
+	 * confused.
+	 *
+	 * If you don't return an ETag, you can just return null.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 * @param string $cardData
+	 *
+	 * @return string|null
+	 */
+	public function updateCard($addressBookId, $cardUri, $cardData);
 
-    /**
-     * Deletes a card.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     *
-     * @return bool
-     */
-    public function deleteCard($addressBookId, $cardUri);
+	/**
+	 * Deletes a card.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 *
+	 * @return bool
+	 */
+	public function deleteCard($addressBookId, $cardUri);
 }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Backend/AbstractBackend.php

Indentation +18 added lines, -18 removed lines

@@ -17,22 +17,22 @@
  */
 abstract class AbstractBackend implements BackendInterface
 {
-    /**
-     * Returns a list of cards.
-     *
-     * This method should work identical to getCard, but instead return all the
-     * cards in the list as an array.
-     *
-     * If the backend supports this, it may allow for some speed-ups.
-     *
-     * @param mixed $addressBookId
-     *
-     * @return array
-     */
-    public function getMultipleCards($addressBookId, array $uris)
-    {
-        return array_map(function ($uri) use ($addressBookId) {
-            return $this->getCard($addressBookId, $uri);
-        }, $uris);
-    }
+	/**
+	 * Returns a list of cards.
+	 *
+	 * This method should work identical to getCard, but instead return all the
+	 * cards in the list as an array.
+	 *
+	 * If the backend supports this, it may allow for some speed-ups.
+	 *
+	 * @param mixed $addressBookId
+	 *
+	 * @return array
+	 */
+	public function getMultipleCards($addressBookId, array $uris)
+	{
+		return array_map(function ($uri) use ($addressBookId) {
+			return $this->getCard($addressBookId, $uri);
+		}, $uris);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      */
     public function getMultipleCards($addressBookId, array $uris)
     {
-        return array_map(function ($uri) use ($addressBookId) {
+        return array_map(function($uri) use ($addressBookId) {
             return $this->getCard($addressBookId, $uri);
         }, $uris);
     }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Backend/PDO.php

Spacing +1 added lines, -1 removed lines

@@ -101,7 +101,7 @@
             '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description',
         ];
 
-        $propPatch->handle($supportedProperties, function ($mutations) use ($addressBookId) {
+        $propPatch->handle($supportedProperties, function($mutations) use ($addressBookId) {
             $updates = [];
             foreach ($mutations as $property => $newValue) {
                 switch ($property) {

Indentation +517 added lines, -517 removed lines

@@ -19,521 +19,521 @@
  */
 class PDO extends AbstractBackend implements SyncSupport
 {
-    /**
-     * PDO connection.
-     *
-     * @var PDO
-     */
-    protected $pdo;
-
-    /**
-     * The PDO table name used to store addressbooks.
-     */
-    public $addressBooksTableName = 'addressbooks';
-
-    /**
-     * The PDO table name used to store cards.
-     */
-    public $cardsTableName = 'cards';
-
-    /**
-     * The table name that will be used for tracking changes in address books.
-     *
-     * @var string
-     */
-    public $addressBookChangesTableName = 'addressbookchanges';
-
-    /**
-     * Sets up the object.
-     */
-    public function __construct(\PDO $pdo)
-    {
-        $this->pdo = $pdo;
-    }
-
-    /**
-     * Returns the list of addressbooks for a specific user.
-     *
-     * @param string $principalUri
-     *
-     * @return array
-     */
-    public function getAddressBooksForUser($principalUri)
-    {
-        $stmt = $this->pdo->prepare('SELECT id, uri, displayname, principaluri, description, synctoken FROM '.$this->addressBooksTableName.' WHERE principaluri = ?');
-        $stmt->execute([$principalUri]);
-
-        $addressBooks = [];
-
-        foreach ($stmt->fetchAll() as $row) {
-            $addressBooks[] = [
-                'id' => $row['id'],
-                'uri' => $row['uri'],
-                'principaluri' => $row['principaluri'],
-                '{DAV:}displayname' => $row['displayname'],
-                '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
-                '{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
-                '{http://sabredav.org/ns}sync-token' => $row['synctoken'] ? $row['synctoken'] : '0',
-            ];
-        }
-
-        return $addressBooks;
-    }
-
-    /**
-     * Updates properties for an address book.
-     *
-     * The list of mutations is stored in a Sabre\DAV\PropPatch object.
-     * To do the actual updates, you must tell this object which properties
-     * you're going to process with the handle() method.
-     *
-     * Calling the handle method is like telling the PropPatch object "I
-     * promise I can handle updating this property".
-     *
-     * Read the PropPatch documentation for more info and examples.
-     *
-     * @param string $addressBookId
-     */
-    public function updateAddressBook($addressBookId, PropPatch $propPatch)
-    {
-        $supportedProperties = [
-            '{DAV:}displayname',
-            '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description',
-        ];
-
-        $propPatch->handle($supportedProperties, function ($mutations) use ($addressBookId) {
-            $updates = [];
-            foreach ($mutations as $property => $newValue) {
-                switch ($property) {
-                    case '{DAV:}displayname':
-                        $updates['displayname'] = $newValue;
-                        break;
-                    case '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description':
-                        $updates['description'] = $newValue;
-                        break;
-                }
-            }
-            $query = 'UPDATE '.$this->addressBooksTableName.' SET ';
-            $first = true;
-            foreach ($updates as $key => $value) {
-                if ($first) {
-                    $first = false;
-                } else {
-                    $query .= ', ';
-                }
-                $query .= ' '.$key.' = :'.$key.' ';
-            }
-            $query .= ' WHERE id = :addressbookid';
-
-            $stmt = $this->pdo->prepare($query);
-            $updates['addressbookid'] = $addressBookId;
-
-            $stmt->execute($updates);
-
-            $this->addChange($addressBookId, '', 2);
-
-            return true;
-        });
-    }
-
-    /**
-     * Creates a new address book.
-     *
-     * @param string $principalUri
-     * @param string $url          just the 'basename' of the url
-     *
-     * @return int Last insert id
-     */
-    public function createAddressBook($principalUri, $url, array $properties)
-    {
-        $values = [
-            'displayname' => null,
-            'description' => null,
-            'principaluri' => $principalUri,
-            'uri' => $url,
-        ];
-
-        foreach ($properties as $property => $newValue) {
-            switch ($property) {
-                case '{DAV:}displayname':
-                    $values['displayname'] = $newValue;
-                    break;
-                case '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description':
-                    $values['description'] = $newValue;
-                    break;
-                default:
-                    throw new DAV\Exception\BadRequest('Unknown property: '.$property);
-            }
-        }
-
-        $query = 'INSERT INTO '.$this->addressBooksTableName.' (uri, displayname, description, principaluri, synctoken) VALUES (:uri, :displayname, :description, :principaluri, 1)';
-        $stmt = $this->pdo->prepare($query);
-        $stmt->execute($values);
-
-        return $this->pdo->lastInsertId(
-            $this->addressBooksTableName.'_id_seq'
-        );
-    }
-
-    /**
-     * Deletes an entire addressbook and all its contents.
-     *
-     * @param int $addressBookId
-     */
-    public function deleteAddressBook($addressBookId)
-    {
-        $stmt = $this->pdo->prepare('DELETE FROM '.$this->cardsTableName.' WHERE addressbookid = ?');
-        $stmt->execute([$addressBookId]);
-
-        $stmt = $this->pdo->prepare('DELETE FROM '.$this->addressBooksTableName.' WHERE id = ?');
-        $stmt->execute([$addressBookId]);
-
-        $stmt = $this->pdo->prepare('DELETE FROM '.$this->addressBookChangesTableName.' WHERE addressbookid = ?');
-        $stmt->execute([$addressBookId]);
-    }
-
-    /**
-     * Returns all cards for a specific addressbook id.
-     *
-     * This method should return the following properties for each card:
-     *   * carddata - raw vcard data
-     *   * uri - Some unique url
-     *   * lastmodified - A unix timestamp
-     *
-     * It's recommended to also return the following properties:
-     *   * etag - A unique etag. This must change every time the card changes.
-     *   * size - The size of the card in bytes.
-     *
-     * If these last two properties are provided, less time will be spent
-     * calculating them. If they are specified, you can also ommit carddata.
-     * This may speed up certain requests, especially with large cards.
-     *
-     * @param mixed $addressbookId
-     *
-     * @return array
-     */
-    public function getCards($addressbookId)
-    {
-        $stmt = $this->pdo->prepare('SELECT id, uri, lastmodified, etag, size FROM '.$this->cardsTableName.' WHERE addressbookid = ?');
-        $stmt->execute([$addressbookId]);
-
-        $result = [];
-        while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
-            $row['etag'] = '"'.$row['etag'].'"';
-            $row['lastmodified'] = (int) $row['lastmodified'];
-            $result[] = $row;
-        }
-
-        return $result;
-    }
-
-    /**
-     * Returns a specific card.
-     *
-     * The same set of properties must be returned as with getCards. The only
-     * exception is that 'carddata' is absolutely required.
-     *
-     * If the card does not exist, you must return false.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     *
-     * @return array
-     */
-    public function getCard($addressBookId, $cardUri)
-    {
-        $stmt = $this->pdo->prepare('SELECT id, carddata, uri, lastmodified, etag, size FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri = ? LIMIT 1');
-        $stmt->execute([$addressBookId, $cardUri]);
-
-        $result = $stmt->fetch(\PDO::FETCH_ASSOC);
-
-        if (!$result) {
-            return false;
-        }
-
-        $result['etag'] = '"'.$result['etag'].'"';
-        $result['lastmodified'] = (int) $result['lastmodified'];
-
-        return $result;
-    }
-
-    /**
-     * Returns a list of cards.
-     *
-     * This method should work identical to getCard, but instead return all the
-     * cards in the list as an array.
-     *
-     * If the backend supports this, it may allow for some speed-ups.
-     *
-     * @param mixed $addressBookId
-     *
-     * @return array
-     */
-    public function getMultipleCards($addressBookId, array $uris)
-    {
-        $query = 'SELECT id, uri, lastmodified, etag, size, carddata FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri IN (';
-        // Inserting a whole bunch of question marks
-        $query .= implode(',', array_fill(0, count($uris), '?'));
-        $query .= ')';
-
-        $stmt = $this->pdo->prepare($query);
-        $stmt->execute(array_merge([$addressBookId], $uris));
-        $result = [];
-        while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
-            $row['etag'] = '"'.$row['etag'].'"';
-            $row['lastmodified'] = (int) $row['lastmodified'];
-            $result[] = $row;
-        }
-
-        return $result;
-    }
-
-    /**
-     * Creates a new card.
-     *
-     * The addressbook id will be passed as the first argument. This is the
-     * same id as it is returned from the getAddressBooksForUser method.
-     *
-     * The cardUri is a base uri, and doesn't include the full path. The
-     * cardData argument is the vcard body, and is passed as a string.
-     *
-     * It is possible to return an ETag from this method. This ETag is for the
-     * newly created resource, and must be enclosed with double quotes (that
-     * is, the string itself must contain the double quotes).
-     *
-     * You should only return the ETag if you store the carddata as-is. If a
-     * subsequent GET request on the same card does not have the same body,
-     * byte-by-byte and you did return an ETag here, clients tend to get
-     * confused.
-     *
-     * If you don't return an ETag, you can just return null.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     * @param string $cardData
-     *
-     * @return string|null
-     */
-    public function createCard($addressBookId, $cardUri, $cardData)
-    {
-        $stmt = $this->pdo->prepare('INSERT INTO '.$this->cardsTableName.' (carddata, uri, lastmodified, addressbookid, size, etag) VALUES (?, ?, ?, ?, ?, ?)');
-
-        $etag = md5($cardData);
-
-        $stmt->execute([
-            $cardData,
-            $cardUri,
-            time(),
-            $addressBookId,
-            strlen($cardData),
-            $etag,
-        ]);
-
-        $this->addChange($addressBookId, $cardUri, 1);
-
-        return '"'.$etag.'"';
-    }
-
-    /**
-     * Updates a card.
-     *
-     * The addressbook id will be passed as the first argument. This is the
-     * same id as it is returned from the getAddressBooksForUser method.
-     *
-     * The cardUri is a base uri, and doesn't include the full path. The
-     * cardData argument is the vcard body, and is passed as a string.
-     *
-     * It is possible to return an ETag from this method. This ETag should
-     * match that of the updated resource, and must be enclosed with double
-     * quotes (that is: the string itself must contain the actual quotes).
-     *
-     * You should only return the ETag if you store the carddata as-is. If a
-     * subsequent GET request on the same card does not have the same body,
-     * byte-by-byte and you did return an ETag here, clients tend to get
-     * confused.
-     *
-     * If you don't return an ETag, you can just return null.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     * @param string $cardData
-     *
-     * @return string|null
-     */
-    public function updateCard($addressBookId, $cardUri, $cardData)
-    {
-        $stmt = $this->pdo->prepare('UPDATE '.$this->cardsTableName.' SET carddata = ?, lastmodified = ?, size = ?, etag = ? WHERE uri = ? AND addressbookid =?');
-
-        $etag = md5($cardData);
-        $stmt->execute([
-            $cardData,
-            time(),
-            strlen($cardData),
-            $etag,
-            $cardUri,
-            $addressBookId,
-        ]);
-
-        $this->addChange($addressBookId, $cardUri, 2);
-
-        return '"'.$etag.'"';
-    }
-
-    /**
-     * Deletes a card.
-     *
-     * @param mixed  $addressBookId
-     * @param string $cardUri
-     *
-     * @return bool
-     */
-    public function deleteCard($addressBookId, $cardUri)
-    {
-        $stmt = $this->pdo->prepare('DELETE FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri = ?');
-        $stmt->execute([$addressBookId, $cardUri]);
-
-        $this->addChange($addressBookId, $cardUri, 3);
-
-        return 1 === $stmt->rowCount();
-    }
-
-    /**
-     * The getChanges method returns all the changes that have happened, since
-     * the specified syncToken in the specified address book.
-     *
-     * This function should return an array, such as the following:
-     *
-     * [
-     *   'syncToken' => 'The current synctoken',
-     *   'added'   => [
-     *      'new.txt',
-     *   ],
-     *   'modified'   => [
-     *      'updated.txt',
-     *   ],
-     *   'deleted' => [
-     *      'foo.php.bak',
-     *      'old.txt'
-     *   ]
-     * ];
-     *
-     * The returned syncToken property should reflect the *current* syncToken
-     * of the addressbook, as reported in the {http://sabredav.org/ns}sync-token
-     * property. This is needed here too, to ensure the operation is atomic.
-     *
-     * If the $syncToken argument is specified as null, this is an initial
-     * sync, and all members should be reported.
-     *
-     * The modified property is an array of nodenames that have changed since
-     * the last token.
-     *
-     * The deleted property is an array with nodenames, that have been deleted
-     * from collection.
-     *
-     * The $syncLevel argument is basically the 'depth' of the report. If it's
-     * 1, you only have to report changes that happened only directly in
-     * immediate descendants. If it's 2, it should also include changes from
-     * the nodes below the child collections. (grandchildren)
-     *
-     * The $limit argument allows a client to specify how many results should
-     * be returned at most. If the limit is not specified, it should be treated
-     * as infinite.
-     *
-     * If the limit (infinite or not) is higher than you're willing to return,
-     * you should throw a Sabre\DAV\Exception\TooMuchMatches() exception.
-     *
-     * If the syncToken is expired (due to data cleanup) or unknown, you must
-     * return null.
-     *
-     * The limit is 'suggestive'. You are free to ignore it.
-     *
-     * @param string $addressBookId
-     * @param string $syncToken
-     * @param int    $syncLevel
-     * @param int    $limit
-     *
-     * @return array|null
-     */
-    public function getChangesForAddressBook($addressBookId, $syncToken, $syncLevel, $limit = null)
-    {
-        // Current synctoken
-        $stmt = $this->pdo->prepare('SELECT synctoken FROM '.$this->addressBooksTableName.' WHERE id = ?');
-        $stmt->execute([$addressBookId]);
-        $currentToken = $stmt->fetchColumn(0);
-
-        if (is_null($currentToken)) {
-            return null;
-        }
-
-        $result = [
-            'syncToken' => $currentToken,
-            'added' => [],
-            'modified' => [],
-            'deleted' => [],
-        ];
-
-        if ($syncToken) {
-            $query = 'SELECT uri, operation FROM '.$this->addressBookChangesTableName.' WHERE synctoken >= ? AND synctoken < ? AND addressbookid = ? ORDER BY synctoken';
-            if ($limit > 0) {
-                $query .= ' LIMIT '.(int) $limit;
-            }
-
-            // Fetching all changes
-            $stmt = $this->pdo->prepare($query);
-            $stmt->execute([$syncToken, $currentToken, $addressBookId]);
-
-            $changes = [];
-
-            // This loop ensures that any duplicates are overwritten, only the
-            // last change on a node is relevant.
-            while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
-                $changes[$row['uri']] = $row['operation'];
-            }
-
-            foreach ($changes as $uri => $operation) {
-                switch ($operation) {
-                    case 1:
-                        $result['added'][] = $uri;
-                        break;
-                    case 2:
-                        $result['modified'][] = $uri;
-                        break;
-                    case 3:
-                        $result['deleted'][] = $uri;
-                        break;
-                }
-            }
-        } else {
-            // No synctoken supplied, this is the initial sync.
-            $query = 'SELECT uri FROM '.$this->cardsTableName.' WHERE addressbookid = ?';
-            $stmt = $this->pdo->prepare($query);
-            $stmt->execute([$addressBookId]);
-
-            $result['added'] = $stmt->fetchAll(\PDO::FETCH_COLUMN);
-        }
-
-        return $result;
-    }
-
-    /**
-     * Adds a change record to the addressbookchanges table.
-     *
-     * @param mixed  $addressBookId
-     * @param string $objectUri
-     * @param int    $operation     1 = add, 2 = modify, 3 = delete
-     */
-    protected function addChange($addressBookId, $objectUri, $operation)
-    {
-        $stmt = $this->pdo->prepare('INSERT INTO '.$this->addressBookChangesTableName.' (uri, synctoken, addressbookid, operation) SELECT ?, synctoken, ?, ? FROM '.$this->addressBooksTableName.' WHERE id = ?');
-        $stmt->execute([
-            $objectUri,
-            $addressBookId,
-            $operation,
-            $addressBookId,
-        ]);
-        $stmt = $this->pdo->prepare('UPDATE '.$this->addressBooksTableName.' SET synctoken = synctoken + 1 WHERE id = ?');
-        $stmt->execute([
-            $addressBookId,
-        ]);
-    }
+	/**
+	 * PDO connection.
+	 *
+	 * @var PDO
+	 */
+	protected $pdo;
+
+	/**
+	 * The PDO table name used to store addressbooks.
+	 */
+	public $addressBooksTableName = 'addressbooks';
+
+	/**
+	 * The PDO table name used to store cards.
+	 */
+	public $cardsTableName = 'cards';
+
+	/**
+	 * The table name that will be used for tracking changes in address books.
+	 *
+	 * @var string
+	 */
+	public $addressBookChangesTableName = 'addressbookchanges';
+
+	/**
+	 * Sets up the object.
+	 */
+	public function __construct(\PDO $pdo)
+	{
+		$this->pdo = $pdo;
+	}
+
+	/**
+	 * Returns the list of addressbooks for a specific user.
+	 *
+	 * @param string $principalUri
+	 *
+	 * @return array
+	 */
+	public function getAddressBooksForUser($principalUri)
+	{
+		$stmt = $this->pdo->prepare('SELECT id, uri, displayname, principaluri, description, synctoken FROM '.$this->addressBooksTableName.' WHERE principaluri = ?');
+		$stmt->execute([$principalUri]);
+
+		$addressBooks = [];
+
+		foreach ($stmt->fetchAll() as $row) {
+			$addressBooks[] = [
+				'id' => $row['id'],
+				'uri' => $row['uri'],
+				'principaluri' => $row['principaluri'],
+				'{DAV:}displayname' => $row['displayname'],
+				'{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
+				'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
+				'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ? $row['synctoken'] : '0',
+			];
+		}
+
+		return $addressBooks;
+	}
+
+	/**
+	 * Updates properties for an address book.
+	 *
+	 * The list of mutations is stored in a Sabre\DAV\PropPatch object.
+	 * To do the actual updates, you must tell this object which properties
+	 * you're going to process with the handle() method.
+	 *
+	 * Calling the handle method is like telling the PropPatch object "I
+	 * promise I can handle updating this property".
+	 *
+	 * Read the PropPatch documentation for more info and examples.
+	 *
+	 * @param string $addressBookId
+	 */
+	public function updateAddressBook($addressBookId, PropPatch $propPatch)
+	{
+		$supportedProperties = [
+			'{DAV:}displayname',
+			'{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description',
+		];
+
+		$propPatch->handle($supportedProperties, function ($mutations) use ($addressBookId) {
+			$updates = [];
+			foreach ($mutations as $property => $newValue) {
+				switch ($property) {
+					case '{DAV:}displayname':
+						$updates['displayname'] = $newValue;
+						break;
+					case '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description':
+						$updates['description'] = $newValue;
+						break;
+				}
+			}
+			$query = 'UPDATE '.$this->addressBooksTableName.' SET ';
+			$first = true;
+			foreach ($updates as $key => $value) {
+				if ($first) {
+					$first = false;
+				} else {
+					$query .= ', ';
+				}
+				$query .= ' '.$key.' = :'.$key.' ';
+			}
+			$query .= ' WHERE id = :addressbookid';
+
+			$stmt = $this->pdo->prepare($query);
+			$updates['addressbookid'] = $addressBookId;
+
+			$stmt->execute($updates);
+
+			$this->addChange($addressBookId, '', 2);
+
+			return true;
+		});
+	}
+
+	/**
+	 * Creates a new address book.
+	 *
+	 * @param string $principalUri
+	 * @param string $url          just the 'basename' of the url
+	 *
+	 * @return int Last insert id
+	 */
+	public function createAddressBook($principalUri, $url, array $properties)
+	{
+		$values = [
+			'displayname' => null,
+			'description' => null,
+			'principaluri' => $principalUri,
+			'uri' => $url,
+		];
+
+		foreach ($properties as $property => $newValue) {
+			switch ($property) {
+				case '{DAV:}displayname':
+					$values['displayname'] = $newValue;
+					break;
+				case '{'.CardDAV\Plugin::NS_CARDDAV.'}addressbook-description':
+					$values['description'] = $newValue;
+					break;
+				default:
+					throw new DAV\Exception\BadRequest('Unknown property: '.$property);
+			}
+		}
+
+		$query = 'INSERT INTO '.$this->addressBooksTableName.' (uri, displayname, description, principaluri, synctoken) VALUES (:uri, :displayname, :description, :principaluri, 1)';
+		$stmt = $this->pdo->prepare($query);
+		$stmt->execute($values);
+
+		return $this->pdo->lastInsertId(
+			$this->addressBooksTableName.'_id_seq'
+		);
+	}
+
+	/**
+	 * Deletes an entire addressbook and all its contents.
+	 *
+	 * @param int $addressBookId
+	 */
+	public function deleteAddressBook($addressBookId)
+	{
+		$stmt = $this->pdo->prepare('DELETE FROM '.$this->cardsTableName.' WHERE addressbookid = ?');
+		$stmt->execute([$addressBookId]);
+
+		$stmt = $this->pdo->prepare('DELETE FROM '.$this->addressBooksTableName.' WHERE id = ?');
+		$stmt->execute([$addressBookId]);
+
+		$stmt = $this->pdo->prepare('DELETE FROM '.$this->addressBookChangesTableName.' WHERE addressbookid = ?');
+		$stmt->execute([$addressBookId]);
+	}
+
+	/**
+	 * Returns all cards for a specific addressbook id.
+	 *
+	 * This method should return the following properties for each card:
+	 *   * carddata - raw vcard data
+	 *   * uri - Some unique url
+	 *   * lastmodified - A unix timestamp
+	 *
+	 * It's recommended to also return the following properties:
+	 *   * etag - A unique etag. This must change every time the card changes.
+	 *   * size - The size of the card in bytes.
+	 *
+	 * If these last two properties are provided, less time will be spent
+	 * calculating them. If they are specified, you can also ommit carddata.
+	 * This may speed up certain requests, especially with large cards.
+	 *
+	 * @param mixed $addressbookId
+	 *
+	 * @return array
+	 */
+	public function getCards($addressbookId)
+	{
+		$stmt = $this->pdo->prepare('SELECT id, uri, lastmodified, etag, size FROM '.$this->cardsTableName.' WHERE addressbookid = ?');
+		$stmt->execute([$addressbookId]);
+
+		$result = [];
+		while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
+			$row['etag'] = '"'.$row['etag'].'"';
+			$row['lastmodified'] = (int) $row['lastmodified'];
+			$result[] = $row;
+		}
+
+		return $result;
+	}
+
+	/**
+	 * Returns a specific card.
+	 *
+	 * The same set of properties must be returned as with getCards. The only
+	 * exception is that 'carddata' is absolutely required.
+	 *
+	 * If the card does not exist, you must return false.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 *
+	 * @return array
+	 */
+	public function getCard($addressBookId, $cardUri)
+	{
+		$stmt = $this->pdo->prepare('SELECT id, carddata, uri, lastmodified, etag, size FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri = ? LIMIT 1');
+		$stmt->execute([$addressBookId, $cardUri]);
+
+		$result = $stmt->fetch(\PDO::FETCH_ASSOC);
+
+		if (!$result) {
+			return false;
+		}
+
+		$result['etag'] = '"'.$result['etag'].'"';
+		$result['lastmodified'] = (int) $result['lastmodified'];
+
+		return $result;
+	}
+
+	/**
+	 * Returns a list of cards.
+	 *
+	 * This method should work identical to getCard, but instead return all the
+	 * cards in the list as an array.
+	 *
+	 * If the backend supports this, it may allow for some speed-ups.
+	 *
+	 * @param mixed $addressBookId
+	 *
+	 * @return array
+	 */
+	public function getMultipleCards($addressBookId, array $uris)
+	{
+		$query = 'SELECT id, uri, lastmodified, etag, size, carddata FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri IN (';
+		// Inserting a whole bunch of question marks
+		$query .= implode(',', array_fill(0, count($uris), '?'));
+		$query .= ')';
+
+		$stmt = $this->pdo->prepare($query);
+		$stmt->execute(array_merge([$addressBookId], $uris));
+		$result = [];
+		while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
+			$row['etag'] = '"'.$row['etag'].'"';
+			$row['lastmodified'] = (int) $row['lastmodified'];
+			$result[] = $row;
+		}
+
+		return $result;
+	}
+
+	/**
+	 * Creates a new card.
+	 *
+	 * The addressbook id will be passed as the first argument. This is the
+	 * same id as it is returned from the getAddressBooksForUser method.
+	 *
+	 * The cardUri is a base uri, and doesn't include the full path. The
+	 * cardData argument is the vcard body, and is passed as a string.
+	 *
+	 * It is possible to return an ETag from this method. This ETag is for the
+	 * newly created resource, and must be enclosed with double quotes (that
+	 * is, the string itself must contain the double quotes).
+	 *
+	 * You should only return the ETag if you store the carddata as-is. If a
+	 * subsequent GET request on the same card does not have the same body,
+	 * byte-by-byte and you did return an ETag here, clients tend to get
+	 * confused.
+	 *
+	 * If you don't return an ETag, you can just return null.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 * @param string $cardData
+	 *
+	 * @return string|null
+	 */
+	public function createCard($addressBookId, $cardUri, $cardData)
+	{
+		$stmt = $this->pdo->prepare('INSERT INTO '.$this->cardsTableName.' (carddata, uri, lastmodified, addressbookid, size, etag) VALUES (?, ?, ?, ?, ?, ?)');
+
+		$etag = md5($cardData);
+
+		$stmt->execute([
+			$cardData,
+			$cardUri,
+			time(),
+			$addressBookId,
+			strlen($cardData),
+			$etag,
+		]);
+
+		$this->addChange($addressBookId, $cardUri, 1);
+
+		return '"'.$etag.'"';
+	}
+
+	/**
+	 * Updates a card.
+	 *
+	 * The addressbook id will be passed as the first argument. This is the
+	 * same id as it is returned from the getAddressBooksForUser method.
+	 *
+	 * The cardUri is a base uri, and doesn't include the full path. The
+	 * cardData argument is the vcard body, and is passed as a string.
+	 *
+	 * It is possible to return an ETag from this method. This ETag should
+	 * match that of the updated resource, and must be enclosed with double
+	 * quotes (that is: the string itself must contain the actual quotes).
+	 *
+	 * You should only return the ETag if you store the carddata as-is. If a
+	 * subsequent GET request on the same card does not have the same body,
+	 * byte-by-byte and you did return an ETag here, clients tend to get
+	 * confused.
+	 *
+	 * If you don't return an ETag, you can just return null.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 * @param string $cardData
+	 *
+	 * @return string|null
+	 */
+	public function updateCard($addressBookId, $cardUri, $cardData)
+	{
+		$stmt = $this->pdo->prepare('UPDATE '.$this->cardsTableName.' SET carddata = ?, lastmodified = ?, size = ?, etag = ? WHERE uri = ? AND addressbookid =?');
+
+		$etag = md5($cardData);
+		$stmt->execute([
+			$cardData,
+			time(),
+			strlen($cardData),
+			$etag,
+			$cardUri,
+			$addressBookId,
+		]);
+
+		$this->addChange($addressBookId, $cardUri, 2);
+
+		return '"'.$etag.'"';
+	}
+
+	/**
+	 * Deletes a card.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $cardUri
+	 *
+	 * @return bool
+	 */
+	public function deleteCard($addressBookId, $cardUri)
+	{
+		$stmt = $this->pdo->prepare('DELETE FROM '.$this->cardsTableName.' WHERE addressbookid = ? AND uri = ?');
+		$stmt->execute([$addressBookId, $cardUri]);
+
+		$this->addChange($addressBookId, $cardUri, 3);
+
+		return 1 === $stmt->rowCount();
+	}
+
+	/**
+	 * The getChanges method returns all the changes that have happened, since
+	 * the specified syncToken in the specified address book.
+	 *
+	 * This function should return an array, such as the following:
+	 *
+	 * [
+	 *   'syncToken' => 'The current synctoken',
+	 *   'added'   => [
+	 *      'new.txt',
+	 *   ],
+	 *   'modified'   => [
+	 *      'updated.txt',
+	 *   ],
+	 *   'deleted' => [
+	 *      'foo.php.bak',
+	 *      'old.txt'
+	 *   ]
+	 * ];
+	 *
+	 * The returned syncToken property should reflect the *current* syncToken
+	 * of the addressbook, as reported in the {http://sabredav.org/ns}sync-token
+	 * property. This is needed here too, to ensure the operation is atomic.
+	 *
+	 * If the $syncToken argument is specified as null, this is an initial
+	 * sync, and all members should be reported.
+	 *
+	 * The modified property is an array of nodenames that have changed since
+	 * the last token.
+	 *
+	 * The deleted property is an array with nodenames, that have been deleted
+	 * from collection.
+	 *
+	 * The $syncLevel argument is basically the 'depth' of the report. If it's
+	 * 1, you only have to report changes that happened only directly in
+	 * immediate descendants. If it's 2, it should also include changes from
+	 * the nodes below the child collections. (grandchildren)
+	 *
+	 * The $limit argument allows a client to specify how many results should
+	 * be returned at most. If the limit is not specified, it should be treated
+	 * as infinite.
+	 *
+	 * If the limit (infinite or not) is higher than you're willing to return,
+	 * you should throw a Sabre\DAV\Exception\TooMuchMatches() exception.
+	 *
+	 * If the syncToken is expired (due to data cleanup) or unknown, you must
+	 * return null.
+	 *
+	 * The limit is 'suggestive'. You are free to ignore it.
+	 *
+	 * @param string $addressBookId
+	 * @param string $syncToken
+	 * @param int    $syncLevel
+	 * @param int    $limit
+	 *
+	 * @return array|null
+	 */
+	public function getChangesForAddressBook($addressBookId, $syncToken, $syncLevel, $limit = null)
+	{
+		// Current synctoken
+		$stmt = $this->pdo->prepare('SELECT synctoken FROM '.$this->addressBooksTableName.' WHERE id = ?');
+		$stmt->execute([$addressBookId]);
+		$currentToken = $stmt->fetchColumn(0);
+
+		if (is_null($currentToken)) {
+			return null;
+		}
+
+		$result = [
+			'syncToken' => $currentToken,
+			'added' => [],
+			'modified' => [],
+			'deleted' => [],
+		];
+
+		if ($syncToken) {
+			$query = 'SELECT uri, operation FROM '.$this->addressBookChangesTableName.' WHERE synctoken >= ? AND synctoken < ? AND addressbookid = ? ORDER BY synctoken';
+			if ($limit > 0) {
+				$query .= ' LIMIT '.(int) $limit;
+			}
+
+			// Fetching all changes
+			$stmt = $this->pdo->prepare($query);
+			$stmt->execute([$syncToken, $currentToken, $addressBookId]);
+
+			$changes = [];
+
+			// This loop ensures that any duplicates are overwritten, only the
+			// last change on a node is relevant.
+			while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
+				$changes[$row['uri']] = $row['operation'];
+			}
+
+			foreach ($changes as $uri => $operation) {
+				switch ($operation) {
+					case 1:
+						$result['added'][] = $uri;
+						break;
+					case 2:
+						$result['modified'][] = $uri;
+						break;
+					case 3:
+						$result['deleted'][] = $uri;
+						break;
+				}
+			}
+		} else {
+			// No synctoken supplied, this is the initial sync.
+			$query = 'SELECT uri FROM '.$this->cardsTableName.' WHERE addressbookid = ?';
+			$stmt = $this->pdo->prepare($query);
+			$stmt->execute([$addressBookId]);
+
+			$result['added'] = $stmt->fetchAll(\PDO::FETCH_COLUMN);
+		}
+
+		return $result;
+	}
+
+	/**
+	 * Adds a change record to the addressbookchanges table.
+	 *
+	 * @param mixed  $addressBookId
+	 * @param string $objectUri
+	 * @param int    $operation     1 = add, 2 = modify, 3 = delete
+	 */
+	protected function addChange($addressBookId, $objectUri, $operation)
+	{
+		$stmt = $this->pdo->prepare('INSERT INTO '.$this->addressBookChangesTableName.' (uri, synctoken, addressbookid, operation) SELECT ?, synctoken, ?, ? FROM '.$this->addressBooksTableName.' WHERE id = ?');
+		$stmt->execute([
+			$objectUri,
+			$addressBookId,
+			$operation,
+			$addressBookId,
+		]);
+		$stmt = $this->pdo->prepare('UPDATE '.$this->addressBooksTableName.' SET synctoken = synctoken + 1 WHERE id = ?');
+		$stmt->execute([
+			$addressBookId,
+		]);
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/AddressBookRoot.php

Indentation +51 added lines, -51 removed lines

@@ -17,59 +17,59 @@
  */
 class AddressBookRoot extends DAVACL\AbstractPrincipalCollection
 {
-    /**
-     * Principal Backend.
-     *
-     * @var DAVACL\PrincipalBackend\BackendInterface
-     */
-    protected $principalBackend;
+	/**
+	 * Principal Backend.
+	 *
+	 * @var DAVACL\PrincipalBackend\BackendInterface
+	 */
+	protected $principalBackend;
 
-    /**
-     * CardDAV backend.
-     *
-     * @var Backend\BackendInterface
-     */
-    protected $carddavBackend;
+	/**
+	 * CardDAV backend.
+	 *
+	 * @var Backend\BackendInterface
+	 */
+	protected $carddavBackend;
 
-    /**
-     * Constructor.
-     *
-     * This constructor needs both a principal and a carddav backend.
-     *
-     * By default this class will show a list of addressbook collections for
-     * principals in the 'principals' collection. If your main principals are
-     * actually located in a different path, use the $principalPrefix argument
-     * to override this.
-     *
-     * @param string $principalPrefix
-     */
-    public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, Backend\BackendInterface $carddavBackend, $principalPrefix = 'principals')
-    {
-        $this->carddavBackend = $carddavBackend;
-        parent::__construct($principalBackend, $principalPrefix);
-    }
+	/**
+	 * Constructor.
+	 *
+	 * This constructor needs both a principal and a carddav backend.
+	 *
+	 * By default this class will show a list of addressbook collections for
+	 * principals in the 'principals' collection. If your main principals are
+	 * actually located in a different path, use the $principalPrefix argument
+	 * to override this.
+	 *
+	 * @param string $principalPrefix
+	 */
+	public function __construct(DAVACL\PrincipalBackend\BackendInterface $principalBackend, Backend\BackendInterface $carddavBackend, $principalPrefix = 'principals')
+	{
+		$this->carddavBackend = $carddavBackend;
+		parent::__construct($principalBackend, $principalPrefix);
+	}
 
-    /**
-     * Returns the name of the node.
-     *
-     * @return string
-     */
-    public function getName()
-    {
-        return Plugin::ADDRESSBOOK_ROOT;
-    }
+	/**
+	 * Returns the name of the node.
+	 *
+	 * @return string
+	 */
+	public function getName()
+	{
+		return Plugin::ADDRESSBOOK_ROOT;
+	}
 
-    /**
-     * This method returns a node for a principal.
-     *
-     * The passed array contains principal information, and is guaranteed to
-     * at least contain a uri item. Other properties may or may not be
-     * supplied by the authentication backend.
-     *
-     * @return \Sabre\DAV\INode
-     */
-    public function getChildForPrincipal(array $principal)
-    {
-        return new AddressBookHome($this->carddavBackend, $principal['uri']);
-    }
+	/**
+	 * This method returns a node for a principal.
+	 *
+	 * The passed array contains principal information, and is guaranteed to
+	 * at least contain a uri item. Other properties may or may not be
+	 * supplied by the authentication backend.
+	 *
+	 * @return \Sabre\DAV\INode
+	 */
+	public function getChildForPrincipal(array $principal)
+	{
+		return new AddressBookHome($this->carddavBackend, $principal['uri']);
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/VCFExportPlugin.php

Indentation +139 added lines, -139 removed lines

@@ -23,143 +23,143 @@
  */
 class VCFExportPlugin extends DAV\ServerPlugin
 {
-    /**
-     * Reference to Server class.
-     *
-     * @var DAV\Server
-     */
-    protected $server;
-
-    /**
-     * Initializes the plugin and registers event handlers.
-     */
-    public function initialize(DAV\Server $server)
-    {
-        $this->server = $server;
-        $this->server->on('method:GET', [$this, 'httpGet'], 90);
-        $server->on('browserButtonActions', function ($path, $node, &$actions) {
-            if ($node instanceof IAddressBook) {
-                $actions .= '<a href="'.htmlspecialchars($path, ENT_QUOTES, 'UTF-8').'?export"><span class="oi" data-glyph="book"></span></a>';
-            }
-        });
-    }
-
-    /**
-     * Intercepts GET requests on addressbook urls ending with ?export.
-     *
-     * @return bool
-     */
-    public function httpGet(RequestInterface $request, ResponseInterface $response)
-    {
-        $queryParams = $request->getQueryParameters();
-        if (!array_key_exists('export', $queryParams)) {
-            return;
-        }
-
-        $path = $request->getPath();
-
-        $node = $this->server->tree->getNodeForPath($path);
-
-        if (!($node instanceof IAddressBook)) {
-            return;
-        }
-
-        $this->server->transactionType = 'get-addressbook-export';
-
-        // Checking ACL, if available.
-        if ($aclPlugin = $this->server->getPlugin('acl')) {
-            $aclPlugin->checkPrivileges($path, '{DAV:}read');
-        }
-
-        $nodes = $this->server->getPropertiesForPath($path, [
-            '{'.Plugin::NS_CARDDAV.'}address-data',
-        ], 1);
-
-        $format = 'text/directory';
-
-        $output = null;
-        $filenameExtension = null;
-
-        switch ($format) {
-            case 'text/directory':
-                $output = $this->generateVCF($nodes);
-                $filenameExtension = '.vcf';
-                break;
-        }
-
-        $filename = preg_replace(
-            '/[^a-zA-Z0-9-_ ]/um',
-            '',
-            $node->getName()
-        );
-        $filename .= '-'.date('Y-m-d').$filenameExtension;
-
-        $response->setHeader('Content-Disposition', 'attachment; filename="'.$filename.'"');
-        $response->setHeader('Content-Type', $format);
-
-        $response->setStatus(200);
-        $response->setBody($output);
-
-        // Returning false to break the event chain
-        return false;
-    }
-
-    /**
-     * Merges all vcard objects, and builds one big vcf export.
-     *
-     * @return string
-     */
-    public function generateVCF(array $nodes)
-    {
-        $output = '';
-
-        foreach ($nodes as $node) {
-            if (!isset($node[200]['{'.Plugin::NS_CARDDAV.'}address-data'])) {
-                continue;
-            }
-            $nodeData = $node[200]['{'.Plugin::NS_CARDDAV.'}address-data'];
-
-            // Parsing this node so VObject can clean up the output.
-            $vcard = VObject\Reader::read($nodeData);
-            $output .= $vcard->serialize();
-
-            // Destroy circular references to PHP will GC the object.
-            $vcard->destroy();
-        }
-
-        return $output;
-    }
-
-    /**
-     * Returns a plugin name.
-     *
-     * Using this name other plugins will be able to access other plugins
-     * using \Sabre\DAV\Server::getPlugin
-     *
-     * @return string
-     */
-    public function getPluginName()
-    {
-        return 'vcf-export';
-    }
-
-    /**
-     * Returns a bunch of meta-data about the plugin.
-     *
-     * Providing this information is optional, and is mainly displayed by the
-     * Browser plugin.
-     *
-     * The description key in the returned array may contain html and will not
-     * be sanitized.
-     *
-     * @return array
-     */
-    public function getPluginInfo()
-    {
-        return [
-            'name' => $this->getPluginName(),
-            'description' => 'Adds the ability to export CardDAV addressbooks as a single vCard file.',
-            'link' => 'http://sabre.io/dav/vcf-export-plugin/',
-        ];
-    }
+	/**
+	 * Reference to Server class.
+	 *
+	 * @var DAV\Server
+	 */
+	protected $server;
+
+	/**
+	 * Initializes the plugin and registers event handlers.
+	 */
+	public function initialize(DAV\Server $server)
+	{
+		$this->server = $server;
+		$this->server->on('method:GET', [$this, 'httpGet'], 90);
+		$server->on('browserButtonActions', function ($path, $node, &$actions) {
+			if ($node instanceof IAddressBook) {
+				$actions .= '<a href="'.htmlspecialchars($path, ENT_QUOTES, 'UTF-8').'?export"><span class="oi" data-glyph="book"></span></a>';
+			}
+		});
+	}
+
+	/**
+	 * Intercepts GET requests on addressbook urls ending with ?export.
+	 *
+	 * @return bool
+	 */
+	public function httpGet(RequestInterface $request, ResponseInterface $response)
+	{
+		$queryParams = $request->getQueryParameters();
+		if (!array_key_exists('export', $queryParams)) {
+			return;
+		}
+
+		$path = $request->getPath();
+
+		$node = $this->server->tree->getNodeForPath($path);
+
+		if (!($node instanceof IAddressBook)) {
+			return;
+		}
+
+		$this->server->transactionType = 'get-addressbook-export';
+
+		// Checking ACL, if available.
+		if ($aclPlugin = $this->server->getPlugin('acl')) {
+			$aclPlugin->checkPrivileges($path, '{DAV:}read');
+		}
+
+		$nodes = $this->server->getPropertiesForPath($path, [
+			'{'.Plugin::NS_CARDDAV.'}address-data',
+		], 1);
+
+		$format = 'text/directory';
+
+		$output = null;
+		$filenameExtension = null;
+
+		switch ($format) {
+			case 'text/directory':
+				$output = $this->generateVCF($nodes);
+				$filenameExtension = '.vcf';
+				break;
+		}
+
+		$filename = preg_replace(
+			'/[^a-zA-Z0-9-_ ]/um',
+			'',
+			$node->getName()
+		);
+		$filename .= '-'.date('Y-m-d').$filenameExtension;
+
+		$response->setHeader('Content-Disposition', 'attachment; filename="'.$filename.'"');
+		$response->setHeader('Content-Type', $format);
+
+		$response->setStatus(200);
+		$response->setBody($output);
+
+		// Returning false to break the event chain
+		return false;
+	}
+
+	/**
+	 * Merges all vcard objects, and builds one big vcf export.
+	 *
+	 * @return string
+	 */
+	public function generateVCF(array $nodes)
+	{
+		$output = '';
+
+		foreach ($nodes as $node) {
+			if (!isset($node[200]['{'.Plugin::NS_CARDDAV.'}address-data'])) {
+				continue;
+			}
+			$nodeData = $node[200]['{'.Plugin::NS_CARDDAV.'}address-data'];
+
+			// Parsing this node so VObject can clean up the output.
+			$vcard = VObject\Reader::read($nodeData);
+			$output .= $vcard->serialize();
+
+			// Destroy circular references to PHP will GC the object.
+			$vcard->destroy();
+		}
+
+		return $output;
+	}
+
+	/**
+	 * Returns a plugin name.
+	 *
+	 * Using this name other plugins will be able to access other plugins
+	 * using \Sabre\DAV\Server::getPlugin
+	 *
+	 * @return string
+	 */
+	public function getPluginName()
+	{
+		return 'vcf-export';
+	}
+
+	/**
+	 * Returns a bunch of meta-data about the plugin.
+	 *
+	 * Providing this information is optional, and is mainly displayed by the
+	 * Browser plugin.
+	 *
+	 * The description key in the returned array may contain html and will not
+	 * be sanitized.
+	 *
+	 * @return array
+	 */
+	public function getPluginInfo()
+	{
+		return [
+			'name' => $this->getPluginName(),
+			'description' => 'Adds the ability to export CardDAV addressbooks as a single vCard file.',
+			'link' => 'http://sabre.io/dav/vcf-export-plugin/',
+		];
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         $this->server = $server;
         $this->server->on('method:GET', [$this, 'httpGet'], 90);
-        $server->on('browserButtonActions', function ($path, $node, &$actions) {
+        $server->on('browserButtonActions', function($path, $node, &$actions) {
             if ($node instanceof IAddressBook) {
                 $actions .= '<a href="'.htmlspecialchars($path, ENT_QUOTES, 'UTF-8').'?export"><span class="oi" data-glyph="book"></span></a>';
             }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Request/AddressBookQueryReport.php

Indentation +161 added lines, -161 removed lines

@@ -23,171 +23,171 @@
  */
 class AddressBookQueryReport implements XmlDeserializable
 {
-    /**
-     * An array with requested properties.
-     *
-     * @var array
-     */
-    public $properties;
-
-    /**
-     * An array with requested vcard properties.
-     *
-     * @var array
-     */
-    public $addressDataProperties = [];
-
-    /**
-     * List of property/component filters.
-     *
-     * This is an array with filters. Every item is a property filter. Every
-     * property filter has the following keys:
-     *   * name - name of the component to filter on
-     *   * test - anyof or allof
-     *   * is-not-defined - Test for non-existence
-     *   * param-filters - A list of parameter filters on the property
-     *   * text-matches - A list of text values the filter needs to match
-     *
-     * Each param-filter has the following keys:
-     *   * name - name of the parameter
-     *   * is-not-defined - Test for non-existence
-     *   * text-match - Match the parameter value
-     *
-     * Each text-match in property filters, and the single text-match in
-     * param-filters have the following keys:
-     *
-     *   * value - value to match
-     *   * match-type - contains, starts-with, ends-with, equals
-     *   * negate-condition - Do the opposite match
-     *   * collation - Usually i;unicode-casemap
-     *
-     * @var array
-     */
-    public $filters;
-
-    /**
-     * The number of results the client wants.
-     *
-     * null means it wasn't specified, which in most cases means 'all results'.
-     *
-     * @var int|null
-     */
-    public $limit;
-
-    /**
-     * Either 'anyof' or 'allof'.
-     *
-     * @var string
-     */
-    public $test;
-
-    /**
-     * The mimetype of the content that should be returend. Usually
-     * text/vcard.
-     *
-     * @var string
-     */
-    public $contentType = null;
-
-    /**
-     * The version of vcard data that should be returned. Usually 3.0,
-     * referring to vCard 3.0.
-     *
-     * @var string
-     */
-    public $version = null;
-
-    /**
-     * The deserialize method is called during xml parsing.
-     *
-     * This method is called statically, this is because in theory this method
-     * may be used as a type of constructor, or factory method.
-     *
-     * Often you want to return an instance of the current class, but you are
-     * free to return other data as well.
-     *
-     * You are responsible for advancing the reader to the next element. Not
-     * doing anything will result in a never-ending loop.
-     *
-     * If you just want to skip parsing for this element altogether, you can
-     * just call $reader->next();
-     *
-     * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
-     * the next element.
-     *
-     * @return mixed
-     */
-    public static function xmlDeserialize(Reader $reader)
-    {
-        $elems = (array) $reader->parseInnerTree([
-            '{urn:ietf:params:xml:ns:carddav}prop-filter' => 'Sabre\\CardDAV\\Xml\\Filter\\PropFilter',
-            '{urn:ietf:params:xml:ns:carddav}param-filter' => 'Sabre\\CardDAV\\Xml\\Filter\\ParamFilter',
-            '{urn:ietf:params:xml:ns:carddav}address-data' => 'Sabre\\CardDAV\\Xml\\Filter\\AddressData',
-            '{DAV:}prop' => 'Sabre\\Xml\\Element\\KeyValue',
-        ]);
-
-        $newProps = [
-            'filters' => null,
-            'properties' => [],
-            'test' => 'anyof',
-            'limit' => null,
-        ];
-
-        if (!is_array($elems)) {
-            $elems = [];
-        }
-
-        foreach ($elems as $elem) {
-            switch ($elem['name']) {
-                case '{DAV:}prop':
-                    $newProps['properties'] = array_keys($elem['value']);
-                    if (isset($elem['value']['{'.Plugin::NS_CARDDAV.'}address-data'])) {
-                        $newProps += $elem['value']['{'.Plugin::NS_CARDDAV.'}address-data'];
-                    }
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}filter':
-                    if (!is_null($newProps['filters'])) {
-                        throw new BadRequest('You can only include 1 {'.Plugin::NS_CARDDAV.'}filter element');
-                    }
-                    if (isset($elem['attributes']['test'])) {
-                        $newProps['test'] = $elem['attributes']['test'];
-                        if ('allof' !== $newProps['test'] && 'anyof' !== $newProps['test']) {
-                            throw new BadRequest('The "test" attribute must be one of "allof" or "anyof"');
-                        }
-                    }
-
-                    $newProps['filters'] = [];
-                    foreach ((array) $elem['value'] as $subElem) {
-                        if ($subElem['name'] === '{'.Plugin::NS_CARDDAV.'}prop-filter') {
-                            $newProps['filters'][] = $subElem['value'];
-                        }
-                    }
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}limit':
-                    foreach ($elem['value'] as $child) {
-                        if ($child['name'] === '{'.Plugin::NS_CARDDAV.'}nresults') {
-                            $newProps['limit'] = (int) $child['value'];
-                        }
-                    }
-                    break;
-            }
-        }
-
-        if (is_null($newProps['filters'])) {
-            /*
+	/**
+	 * An array with requested properties.
+	 *
+	 * @var array
+	 */
+	public $properties;
+
+	/**
+	 * An array with requested vcard properties.
+	 *
+	 * @var array
+	 */
+	public $addressDataProperties = [];
+
+	/**
+	 * List of property/component filters.
+	 *
+	 * This is an array with filters. Every item is a property filter. Every
+	 * property filter has the following keys:
+	 *   * name - name of the component to filter on
+	 *   * test - anyof or allof
+	 *   * is-not-defined - Test for non-existence
+	 *   * param-filters - A list of parameter filters on the property
+	 *   * text-matches - A list of text values the filter needs to match
+	 *
+	 * Each param-filter has the following keys:
+	 *   * name - name of the parameter
+	 *   * is-not-defined - Test for non-existence
+	 *   * text-match - Match the parameter value
+	 *
+	 * Each text-match in property filters, and the single text-match in
+	 * param-filters have the following keys:
+	 *
+	 *   * value - value to match
+	 *   * match-type - contains, starts-with, ends-with, equals
+	 *   * negate-condition - Do the opposite match
+	 *   * collation - Usually i;unicode-casemap
+	 *
+	 * @var array
+	 */
+	public $filters;
+
+	/**
+	 * The number of results the client wants.
+	 *
+	 * null means it wasn't specified, which in most cases means 'all results'.
+	 *
+	 * @var int|null
+	 */
+	public $limit;
+
+	/**
+	 * Either 'anyof' or 'allof'.
+	 *
+	 * @var string
+	 */
+	public $test;
+
+	/**
+	 * The mimetype of the content that should be returend. Usually
+	 * text/vcard.
+	 *
+	 * @var string
+	 */
+	public $contentType = null;
+
+	/**
+	 * The version of vcard data that should be returned. Usually 3.0,
+	 * referring to vCard 3.0.
+	 *
+	 * @var string
+	 */
+	public $version = null;
+
+	/**
+	 * The deserialize method is called during xml parsing.
+	 *
+	 * This method is called statically, this is because in theory this method
+	 * may be used as a type of constructor, or factory method.
+	 *
+	 * Often you want to return an instance of the current class, but you are
+	 * free to return other data as well.
+	 *
+	 * You are responsible for advancing the reader to the next element. Not
+	 * doing anything will result in a never-ending loop.
+	 *
+	 * If you just want to skip parsing for this element altogether, you can
+	 * just call $reader->next();
+	 *
+	 * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
+	 * the next element.
+	 *
+	 * @return mixed
+	 */
+	public static function xmlDeserialize(Reader $reader)
+	{
+		$elems = (array) $reader->parseInnerTree([
+			'{urn:ietf:params:xml:ns:carddav}prop-filter' => 'Sabre\\CardDAV\\Xml\\Filter\\PropFilter',
+			'{urn:ietf:params:xml:ns:carddav}param-filter' => 'Sabre\\CardDAV\\Xml\\Filter\\ParamFilter',
+			'{urn:ietf:params:xml:ns:carddav}address-data' => 'Sabre\\CardDAV\\Xml\\Filter\\AddressData',
+			'{DAV:}prop' => 'Sabre\\Xml\\Element\\KeyValue',
+		]);
+
+		$newProps = [
+			'filters' => null,
+			'properties' => [],
+			'test' => 'anyof',
+			'limit' => null,
+		];
+
+		if (!is_array($elems)) {
+			$elems = [];
+		}
+
+		foreach ($elems as $elem) {
+			switch ($elem['name']) {
+				case '{DAV:}prop':
+					$newProps['properties'] = array_keys($elem['value']);
+					if (isset($elem['value']['{'.Plugin::NS_CARDDAV.'}address-data'])) {
+						$newProps += $elem['value']['{'.Plugin::NS_CARDDAV.'}address-data'];
+					}
+					break;
+				case '{'.Plugin::NS_CARDDAV.'}filter':
+					if (!is_null($newProps['filters'])) {
+						throw new BadRequest('You can only include 1 {'.Plugin::NS_CARDDAV.'}filter element');
+					}
+					if (isset($elem['attributes']['test'])) {
+						$newProps['test'] = $elem['attributes']['test'];
+						if ('allof' !== $newProps['test'] && 'anyof' !== $newProps['test']) {
+							throw new BadRequest('The "test" attribute must be one of "allof" or "anyof"');
+						}
+					}
+
+					$newProps['filters'] = [];
+					foreach ((array) $elem['value'] as $subElem) {
+						if ($subElem['name'] === '{'.Plugin::NS_CARDDAV.'}prop-filter') {
+							$newProps['filters'][] = $subElem['value'];
+						}
+					}
+					break;
+				case '{'.Plugin::NS_CARDDAV.'}limit':
+					foreach ($elem['value'] as $child) {
+						if ($child['name'] === '{'.Plugin::NS_CARDDAV.'}nresults') {
+							$newProps['limit'] = (int) $child['value'];
+						}
+					}
+					break;
+			}
+		}
+
+		if (is_null($newProps['filters'])) {
+			/*
              * We are supposed to throw this error, but KDE sometimes does not
              * include the filter element, and we need to treat it as if no
              * filters are supplied
              */
-            //throw new BadRequest('The {' . Plugin::NS_CARDDAV . '}filter element is required for this request');
-            $newProps['filters'] = [];
-        }
+			//throw new BadRequest('The {' . Plugin::NS_CARDDAV . '}filter element is required for this request');
+			$newProps['filters'] = [];
+		}
 
-        $obj = new self();
-        foreach ($newProps as $key => $value) {
-            $obj->$key = $value;
-        }
+		$obj = new self();
+		foreach ($newProps as $key => $value) {
+			$obj->$key = $value;
+		}
 
-        return $obj;
-    }
+		return $obj;
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Filter/ParamFilter.php

Indentation +53 added lines, -53 removed lines

@@ -25,62 +25,62 @@
  */
 abstract class ParamFilter implements Element
 {
-    /**
-     * The deserialize method is called during xml parsing.
-     *
-     * This method is called statically, this is because in theory this method
-     * may be used as a type of constructor, or factory method.
-     *
-     * Often you want to return an instance of the current class, but you are
-     * free to return other data as well.
-     *
-     * You are responsible for advancing the reader to the next element. Not
-     * doing anything will result in a never-ending loop.
-     *
-     * If you just want to skip parsing for this element altogether, you can
-     * just call $reader->next();
-     *
-     * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
-     * the next element.
-     *
-     * @return mixed
-     */
-    public static function xmlDeserialize(Reader $reader)
-    {
-        $result = [
-            'name' => null,
-            'is-not-defined' => false,
-            'text-match' => null,
-        ];
+	/**
+	 * The deserialize method is called during xml parsing.
+	 *
+	 * This method is called statically, this is because in theory this method
+	 * may be used as a type of constructor, or factory method.
+	 *
+	 * Often you want to return an instance of the current class, but you are
+	 * free to return other data as well.
+	 *
+	 * You are responsible for advancing the reader to the next element. Not
+	 * doing anything will result in a never-ending loop.
+	 *
+	 * If you just want to skip parsing for this element altogether, you can
+	 * just call $reader->next();
+	 *
+	 * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
+	 * the next element.
+	 *
+	 * @return mixed
+	 */
+	public static function xmlDeserialize(Reader $reader)
+	{
+		$result = [
+			'name' => null,
+			'is-not-defined' => false,
+			'text-match' => null,
+		];
 
-        $att = $reader->parseAttributes();
-        $result['name'] = $att['name'];
+		$att = $reader->parseAttributes();
+		$result['name'] = $att['name'];
 
-        $elems = $reader->parseInnerTree();
+		$elems = $reader->parseInnerTree();
 
-        if (is_array($elems)) {
-            foreach ($elems as $elem) {
-                switch ($elem['name']) {
-                case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
-                    $result['is-not-defined'] = true;
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}text-match':
-                    $matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
+		if (is_array($elems)) {
+			foreach ($elems as $elem) {
+				switch ($elem['name']) {
+				case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
+					$result['is-not-defined'] = true;
+					break;
+				case '{'.Plugin::NS_CARDDAV.'}text-match':
+					$matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
 
-                    if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
-                        throw new BadRequest('Unknown match-type: '.$matchType);
-                    }
-                    $result['text-match'] = [
-                        'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
-                        'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
-                        'value' => $elem['value'],
-                        'match-type' => $matchType,
-                    ];
-                    break;
-            }
-            }
-        }
+					if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
+						throw new BadRequest('Unknown match-type: '.$matchType);
+					}
+					$result['text-match'] = [
+						'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
+						'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
+						'value' => $elem['value'],
+						'match-type' => $matchType,
+					];
+					break;
+			}
+			}
+		}
 
-        return $result;
-    }
+		return $result;
+	}
 }

Switch Indentation +15 added lines, -15 removed lines

@@ -61,22 +61,22 @@
         if (is_array($elems)) {
             foreach ($elems as $elem) {
                 switch ($elem['name']) {
-                case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
-                    $result['is-not-defined'] = true;
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}text-match':
-                    $matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
+                	case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
+                    	$result['is-not-defined'] = true;
+                    	break;
+                	case '{'.Plugin::NS_CARDDAV.'}text-match':
+                    	$matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
 
-                    if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
-                        throw new BadRequest('Unknown match-type: '.$matchType);
-                    }
-                    $result['text-match'] = [
-                        'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
-                        'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
-                        'value' => $elem['value'],
-                        'match-type' => $matchType,
-                    ];
-                    break;
+                    	if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
+                        	throw new BadRequest('Unknown match-type: '.$matchType);
+                    	}
+                    	$result['text-match'] = [
+                        	'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
+                        	'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
+                        	'value' => $elem['value'],
+                        	'match-type' => $matchType,
+                    	];
+                    	break;
             }
             }
         }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Filter/PropFilter.php

Indentation +61 added lines, -61 removed lines

@@ -25,71 +25,71 @@
  */
 class PropFilter implements XmlDeserializable
 {
-    /**
-     * The deserialize method is called during xml parsing.
-     *
-     * This method is called statically, this is because in theory this method
-     * may be used as a type of constructor, or factory method.
-     *
-     * Often you want to return an instance of the current class, but you are
-     * free to return other data as well.
-     *
-     * You are responsible for advancing the reader to the next element. Not
-     * doing anything will result in a never-ending loop.
-     *
-     * If you just want to skip parsing for this element altogether, you can
-     * just call $reader->next();
-     *
-     * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
-     * the next element.
-     *
-     * @return mixed
-     */
-    public static function xmlDeserialize(Reader $reader)
-    {
-        $result = [
-            'name' => null,
-            'test' => 'anyof',
-            'is-not-defined' => false,
-            'param-filters' => [],
-            'text-matches' => [],
-        ];
+	/**
+	 * The deserialize method is called during xml parsing.
+	 *
+	 * This method is called statically, this is because in theory this method
+	 * may be used as a type of constructor, or factory method.
+	 *
+	 * Often you want to return an instance of the current class, but you are
+	 * free to return other data as well.
+	 *
+	 * You are responsible for advancing the reader to the next element. Not
+	 * doing anything will result in a never-ending loop.
+	 *
+	 * If you just want to skip parsing for this element altogether, you can
+	 * just call $reader->next();
+	 *
+	 * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
+	 * the next element.
+	 *
+	 * @return mixed
+	 */
+	public static function xmlDeserialize(Reader $reader)
+	{
+		$result = [
+			'name' => null,
+			'test' => 'anyof',
+			'is-not-defined' => false,
+			'param-filters' => [],
+			'text-matches' => [],
+		];
 
-        $att = $reader->parseAttributes();
-        $result['name'] = $att['name'];
+		$att = $reader->parseAttributes();
+		$result['name'] = $att['name'];
 
-        if (isset($att['test']) && 'allof' === $att['test']) {
-            $result['test'] = 'allof';
-        }
+		if (isset($att['test']) && 'allof' === $att['test']) {
+			$result['test'] = 'allof';
+		}
 
-        $elems = $reader->parseInnerTree();
+		$elems = $reader->parseInnerTree();
 
-        if (is_array($elems)) {
-            foreach ($elems as $elem) {
-                switch ($elem['name']) {
-                case '{'.Plugin::NS_CARDDAV.'}param-filter':
-                    $result['param-filters'][] = $elem['value'];
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
-                    $result['is-not-defined'] = true;
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}text-match':
-                    $matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
+		if (is_array($elems)) {
+			foreach ($elems as $elem) {
+				switch ($elem['name']) {
+				case '{'.Plugin::NS_CARDDAV.'}param-filter':
+					$result['param-filters'][] = $elem['value'];
+					break;
+				case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
+					$result['is-not-defined'] = true;
+					break;
+				case '{'.Plugin::NS_CARDDAV.'}text-match':
+					$matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
 
-                    if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
-                        throw new BadRequest('Unknown match-type: '.$matchType);
-                    }
-                    $result['text-matches'][] = [
-                        'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
-                        'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
-                        'value' => $elem['value'],
-                        'match-type' => $matchType,
-                    ];
-                    break;
-            }
-            }
-        }
+					if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
+						throw new BadRequest('Unknown match-type: '.$matchType);
+					}
+					$result['text-matches'][] = [
+						'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
+						'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
+						'value' => $elem['value'],
+						'match-type' => $matchType,
+					];
+					break;
+			}
+			}
+		}
 
-        return $result;
-    }
+		return $result;
+	}
 }

Switch Indentation +18 added lines, -18 removed lines

@@ -67,25 +67,25 @@
         if (is_array($elems)) {
             foreach ($elems as $elem) {
                 switch ($elem['name']) {
-                case '{'.Plugin::NS_CARDDAV.'}param-filter':
-                    $result['param-filters'][] = $elem['value'];
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
-                    $result['is-not-defined'] = true;
-                    break;
-                case '{'.Plugin::NS_CARDDAV.'}text-match':
-                    $matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
+                	case '{'.Plugin::NS_CARDDAV.'}param-filter':
+                    	$result['param-filters'][] = $elem['value'];
+                    	break;
+                	case '{'.Plugin::NS_CARDDAV.'}is-not-defined':
+                    	$result['is-not-defined'] = true;
+                    	break;
+                	case '{'.Plugin::NS_CARDDAV.'}text-match':
+                    	$matchType = isset($elem['attributes']['match-type']) ? $elem['attributes']['match-type'] : 'contains';
 
-                    if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
-                        throw new BadRequest('Unknown match-type: '.$matchType);
-                    }
-                    $result['text-matches'][] = [
-                        'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
-                        'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
-                        'value' => $elem['value'],
-                        'match-type' => $matchType,
-                    ];
-                    break;
+                    	if (!in_array($matchType, ['contains', 'equals', 'starts-with', 'ends-with'])) {
+                        	throw new BadRequest('Unknown match-type: '.$matchType);
+                    	}
+                    	$result['text-matches'][] = [
+                        	'negate-condition' => isset($elem['attributes']['negate-condition']) && 'yes' === $elem['attributes']['negate-condition'],
+                        	'collation' => isset($elem['attributes']['collation']) ? $elem['attributes']['collation'] : 'i;unicode-casemap',
+                        	'value' => $elem['value'],
+                        	'match-type' => $matchType,
+                    	];
+                    	break;
             }
             }
         }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Filter/AddressData.php

Indentation +36 added lines, -36 removed lines

@@ -25,42 +25,42 @@
  */
 class AddressData implements XmlDeserializable
 {
-    /**
-     * The deserialize method is called during xml parsing.
-     *
-     * This method is called statically, this is because in theory this method
-     * may be used as a type of constructor, or factory method.
-     *
-     * Often you want to return an instance of the current class, but you are
-     * free to return other data as well.
-     *
-     * You are responsible for advancing the reader to the next element. Not
-     * doing anything will result in a never-ending loop.
-     *
-     * If you just want to skip parsing for this element altogether, you can
-     * just call $reader->next();
-     *
-     * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
-     * the next element.
-     *
-     * @return mixed
-     */
-    public static function xmlDeserialize(Reader $reader)
-    {
-        $result = [
-            'contentType' => $reader->getAttribute('content-type') ?: 'text/vcard',
-            'version' => $reader->getAttribute('version') ?: '3.0',
-        ];
+	/**
+	 * The deserialize method is called during xml parsing.
+	 *
+	 * This method is called statically, this is because in theory this method
+	 * may be used as a type of constructor, or factory method.
+	 *
+	 * Often you want to return an instance of the current class, but you are
+	 * free to return other data as well.
+	 *
+	 * You are responsible for advancing the reader to the next element. Not
+	 * doing anything will result in a never-ending loop.
+	 *
+	 * If you just want to skip parsing for this element altogether, you can
+	 * just call $reader->next();
+	 *
+	 * $reader->parseInnerTree() will parse the entire sub-tree, and advance to
+	 * the next element.
+	 *
+	 * @return mixed
+	 */
+	public static function xmlDeserialize(Reader $reader)
+	{
+		$result = [
+			'contentType' => $reader->getAttribute('content-type') ?: 'text/vcard',
+			'version' => $reader->getAttribute('version') ?: '3.0',
+		];
 
-        $elems = (array) $reader->parseInnerTree();
-        $elems = array_filter($elems, function ($element) {
-            return '{urn:ietf:params:xml:ns:carddav}prop' === $element['name'] &&
-                isset($element['attributes']['name']);
-        });
-        $result['addressDataProperties'] = array_map(function ($element) {
-            return $element['attributes']['name'];
-        }, $elems);
+		$elems = (array) $reader->parseInnerTree();
+		$elems = array_filter($elems, function ($element) {
+			return '{urn:ietf:params:xml:ns:carddav}prop' === $element['name'] &&
+				isset($element['attributes']['name']);
+		});
+		$result['addressDataProperties'] = array_map(function ($element) {
+			return $element['attributes']['name'];
+		}, $elems);
 
-        return $result;
-    }
+		return $result;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -53,11 +53,11 @@
         ];
 
         $elems = (array) $reader->parseInnerTree();
-        $elems = array_filter($elems, function ($element) {
+        $elems = array_filter($elems, function($element) {
             return '{urn:ietf:params:xml:ns:carddav}prop' === $element['name'] &&
                 isset($element['attributes']['name']);
         });
-        $result['addressDataProperties'] = array_map(function ($element) {
+        $result['addressDataProperties'] = array_map(function($element) {
             return $element['attributes']['name'];
         }, $elems);