nextcloud / server

apps/dav/lib/CardDAV/Integration/ExternalAddressBook.php

Indentation +96 added lines, -96 removed lines

@@ -33,100 +33,100 @@
  */
 abstract class ExternalAddressBook implements IAddressBook, DAV\IProperties {
 
-	/** @var string */
-	private const PREFIX = 'z-app-generated';
-
-	/**
-	 * @var string
-	 *
-	 * Double dash is a valid delimiter,
-	 * because it will always split the URIs correctly:
-	 * - our prefix contains only one dash and won't be split
-	 * - appIds are not allowed to contain dashes as per spec:
-	 * > must contain only lowercase ASCII characters and underscore
-	 * - explode has a limit of three, so even if the app-generated
-	 *   URI has double dashes, it won't be split
-	 */
-	private const DELIMITER = '--';
-
-	/** @var string */
-	private $appId;
-
-	/** @var string */
-	private $uri;
-
-	/**
-	 * @param string $appId
-	 * @param string $uri
-	 */
-	public function __construct(string $appId, string $uri) {
-		$this->appId = $appId;
-		$this->uri = $uri;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	final public function getName() {
-		return implode(self::DELIMITER, [
-			self::PREFIX,
-			$this->appId,
-			$this->uri,
-		]);
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	final public function setName($name) {
-		throw new DAV\Exception\MethodNotAllowed('Renaming address books is not yet supported');
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	final public function createDirectory($name) {
-		throw new DAV\Exception\MethodNotAllowed('Creating collections in address book objects is not allowed');
-	}
-
-	/**
-	 * Checks whether the address book uri is app-generated
-	 *
-	 * @param string $uri
-	 *
-	 * @return bool
-	 */
-	public static function isAppGeneratedAddressBook(string $uri): bool {
-		return strpos($uri, self::PREFIX) === 0 && substr_count($uri, self::DELIMITER) >= 2;
-	}
-
-	/**
-	 * Splits an app-generated uri into appId and uri
-	 *
-	 * @param string $uri
-	 *
-	 * @return array
-	 */
-	public static function splitAppGeneratedAddressBookUri(string $uri): array {
-		$array = array_slice(explode(self::DELIMITER, $uri, 3), 1);
-		// Check the array has expected amount of elements
-		// and none of them is an empty string
-		if (\count($array) !== 2 || \in_array('', $array, true)) {
-			throw new \InvalidArgumentException('Provided address book uri was not app-generated');
-		}
-
-		return $array;
-	}
-
-	/**
-	 * Checks whether a address book name the user wants to create violates
-	 * the reserved name for URIs
-	 *
-	 * @param string $uri
-	 *
-	 * @return bool
-	 */
-	public static function doesViolateReservedName(string $uri): bool {
-		return strpos($uri, self::PREFIX) === 0;
-	}
+    /** @var string */
+    private const PREFIX = 'z-app-generated';
+
+    /**
+     * @var string
+     *
+     * Double dash is a valid delimiter,
+     * because it will always split the URIs correctly:
+     * - our prefix contains only one dash and won't be split
+     * - appIds are not allowed to contain dashes as per spec:
+     * > must contain only lowercase ASCII characters and underscore
+     * - explode has a limit of three, so even if the app-generated
+     *   URI has double dashes, it won't be split
+     */
+    private const DELIMITER = '--';
+
+    /** @var string */
+    private $appId;
+
+    /** @var string */
+    private $uri;
+
+    /**
+     * @param string $appId
+     * @param string $uri
+     */
+    public function __construct(string $appId, string $uri) {
+        $this->appId = $appId;
+        $this->uri = $uri;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    final public function getName() {
+        return implode(self::DELIMITER, [
+            self::PREFIX,
+            $this->appId,
+            $this->uri,
+        ]);
+    }
+
+    /**
+     * @inheritDoc
+     */
+    final public function setName($name) {
+        throw new DAV\Exception\MethodNotAllowed('Renaming address books is not yet supported');
+    }
+
+    /**
+     * @inheritDoc
+     */
+    final public function createDirectory($name) {
+        throw new DAV\Exception\MethodNotAllowed('Creating collections in address book objects is not allowed');
+    }
+
+    /**
+     * Checks whether the address book uri is app-generated
+     *
+     * @param string $uri
+     *
+     * @return bool
+     */
+    public static function isAppGeneratedAddressBook(string $uri): bool {
+        return strpos($uri, self::PREFIX) === 0 && substr_count($uri, self::DELIMITER) >= 2;
+    }
+
+    /**
+     * Splits an app-generated uri into appId and uri
+     *
+     * @param string $uri
+     *
+     * @return array
+     */
+    public static function splitAppGeneratedAddressBookUri(string $uri): array {
+        $array = array_slice(explode(self::DELIMITER, $uri, 3), 1);
+        // Check the array has expected amount of elements
+        // and none of them is an empty string
+        if (\count($array) !== 2 || \in_array('', $array, true)) {
+            throw new \InvalidArgumentException('Provided address book uri was not app-generated');
+        }
+
+        return $array;
+    }
+
+    /**
+     * Checks whether a address book name the user wants to create violates
+     * the reserved name for URIs
+     *
+     * @param string $uri
+     *
+     * @return bool
+     */
+    public static function doesViolateReservedName(string $uri): bool {
+        return strpos($uri, self::PREFIX) === 0;
+    }
 }

apps/contactsinteraction/lib/Db/RecentContactMapper.php

Indentation +106 added lines, -106 removed lines

@@ -31,110 +31,110 @@
 use OCP\IUser;
 
 class RecentContactMapper extends QBMapper {
-	public const TABLE_NAME = 'recent_contact';
-
-	public function __construct(IDBConnection $db) {
-		parent::__construct($db, self::TABLE_NAME);
-	}
-
-	/**
-	 * @return RecentContact[]
-	 */
-	public function findAll(string $uid): array {
-		$qb = $this->db->getQueryBuilder();
-
-		$select = $qb
-			->select('*')
-			->from($this->getTableName())
-			->where($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)));
-
-		return $this->findEntities($select);
-	}
-
-	/**
-	 * @param string $uid
-	 * @param int $id
-	 *
-	 * @return RecentContact
-	 * @throws DoesNotExistException
-	 */
-	public function find(string $uid, int $id): RecentContact {
-		$qb = $this->db->getQueryBuilder();
-
-		$select = $qb
-			->select('*')
-			->from($this->getTableName())
-			->where($qb->expr()->eq('id', $qb->createNamedParameter($id, $qb::PARAM_INT)))
-			->andWhere($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)));
-
-		return $this->findEntity($select);
-	}
-
-	/**
-	 * @param IUser $user
-	 * @param string|null $uid
-	 * @param string|null $email
-	 * @param string|null $cloudId
-	 *
-	 * @return RecentContact[]
-	 */
-	public function findMatch(IUser $user,
-							  ?string $uid,
-							  ?string $email,
-							  ?string $cloudId): array {
-		$qb = $this->db->getQueryBuilder();
-
-		$or = $qb->expr()->orX();
-		if ($uid !== null) {
-			$or->add($qb->expr()->eq('uid', $qb->createNamedParameter($uid)));
-		}
-		if ($email !== null) {
-			$or->add($qb->expr()->eq('email', $qb->createNamedParameter($email)));
-		}
-		if ($cloudId !== null) {
-			$or->add($qb->expr()->eq('federated_cloud_id', $qb->createNamedParameter($cloudId)));
-		}
-
-		$select = $qb
-			->select('*')
-			->from($this->getTableName())
-			->where($or)
-			->andWhere($qb->expr()->eq('actor_uid', $qb->createNamedParameter($user->getUID())));
-
-		return $this->findEntities($select);
-	}
-
-	/**
-	 * @param string $uid
-	 * @return int|null
-	 */
-	public function findLastUpdatedForUserId(string $uid):?int {
-		$qb = $this->db->getQueryBuilder();
-
-		$select = $qb
-			->select('last_contact')
-			->from($this->getTableName())
-			->where($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)))
-			->orderBy('last_contact', 'DESC')
-			->setMaxResults(1);
-
-		$cursor = $select->execute();
-		$row = $cursor->fetch();
-
-		if ($row === false) {
-			return null;
-		}
-
-		return (int)$row['last_contact'];
-	}
-
-	public function cleanUp(int $olderThan): void {
-		$qb = $this->db->getQueryBuilder();
-
-		$delete = $qb
-			->delete($this->getTableName())
-			->where($qb->expr()->lt('last_contact', $qb->createNamedParameter($olderThan)));
-
-		$delete->execute();
-	}
+    public const TABLE_NAME = 'recent_contact';
+
+    public function __construct(IDBConnection $db) {
+        parent::__construct($db, self::TABLE_NAME);
+    }
+
+    /**
+     * @return RecentContact[]
+     */
+    public function findAll(string $uid): array {
+        $qb = $this->db->getQueryBuilder();
+
+        $select = $qb
+            ->select('*')
+            ->from($this->getTableName())
+            ->where($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)));
+
+        return $this->findEntities($select);
+    }
+
+    /**
+     * @param string $uid
+     * @param int $id
+     *
+     * @return RecentContact
+     * @throws DoesNotExistException
+     */
+    public function find(string $uid, int $id): RecentContact {
+        $qb = $this->db->getQueryBuilder();
+
+        $select = $qb
+            ->select('*')
+            ->from($this->getTableName())
+            ->where($qb->expr()->eq('id', $qb->createNamedParameter($id, $qb::PARAM_INT)))
+            ->andWhere($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)));
+
+        return $this->findEntity($select);
+    }
+
+    /**
+     * @param IUser $user
+     * @param string|null $uid
+     * @param string|null $email
+     * @param string|null $cloudId
+     *
+     * @return RecentContact[]
+     */
+    public function findMatch(IUser $user,
+                              ?string $uid,
+                              ?string $email,
+                              ?string $cloudId): array {
+        $qb = $this->db->getQueryBuilder();
+
+        $or = $qb->expr()->orX();
+        if ($uid !== null) {
+            $or->add($qb->expr()->eq('uid', $qb->createNamedParameter($uid)));
+        }
+        if ($email !== null) {
+            $or->add($qb->expr()->eq('email', $qb->createNamedParameter($email)));
+        }
+        if ($cloudId !== null) {
+            $or->add($qb->expr()->eq('federated_cloud_id', $qb->createNamedParameter($cloudId)));
+        }
+
+        $select = $qb
+            ->select('*')
+            ->from($this->getTableName())
+            ->where($or)
+            ->andWhere($qb->expr()->eq('actor_uid', $qb->createNamedParameter($user->getUID())));
+
+        return $this->findEntities($select);
+    }
+
+    /**
+     * @param string $uid
+     * @return int|null
+     */
+    public function findLastUpdatedForUserId(string $uid):?int {
+        $qb = $this->db->getQueryBuilder();
+
+        $select = $qb
+            ->select('last_contact')
+            ->from($this->getTableName())
+            ->where($qb->expr()->eq('actor_uid', $qb->createNamedParameter($uid)))
+            ->orderBy('last_contact', 'DESC')
+            ->setMaxResults(1);
+
+        $cursor = $select->execute();
+        $row = $cursor->fetch();
+
+        if ($row === false) {
+            return null;
+        }
+
+        return (int)$row['last_contact'];
+    }
+
+    public function cleanUp(int $olderThan): void {
+        $qb = $this->db->getQueryBuilder();
+
+        $delete = $qb
+            ->delete($this->getTableName())
+            ->where($qb->expr()->lt('last_contact', $qb->createNamedParameter($olderThan)));
+
+        $delete->execute();
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
 	 * @param string $uid
 	 * @return int|null
 	 */
-	public function findLastUpdatedForUserId(string $uid):?int {
+	public function findLastUpdatedForUserId(string $uid): ?int {
 		$qb = $this->db->getQueryBuilder();
 
 		$select = $qb
@@ -125,7 +125,7 @@ 
 			return null;
 		}
 
-		return (int)$row['last_contact'];
+		return (int) $row['last_contact'];
 	}
 
 	public function cleanUp(int $olderThan): void {

apps/contactsinteraction/lib/AddressBook.php

Indentation +134 added lines, -134 removed lines

@@ -39,138 +39,138 @@
 use Sabre\DAVACL\IACL;
 
 class AddressBook extends ExternalAddressBook implements IACL {
-	public const URI = 'recent';
-
-	use ACLTrait;
-
-	/** @var RecentContactMapper */
-	private $mapper;
-
-	/** @var IL10N */
-	private $l10n;
-
-	/** @var string */
-	private $principalUri;
-
-	public function __construct(RecentContactMapper $mapper,
-								IL10N $l10n,
-								string $principalUri) {
-		parent::__construct(Application::APP_ID, self::URI);
-
-		$this->mapper = $mapper;
-		$this->l10n = $l10n;
-		$this->principalUri = $principalUri;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function delete(): void {
-		throw new Exception("This addressbook is immutable");
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function createFile($name, $data = null) {
-		throw new Exception("This addressbook is immutable");
-	}
-
-	/**
-	 * @inheritDoc
-	 * @throws NotFound
-	 */
-	public function getChild($name) {
-		try {
-			return new Card(
-				$this->mapper->find(
-					$this->getUid(),
-					(int)$name
-				),
-				$this->principalUri,
-				$this->getACL()
-			);
-		} catch (DoesNotExistException $ex) {
-			throw new NotFound("Contact does not exist: " . $ex->getMessage(), 0, $ex);
-		}
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getChildren(): array {
-		return array_map(
-			function (RecentContact $contact) {
-				return new Card(
-					$contact,
-					$this->principalUri,
-					$this->getACL()
-				);
-			},
-			$this->mapper->findAll($this->getUid())
-		);
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function childExists($name) {
-		try {
-			$this->mapper->find(
-				$this->getUid(),
-				(int)$name
-			);
-			return true;
-		} catch (DoesNotExistException $e) {
-			return false;
-		}
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getLastModified(): ?int {
-		return $this->mapper->findLastUpdatedForUserId($this->getUid());
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function propPatch(PropPatch $propPatch) {
-		throw new Exception("This addressbook is immutable");
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getProperties($properties) {
-		return [
-			'principaluri' => $this->principalUri,
-			'{DAV:}displayname' => $this->l10n->t('Recently contacted'),
-			'{' . Plugin::NS_OWNCLOUD . '}read-only' => true,
-			'{' . \OCA\DAV\CalDAV\Plugin::NS_CALENDARSERVER . '}getctag' => 'http://sabre.io/ns/sync/' . ($this->getLastModified() ?? 0),
-		];
-	}
-
-	public function getOwner(): string {
-		return $this->principalUri;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getACL() {
-		return [
-			[
-				'privilege' => '{DAV:}read',
-				'principal' => $this->getOwner(),
-				'protected' => true,
-			],
-		];
-	}
-
-	private function getUid(): string {
-		list(, $uid) = \Sabre\Uri\split($this->principalUri);
-		return $uid;
-	}
+    public const URI = 'recent';
+
+    use ACLTrait;
+
+    /** @var RecentContactMapper */
+    private $mapper;
+
+    /** @var IL10N */
+    private $l10n;
+
+    /** @var string */
+    private $principalUri;
+
+    public function __construct(RecentContactMapper $mapper,
+                                IL10N $l10n,
+                                string $principalUri) {
+        parent::__construct(Application::APP_ID, self::URI);
+
+        $this->mapper = $mapper;
+        $this->l10n = $l10n;
+        $this->principalUri = $principalUri;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function delete(): void {
+        throw new Exception("This addressbook is immutable");
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function createFile($name, $data = null) {
+        throw new Exception("This addressbook is immutable");
+    }
+
+    /**
+     * @inheritDoc
+     * @throws NotFound
+     */
+    public function getChild($name) {
+        try {
+            return new Card(
+                $this->mapper->find(
+                    $this->getUid(),
+                    (int)$name
+                ),
+                $this->principalUri,
+                $this->getACL()
+            );
+        } catch (DoesNotExistException $ex) {
+            throw new NotFound("Contact does not exist: " . $ex->getMessage(), 0, $ex);
+        }
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getChildren(): array {
+        return array_map(
+            function (RecentContact $contact) {
+                return new Card(
+                    $contact,
+                    $this->principalUri,
+                    $this->getACL()
+                );
+            },
+            $this->mapper->findAll($this->getUid())
+        );
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function childExists($name) {
+        try {
+            $this->mapper->find(
+                $this->getUid(),
+                (int)$name
+            );
+            return true;
+        } catch (DoesNotExistException $e) {
+            return false;
+        }
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getLastModified(): ?int {
+        return $this->mapper->findLastUpdatedForUserId($this->getUid());
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function propPatch(PropPatch $propPatch) {
+        throw new Exception("This addressbook is immutable");
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getProperties($properties) {
+        return [
+            'principaluri' => $this->principalUri,
+            '{DAV:}displayname' => $this->l10n->t('Recently contacted'),
+            '{' . Plugin::NS_OWNCLOUD . '}read-only' => true,
+            '{' . \OCA\DAV\CalDAV\Plugin::NS_CALENDARSERVER . '}getctag' => 'http://sabre.io/ns/sync/' . ($this->getLastModified() ?? 0),
+        ];
+    }
+
+    public function getOwner(): string {
+        return $this->principalUri;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getACL() {
+        return [
+            [
+                'privilege' => '{DAV:}read',
+                'principal' => $this->getOwner(),
+                'protected' => true,
+            ],
+        ];
+    }
+
+    private function getUid(): string {
+        list(, $uid) = \Sabre\Uri\split($this->principalUri);
+        return $uid;
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -85,13 +85,13 @@ 
 			return new Card(
 				$this->mapper->find(
 					$this->getUid(),
-					(int)$name
+					(int) $name
 				),
 				$this->principalUri,
 				$this->getACL()
 			);
 		} catch (DoesNotExistException $ex) {
-			throw new NotFound("Contact does not exist: " . $ex->getMessage(), 0, $ex);
+			throw new NotFound("Contact does not exist: ".$ex->getMessage(), 0, $ex);
 		}
 	}
 
@@ -100,7 +100,7 @@ 
 	 */
 	public function getChildren(): array {
 		return array_map(
-			function (RecentContact $contact) {
+			function(RecentContact $contact) {
 				return new Card(
 					$contact,
 					$this->principalUri,
@@ -118,7 +118,7 @@ 
 		try {
 			$this->mapper->find(
 				$this->getUid(),
-				(int)$name
+				(int) $name
 			);
 			return true;
 		} catch (DoesNotExistException $e) {
@@ -147,8 +147,8 @@ 
 		return [
 			'principaluri' => $this->principalUri,
 			'{DAV:}displayname' => $this->l10n->t('Recently contacted'),
-			'{' . Plugin::NS_OWNCLOUD . '}read-only' => true,
-			'{' . \OCA\DAV\CalDAV\Plugin::NS_CALENDARSERVER . '}getctag' => 'http://sabre.io/ns/sync/' . ($this->getLastModified() ?? 0),
+			'{'.Plugin::NS_OWNCLOUD.'}read-only' => true,
+			'{'.\OCA\DAV\CalDAV\Plugin::NS_CALENDARSERVER.'}getctag' => 'http://sabre.io/ns/sync/'.($this->getLastModified() ?? 0),
 		];
 	}
 

apps/contactsinteraction/lib/Card.php

Indentation +100 added lines, -100 removed lines

@@ -32,104 +32,104 @@
 use Sabre\DAVACL\IACL;
 
 class Card implements ICard, IACL {
-	use ACLTrait;
-
-	/** @var RecentContact */
-	private $contact;
-
-	/** @var string */
-	private $principal;
-
-	/** @var array */
-	private $acls;
-
-	public function __construct(RecentContact $contact, string $principal, array $acls) {
-		$this->contact = $contact;
-		$this->principal = $principal;
-		$this->acls = $acls;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getOwner(): ?string {
-		$this->principal;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getACL(): array {
-		return $this->acls;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function setAcls(array $acls): void {
-		throw new NotImplemented();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function put($data): ?string {
-		throw new NotImplemented();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function get() {
-		return $this->contact->getCard();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getContentType(): ?string {
-		return 'text/vcard; charset=utf-8';
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getETag(): ?string {
-		return '"' . md5((string) $this->getLastModified()) . '"';
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getSize(): int {
-		throw new NotImplemented();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function delete(): void {
-		throw new NotImplemented();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getName(): string {
-		return (string) $this->contact->getId();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function setName($name): void {
-		throw new NotImplemented();
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function getLastModified(): ?int {
-		return $this->contact->getLastContact();
-	}
+    use ACLTrait;
+
+    /** @var RecentContact */
+    private $contact;
+
+    /** @var string */
+    private $principal;
+
+    /** @var array */
+    private $acls;
+
+    public function __construct(RecentContact $contact, string $principal, array $acls) {
+        $this->contact = $contact;
+        $this->principal = $principal;
+        $this->acls = $acls;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getOwner(): ?string {
+        $this->principal;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getACL(): array {
+        return $this->acls;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function setAcls(array $acls): void {
+        throw new NotImplemented();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function put($data): ?string {
+        throw new NotImplemented();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function get() {
+        return $this->contact->getCard();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getContentType(): ?string {
+        return 'text/vcard; charset=utf-8';
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getETag(): ?string {
+        return '"' . md5((string) $this->getLastModified()) . '"';
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getSize(): int {
+        throw new NotImplemented();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function delete(): void {
+        throw new NotImplemented();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getName(): string {
+        return (string) $this->contact->getId();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function setName($name): void {
+        throw new NotImplemented();
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getLastModified(): ?int {
+        return $this->contact->getLastContact();
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 	 * @inheritDoc
 	 */
 	public function getETag(): ?string {
-		return '"' . md5((string) $this->getLastModified()) . '"';
+		return '"'.md5((string) $this->getLastModified()).'"';
 	}
 
 	/**