From 3997856d767ac1f9a94fb5c84ab20d527834b233 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Mon, 22 Jun 2020 11:09:32 +0200 Subject: [PATCH] Fix code style Signed-off-by: Joas Schilling --- lib/private/ContactsManager.php | 305 ++++++++++++++++---------------- 1 file changed, 152 insertions(+), 153 deletions(-) diff --git a/lib/private/ContactsManager.php b/lib/private/ContactsManager.php index 0f041ca2e36..c79d9e01af9 100644 --- a/lib/private/ContactsManager.php +++ b/lib/private/ContactsManager.php @@ -29,184 +29,183 @@ * */ -namespace OC { - - use OCP\IAddressBook; - - class ContactsManager implements \OCP\Contacts\IManager { - - /** - * This function is used to search and find contacts within the users address books. - * In case $pattern is empty all contacts will be returned. - * - * @param string $pattern which should match within the $searchProperties - * @param array $searchProperties defines the properties within the query pattern should match - * @param array $options = array() to define the search behavior - * - 'escape_like_param' - If set to false wildcards _ and % are not escaped - * - 'limit' - Set a numeric limit for the search results - * - 'offset' - Set the offset for the limited search results - * @return array an array of contacts which are arrays of key-value-pairs - */ - public function search($pattern, $searchProperties = [], $options = []) { - $this->loadAddressBooks(); - $result = []; - foreach ($this->addressBooks as $addressBook) { - $r = $addressBook->search($pattern, $searchProperties, $options); - $contacts = []; - foreach ($r as $c) { - $c['addressbook-key'] = $addressBook->getKey(); - $contacts[] = $c; - } - $result = array_merge($result, $contacts); +namespace OC; + +use OCP\IAddressBook; + +class ContactsManager implements \OCP\Contacts\IManager { + + /** + * This function is used to search and find contacts within the users address books. + * In case $pattern is empty all contacts will be returned. + * + * @param string $pattern which should match within the $searchProperties + * @param array $searchProperties defines the properties within the query pattern should match + * @param array $options = array() to define the search behavior + * - 'escape_like_param' - If set to false wildcards _ and % are not escaped + * - 'limit' - Set a numeric limit for the search results + * - 'offset' - Set the offset for the limited search results + * @return array an array of contacts which are arrays of key-value-pairs + */ + public function search($pattern, $searchProperties = [], $options = []) { + $this->loadAddressBooks(); + $result = []; + foreach ($this->addressBooks as $addressBook) { + $r = $addressBook->search($pattern, $searchProperties, $options); + $contacts = []; + foreach ($r as $c) { + $c['addressbook-key'] = $addressBook->getKey(); + $contacts[] = $c; } - - return $result; + $result = array_merge($result, $contacts); } - /** - * This function can be used to delete the contact identified by the given id - * - * @param object $id the unique identifier to a contact - * @param string $addressBookKey identifier of the address book in which the contact shall be deleted - * @return bool successful or not - */ - public function delete($id, $addressBookKey) { - $addressBook = $this->getAddressBook($addressBookKey); - if (!$addressBook) { - return null; - } - - if ($addressBook->getPermissions() & \OCP\Constants::PERMISSION_DELETE) { - return $addressBook->delete($id); - } + return $result; + } + /** + * This function can be used to delete the contact identified by the given id + * + * @param object $id the unique identifier to a contact + * @param string $addressBookKey identifier of the address book in which the contact shall be deleted + * @return bool successful or not + */ + public function delete($id, $addressBookKey) { + $addressBook = $this->getAddressBook($addressBookKey); + if (!$addressBook) { return null; } - /** - * This function is used to create a new contact if 'id' is not given or not present. - * Otherwise the contact will be updated by replacing the entire data set. - * - * @param array $properties this array if key-value-pairs defines a contact - * @param string $addressBookKey identifier of the address book in which the contact shall be created or updated - * @return array representing the contact just created or updated - */ - public function createOrUpdate($properties, $addressBookKey) { - $addressBook = $this->getAddressBook($addressBookKey); - if (!$addressBook) { - return null; - } + if ($addressBook->getPermissions() & \OCP\Constants::PERMISSION_DELETE) { + return $addressBook->delete($id); + } - if ($addressBook->getPermissions() & \OCP\Constants::PERMISSION_CREATE) { - return $addressBook->createOrUpdate($properties); - } + return null; + } + /** + * This function is used to create a new contact if 'id' is not given or not present. + * Otherwise the contact will be updated by replacing the entire data set. + * + * @param array $properties this array if key-value-pairs defines a contact + * @param string $addressBookKey identifier of the address book in which the contact shall be created or updated + * @return array representing the contact just created or updated + */ + public function createOrUpdate($properties, $addressBookKey) { + $addressBook = $this->getAddressBook($addressBookKey); + if (!$addressBook) { return null; } - /** - * Check if contacts are available (e.g. contacts app enabled) - * - * @return bool true if enabled, false if not - */ - public function isEnabled() { - return !empty($this->addressBooks) || !empty($this->addressBookLoaders); + if ($addressBook->getPermissions() & \OCP\Constants::PERMISSION_CREATE) { + return $addressBook->createOrUpdate($properties); } - /** - * @param IAddressBook $addressBook - */ - public function registerAddressBook(IAddressBook $addressBook) { - $this->addressBooks[$addressBook->getKey()] = $addressBook; - } + return null; + } - /** - * @param IAddressBook $addressBook - */ - public function unregisterAddressBook(IAddressBook $addressBook) { - unset($this->addressBooks[$addressBook->getKey()]); - } + /** + * Check if contacts are available (e.g. contacts app enabled) + * + * @return bool true if enabled, false if not + */ + public function isEnabled() { + return !empty($this->addressBooks) || !empty($this->addressBookLoaders); + } - /** - * Return a list of the user's addressbooks display names - * ! The addressBook displayName are not unique, please use getUserAddressBooks - * - * @return array - * @since 6.0.0 - * @deprecated 16.0.0 - Use `$this->getUserAddressBooks()` instead - */ - public function getAddressBooks() { - $this->loadAddressBooks(); - $result = []; - foreach ($this->addressBooks as $addressBook) { - $result[$addressBook->getKey()] = $addressBook->getDisplayName(); - } + /** + * @param IAddressBook $addressBook + */ + public function registerAddressBook(IAddressBook $addressBook) { + $this->addressBooks[$addressBook->getKey()] = $addressBook; + } - return $result; - } + /** + * @param IAddressBook $addressBook + */ + public function unregisterAddressBook(IAddressBook $addressBook) { + unset($this->addressBooks[$addressBook->getKey()]); + } - /** - * Return a list of the user's addressbooks - * - * @return IAddressBook[] - * @since 16.0.0 - */ - public function getUserAddressBooks(): array { - $this->loadAddressBooks(); - return $this->addressBooks; + /** + * Return a list of the user's addressbooks display names + * ! The addressBook displayName are not unique, please use getUserAddressBooks + * + * @return array + * @since 6.0.0 + * @deprecated 16.0.0 - Use `$this->getUserAddressBooks()` instead + */ + public function getAddressBooks() { + $this->loadAddressBooks(); + $result = []; + foreach ($this->addressBooks as $addressBook) { + $result[$addressBook->getKey()] = $addressBook->getDisplayName(); } - /** - * removes all registered address book instances - */ - public function clear() { - $this->addressBooks = []; - $this->addressBookLoaders = []; - } + return $result; + } - /** - * @var IAddressBook[] which holds all registered address books - */ - private $addressBooks = []; - - /** - * @var \Closure[] to call to load/register address books - */ - private $addressBookLoaders = []; - - /** - * In order to improve lazy loading a closure can be registered which will be called in case - * address books are actually requested - * - * @param \Closure $callable - */ - public function register(\Closure $callable) { - $this->addressBookLoaders[] = $callable; - } + /** + * Return a list of the user's addressbooks + * + * @return IAddressBook[] + * @since 16.0.0 + */ + public function getUserAddressBooks(): array { + $this->loadAddressBooks(); + return $this->addressBooks; + } - /** - * Get (and load when needed) the address book for $key - * - * @param string $addressBookKey - * @return IAddressBook - */ - protected function getAddressBook($addressBookKey) { - $this->loadAddressBooks(); - if (!array_key_exists($addressBookKey, $this->addressBooks)) { - return null; - } + /** + * removes all registered address book instances + */ + public function clear() { + $this->addressBooks = []; + $this->addressBookLoaders = []; + } + + /** + * @var IAddressBook[] which holds all registered address books + */ + private $addressBooks = []; + + /** + * @var \Closure[] to call to load/register address books + */ + private $addressBookLoaders = []; + + /** + * In order to improve lazy loading a closure can be registered which will be called in case + * address books are actually requested + * + * @param \Closure $callable + */ + public function register(\Closure $callable) { + $this->addressBookLoaders[] = $callable; + } - return $this->addressBooks[$addressBookKey]; + /** + * Get (and load when needed) the address book for $key + * + * @param string $addressBookKey + * @return IAddressBook + */ + protected function getAddressBook($addressBookKey) { + $this->loadAddressBooks(); + if (!array_key_exists($addressBookKey, $this->addressBooks)) { + return null; } - /** - * Load all address books registered with 'register' - */ - protected function loadAddressBooks() { - foreach ($this->addressBookLoaders as $callable) { - $callable($this); - } - $this->addressBookLoaders = []; + return $this->addressBooks[$addressBookKey]; + } + + /** + * Load all address books registered with 'register' + */ + protected function loadAddressBooks() { + foreach ($this->addressBookLoaders as $callable) { + $callable($this); } + $this->addressBookLoaders = []; } } -- 2.39.5