Dolibarr / dolibarr

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);
 

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Property/SupportedAddressData.php

Indentation +47 added lines, -47 removed lines

@@ -24,54 +24,54 @@
  */
 class SupportedAddressData implements XmlSerializable
 {
-    /**
-     * supported versions.
-     *
-     * @var array
-     */
-    protected $supportedData = [];
+	/**
+	 * supported versions.
+	 *
+	 * @var array
+	 */
+	protected $supportedData = [];
 
-    /**
-     * Creates the property.
-     */
-    public function __construct(array $supportedData = null)
-    {
-        if (is_null($supportedData)) {
-            $supportedData = [
-                ['contentType' => 'text/vcard', 'version' => '3.0'],
-                ['contentType' => 'text/vcard', 'version' => '4.0'],
-                ['contentType' => 'application/vcard+json', 'version' => '4.0'],
-            ];
-        }
+	/**
+	 * Creates the property.
+	 */
+	public function __construct(array $supportedData = null)
+	{
+		if (is_null($supportedData)) {
+			$supportedData = [
+				['contentType' => 'text/vcard', 'version' => '3.0'],
+				['contentType' => 'text/vcard', 'version' => '4.0'],
+				['contentType' => 'application/vcard+json', 'version' => '4.0'],
+			];
+		}
 
-        $this->supportedData = $supportedData;
-    }
+		$this->supportedData = $supportedData;
+	}
 
-    /**
-     * The xmlSerialize method is called during xml writing.
-     *
-     * Use the $writer argument to write its own xml serialization.
-     *
-     * An important note: do _not_ create a parent element. Any element
-     * implementing XmlSerializable should only ever write what's considered
-     * its 'inner xml'.
-     *
-     * The parent of the current element is responsible for writing a
-     * containing element.
-     *
-     * This allows serializers to be re-used for different element names.
-     *
-     * If you are opening new elements, you must also close them again.
-     */
-    public function xmlSerialize(Writer $writer)
-    {
-        foreach ($this->supportedData as $supported) {
-            $writer->startElement('{'.Plugin::NS_CARDDAV.'}address-data-type');
-            $writer->writeAttributes([
-                'content-type' => $supported['contentType'],
-                'version' => $supported['version'],
-                ]);
-            $writer->endElement(); // address-data-type
-        }
-    }
+	/**
+	 * The xmlSerialize method is called during xml writing.
+	 *
+	 * Use the $writer argument to write its own xml serialization.
+	 *
+	 * An important note: do _not_ create a parent element. Any element
+	 * implementing XmlSerializable should only ever write what's considered
+	 * its 'inner xml'.
+	 *
+	 * The parent of the current element is responsible for writing a
+	 * containing element.
+	 *
+	 * This allows serializers to be re-used for different element names.
+	 *
+	 * If you are opening new elements, you must also close them again.
+	 */
+	public function xmlSerialize(Writer $writer)
+	{
+		foreach ($this->supportedData as $supported) {
+			$writer->startElement('{'.Plugin::NS_CARDDAV.'}address-data-type');
+			$writer->writeAttributes([
+				'content-type' => $supported['contentType'],
+				'version' => $supported['version'],
+				]);
+			$writer->endElement(); // address-data-type
+		}
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/CardDAV/Xml/Property/SupportedCollationSet.php

Indentation +22 added lines, -22 removed lines

@@ -19,26 +19,26 @@
  */
 class SupportedCollationSet implements XmlSerializable
 {
-    /**
-     * The xmlSerialize method is called during xml writing.
-     *
-     * Use the $writer argument to write its own xml serialization.
-     *
-     * An important note: do _not_ create a parent element. Any element
-     * implementing XmlSerializable should only ever write what's considered
-     * its 'inner xml'.
-     *
-     * The parent of the current element is responsible for writing a
-     * containing element.
-     *
-     * This allows serializers to be re-used for different element names.
-     *
-     * If you are opening new elements, you must also close them again.
-     */
-    public function xmlSerialize(Writer $writer)
-    {
-        foreach (['i;ascii-casemap', 'i;octet', 'i;unicode-casemap'] as $coll) {
-            $writer->writeElement('{urn:ietf:params:xml:ns:carddav}supported-collation', $coll);
-        }
-    }
+	/**
+	 * The xmlSerialize method is called during xml writing.
+	 *
+	 * Use the $writer argument to write its own xml serialization.
+	 *
+	 * An important note: do _not_ create a parent element. Any element
+	 * implementing XmlSerializable should only ever write what's considered
+	 * its 'inner xml'.
+	 *
+	 * The parent of the current element is responsible for writing a
+	 * containing element.
+	 *
+	 * This allows serializers to be re-used for different element names.
+	 *
+	 * If you are opening new elements, you must also close them again.
+	 */
+	public function xmlSerialize(Writer $writer)
+	{
+		foreach (['i;ascii-casemap', 'i;octet', 'i;unicode-casemap'] as $coll) {
+			$writer->writeElement('{urn:ietf:params:xml:ns:carddav}supported-collation', $coll);
+		}
+	}
 }

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

Indentation +183 added lines, -183 removed lines

@@ -16,187 +16,187 @@
  */
 class Card extends DAV\File implements ICard, DAVACL\IACL
 {
-    use DAVACL\ACLTrait;
-
-    /**
-     * CardDAV backend.
-     *
-     * @var Backend\BackendInterface
-     */
-    protected $carddavBackend;
-
-    /**
-     * Array with information about this Card.
-     *
-     * @var array
-     */
-    protected $cardData;
-
-    /**
-     * Array with information about the containing addressbook.
-     *
-     * @var array
-     */
-    protected $addressBookInfo;
-
-    /**
-     * Constructor.
-     */
-    public function __construct(Backend\BackendInterface $carddavBackend, array $addressBookInfo, array $cardData)
-    {
-        $this->carddavBackend = $carddavBackend;
-        $this->addressBookInfo = $addressBookInfo;
-        $this->cardData = $cardData;
-    }
-
-    /**
-     * Returns the uri for this object.
-     *
-     * @return string
-     */
-    public function getName()
-    {
-        return $this->cardData['uri'];
-    }
-
-    /**
-     * Returns the VCard-formatted object.
-     *
-     * @return string
-     */
-    public function get()
-    {
-        // Pre-populating 'carddata' is optional. If we don't yet have it
-        // already, we fetch it from the backend.
-        if (!isset($this->cardData['carddata'])) {
-            $this->cardData = $this->carddavBackend->getCard($this->addressBookInfo['id'], $this->cardData['uri']);
-        }
-
-        return $this->cardData['carddata'];
-    }
-
-    /**
-     * Updates the VCard-formatted object.
-     *
-     * @param string $cardData
-     *
-     * @return string|null
-     */
-    public function put($cardData)
-    {
-        if (is_resource($cardData)) {
-            $cardData = stream_get_contents($cardData);
-        }
-
-        // Converting to UTF-8, if needed
-        $cardData = DAV\StringUtil::ensureUTF8($cardData);
-
-        $etag = $this->carddavBackend->updateCard($this->addressBookInfo['id'], $this->cardData['uri'], $cardData);
-        $this->cardData['carddata'] = $cardData;
-        $this->cardData['etag'] = $etag;
-
-        return $etag;
-    }
-
-    /**
-     * Deletes the card.
-     */
-    public function delete()
-    {
-        $this->carddavBackend->deleteCard($this->addressBookInfo['id'], $this->cardData['uri']);
-    }
-
-    /**
-     * Returns the mime content-type.
-     *
-     * @return string
-     */
-    public function getContentType()
-    {
-        return 'text/vcard; charset=utf-8';
-    }
-
-    /**
-     * Returns an ETag for this object.
-     *
-     * @return string
-     */
-    public function getETag()
-    {
-        if (isset($this->cardData['etag'])) {
-            return $this->cardData['etag'];
-        } else {
-            $data = $this->get();
-            if (is_string($data)) {
-                return '"'.md5($data).'"';
-            } else {
-                // We refuse to calculate the md5 if it's a stream.
-                return null;
-            }
-        }
-    }
-
-    /**
-     * Returns the last modification date as a unix timestamp.
-     *
-     * @return int
-     */
-    public function getLastModified()
-    {
-        return isset($this->cardData['lastmodified']) ? $this->cardData['lastmodified'] : null;
-    }
-
-    /**
-     * Returns the size of this object in bytes.
-     *
-     * @return int
-     */
-    public function getSize()
-    {
-        if (array_key_exists('size', $this->cardData)) {
-            return $this->cardData['size'];
-        } else {
-            return strlen($this->get());
-        }
-    }
-
-    /**
-     * Returns the owner principal.
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getOwner()
-    {
-        return $this->addressBookInfo['principaluri'];
-    }
-
-    /**
-     * Returns a list of ACE's for this node.
-     *
-     * Each ACE has the following properties:
-     *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
-     *     currently the only supported privileges
-     *   * 'principal', a url to the principal who owns the node
-     *   * 'protected' (optional), indicating that this ACE is not allowed to
-     *      be updated.
-     *
-     * @return array
-     */
-    public function getACL()
-    {
-        // An alternative acl may be specified through the cardData array.
-        if (isset($this->cardData['acl'])) {
-            return $this->cardData['acl'];
-        }
-
-        return [
-            [
-                'privilege' => '{DAV:}all',
-                'principal' => $this->addressBookInfo['principaluri'],
-                'protected' => true,
-            ],
-        ];
-    }
+	use DAVACL\ACLTrait;
+
+	/**
+	 * CardDAV backend.
+	 *
+	 * @var Backend\BackendInterface
+	 */
+	protected $carddavBackend;
+
+	/**
+	 * Array with information about this Card.
+	 *
+	 * @var array
+	 */
+	protected $cardData;
+
+	/**
+	 * Array with information about the containing addressbook.
+	 *
+	 * @var array
+	 */
+	protected $addressBookInfo;
+
+	/**
+	 * Constructor.
+	 */
+	public function __construct(Backend\BackendInterface $carddavBackend, array $addressBookInfo, array $cardData)
+	{
+		$this->carddavBackend = $carddavBackend;
+		$this->addressBookInfo = $addressBookInfo;
+		$this->cardData = $cardData;
+	}
+
+	/**
+	 * Returns the uri for this object.
+	 *
+	 * @return string
+	 */
+	public function getName()
+	{
+		return $this->cardData['uri'];
+	}
+
+	/**
+	 * Returns the VCard-formatted object.
+	 *
+	 * @return string
+	 */
+	public function get()
+	{
+		// Pre-populating 'carddata' is optional. If we don't yet have it
+		// already, we fetch it from the backend.
+		if (!isset($this->cardData['carddata'])) {
+			$this->cardData = $this->carddavBackend->getCard($this->addressBookInfo['id'], $this->cardData['uri']);
+		}
+
+		return $this->cardData['carddata'];
+	}
+
+	/**
+	 * Updates the VCard-formatted object.
+	 *
+	 * @param string $cardData
+	 *
+	 * @return string|null
+	 */
+	public function put($cardData)
+	{
+		if (is_resource($cardData)) {
+			$cardData = stream_get_contents($cardData);
+		}
+
+		// Converting to UTF-8, if needed
+		$cardData = DAV\StringUtil::ensureUTF8($cardData);
+
+		$etag = $this->carddavBackend->updateCard($this->addressBookInfo['id'], $this->cardData['uri'], $cardData);
+		$this->cardData['carddata'] = $cardData;
+		$this->cardData['etag'] = $etag;
+
+		return $etag;
+	}
+
+	/**
+	 * Deletes the card.
+	 */
+	public function delete()
+	{
+		$this->carddavBackend->deleteCard($this->addressBookInfo['id'], $this->cardData['uri']);
+	}
+
+	/**
+	 * Returns the mime content-type.
+	 *
+	 * @return string
+	 */
+	public function getContentType()
+	{
+		return 'text/vcard; charset=utf-8';
+	}
+
+	/**
+	 * Returns an ETag for this object.
+	 *
+	 * @return string
+	 */
+	public function getETag()
+	{
+		if (isset($this->cardData['etag'])) {
+			return $this->cardData['etag'];
+		} else {
+			$data = $this->get();
+			if (is_string($data)) {
+				return '"'.md5($data).'"';
+			} else {
+				// We refuse to calculate the md5 if it's a stream.
+				return null;
+			}
+		}
+	}
+
+	/**
+	 * Returns the last modification date as a unix timestamp.
+	 *
+	 * @return int
+	 */
+	public function getLastModified()
+	{
+		return isset($this->cardData['lastmodified']) ? $this->cardData['lastmodified'] : null;
+	}
+
+	/**
+	 * Returns the size of this object in bytes.
+	 *
+	 * @return int
+	 */
+	public function getSize()
+	{
+		if (array_key_exists('size', $this->cardData)) {
+			return $this->cardData['size'];
+		} else {
+			return strlen($this->get());
+		}
+	}
+
+	/**
+	 * Returns the owner principal.
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getOwner()
+	{
+		return $this->addressBookInfo['principaluri'];
+	}
+
+	/**
+	 * Returns a list of ACE's for this node.
+	 *
+	 * Each ACE has the following properties:
+	 *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
+	 *     currently the only supported privileges
+	 *   * 'principal', a url to the principal who owns the node
+	 *   * 'protected' (optional), indicating that this ACE is not allowed to
+	 *      be updated.
+	 *
+	 * @return array
+	 */
+	public function getACL()
+	{
+		// An alternative acl may be specified through the cardData array.
+		if (isset($this->cardData['acl'])) {
+			return $this->cardData['acl'];
+		}
+
+		return [
+			[
+				'privilege' => '{DAV:}all',
+				'principal' => $this->addressBookInfo['principaluri'],
+				'protected' => true,
+			],
+		];
+	}
 }

htdocs/includes/sabre/sabre/dav/lib/DAVACL/IACL.php

Indentation +48 added lines, -48 removed lines

@@ -17,56 +17,56 @@
  */
 interface IACL extends DAV\INode
 {
-    /**
-     * Returns the owner principal.
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getOwner();
+	/**
+	 * Returns the owner principal.
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getOwner();
 
-    /**
-     * Returns a group principal.
-     *
-     * This must be a url to a principal, or null if there's no owner
-     *
-     * @return string|null
-     */
-    public function getGroup();
+	/**
+	 * Returns a group principal.
+	 *
+	 * This must be a url to a principal, or null if there's no owner
+	 *
+	 * @return string|null
+	 */
+	public function getGroup();
 
-    /**
-     * Returns a list of ACE's for this node.
-     *
-     * Each ACE has the following properties:
-     *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
-     *     currently the only supported privileges
-     *   * 'principal', a url to the principal who owns the node
-     *   * 'protected' (optional), indicating that this ACE is not allowed to
-     *      be updated.
-     *
-     * @return array
-     */
-    public function getACL();
+	/**
+	 * Returns a list of ACE's for this node.
+	 *
+	 * Each ACE has the following properties:
+	 *   * 'privilege', a string such as {DAV:}read or {DAV:}write. These are
+	 *     currently the only supported privileges
+	 *   * 'principal', a url to the principal who owns the node
+	 *   * 'protected' (optional), indicating that this ACE is not allowed to
+	 *      be updated.
+	 *
+	 * @return array
+	 */
+	public function getACL();
 
-    /**
-     * Updates the ACL.
-     *
-     * This method will receive a list of new ACE's as an array argument.
-     */
-    public function setACL(array $acl);
+	/**
+	 * Updates the ACL.
+	 *
+	 * This method will receive a list of new ACE's as an array argument.
+	 */
+	public function setACL(array $acl);
 
-    /**
-     * Returns the list of supported privileges for this node.
-     *
-     * The returned data structure is a list of nested privileges.
-     * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
-     * standard structure.
-     *
-     * If null is returned from this method, the default privilege set is used,
-     * which is fine for most common usecases.
-     *
-     * @return array|null
-     */
-    public function getSupportedPrivilegeSet();
+	/**
+	 * Returns the list of supported privileges for this node.
+	 *
+	 * The returned data structure is a list of nested privileges.
+	 * See Sabre\DAVACL\Plugin::getDefaultSupportedPrivilegeSet for a simple
+	 * standard structure.
+	 *
+	 * If null is returned from this method, the default privilege set is used,
+	 * which is fine for most common usecases.
+	 *
+	 * @return array|null
+	 */
+	public function getSupportedPrivilegeSet();
 }