nextcloud / server

apps/federation/lib/DbHandler.php

Spacing +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
 			return $query->getLastInsertId();
 		}
 
-		$message = 'Internal failure, Could not add trusted server: ' . $url;
+		$message = 'Internal failure, Could not add trusted server: '.$url;
 		$message_t = $this->IL10N->t('Could not add server');
 		throw new HintException($message, $message_t);
 		return -1;
@@ -114,7 +114,7 @@ 
 		$qResult->closeCursor();
 
 		if (empty($result)) {
-			throw new \Exception('No Server found with ID: ' . $id);
+			throw new \Exception('No Server found with ID: '.$id);
 		}
 
 		return $result[0];
@@ -183,7 +183,7 @@ 
 		$statement->closeCursor();
 
 		if (!isset($result['token'])) {
-			throw new \Exception('No token found for: ' . $url);
+			throw new \Exception('No token found for: '.$url);
 		}
 
 		return $result['token'];
@@ -216,7 +216,7 @@ 
 		$statement = $query->executeQuery();
 		$result = $statement->fetch();
 		$statement->closeCursor();
-		return (string)$result['shared_secret'];
+		return (string) $result['shared_secret'];
 	}
 
 	/**
@@ -247,7 +247,7 @@ 
 		$statement = $query->executeQuery();
 		$result = $statement->fetch();
 		$statement->closeCursor();
-		return (int)$result['status'];
+		return (int) $result['status'];
 	}
 
 	/**

Indentation +240 added lines, -240 removed lines

@@ -25,244 +25,244 @@
  * @package OCA\Federation
  */
 class DbHandler {
-	private string $dbTable = 'trusted_servers';
-
-	public function __construct(
-		private IDBConnection $connection,
-		private IL10N $IL10N,
-	) {
-	}
-
-	/**
-	 * Add server to the list of trusted servers
-	 *
-	 * @throws HintException
-	 */
-	public function addServer(string $url): int {
-		$hash = $this->hash($url);
-		$url = rtrim($url, '/');
-		$query = $this->connection->getQueryBuilder();
-		$query->insert($this->dbTable)
-			->values([
-				'url' => $query->createParameter('url'),
-				'url_hash' => $query->createParameter('url_hash'),
-			])
-			->setParameter('url', $url)
-			->setParameter('url_hash', $hash);
-
-		$result = $query->executeStatement();
-
-		if ($result) {
-			return $query->getLastInsertId();
-		}
-
-		$message = 'Internal failure, Could not add trusted server: ' . $url;
-		$message_t = $this->IL10N->t('Could not add server');
-		throw new HintException($message, $message_t);
-		return -1;
-	}
-
-	/**
-	 * Remove server from the list of trusted servers
-	 */
-	public function removeServer(int $id): void {
-		$query = $this->connection->getQueryBuilder();
-		$query->delete($this->dbTable)
-			->where($query->expr()->eq('id', $query->createParameter('id')))
-			->setParameter('id', $id);
-		$query->executeStatement();
-	}
-
-	/**
-	 * Get trusted server with given ID
-	 *
-	 * @return array{id: int, url: string, url_hash: string, token: ?string, shared_secret: ?string, status: int, sync_token: ?string}
-	 * @throws \Exception
-	 */
-	public function getServerById(int $id): array {
-		$query = $this->connection->getQueryBuilder();
-		$query->select('*')->from($this->dbTable)
-			->where($query->expr()->eq('id', $query->createParameter('id')))
-			->setParameter('id', $id, IQueryBuilder::PARAM_INT);
-
-		$qResult = $query->executeQuery();
-		$result = $qResult->fetchAll();
-		$qResult->closeCursor();
-
-		if (empty($result)) {
-			throw new \Exception('No Server found with ID: ' . $id);
-		}
-
-		return $result[0];
-	}
-
-	/**
-	 * Get all trusted servers
-	 *
-	 * @return list<array{id: int, url: string, url_hash: string, shared_secret: ?string, status: int, sync_token: ?string}>
-	 * @throws DBException
-	 */
-	public function getAllServer(): array {
-		$query = $this->connection->getQueryBuilder();
-		$query->select(['url', 'url_hash', 'id', 'status', 'shared_secret', 'sync_token'])
-			->from($this->dbTable);
-		$statement = $query->executeQuery();
-		$result = $statement->fetchAll();
-		$statement->closeCursor();
-		return $result;
-	}
-
-	/**
-	 * Check if server already exists in the database table
-	 */
-	public function serverExists(string $url): bool {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->select('url')
-			->from($this->dbTable)
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash);
-		$statement = $query->executeQuery();
-		$result = $statement->fetchAll();
-		$statement->closeCursor();
-
-		return !empty($result);
-	}
-
-	/**
-	 * Write token to database. Token is used to exchange the secret
-	 */
-	public function addToken(string $url, string $token): void {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->update($this->dbTable)
-			->set('token', $query->createParameter('token'))
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash)
-			->setParameter('token', $token);
-		$query->executeStatement();
-	}
-
-	/**
-	 * Get token stored in database
-	 * @throws \Exception
-	 */
-	public function getToken(string $url): string {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->select('token')->from($this->dbTable)
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash);
-
-		$statement = $query->executeQuery();
-		$result = $statement->fetch();
-		$statement->closeCursor();
-
-		if (!isset($result['token'])) {
-			throw new \Exception('No token found for: ' . $url);
-		}
-
-		return $result['token'];
-	}
-
-	/**
-	 * Add shared Secret to database
-	 */
-	public function addSharedSecret(string $url, string $sharedSecret): void {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->update($this->dbTable)
-			->set('shared_secret', $query->createParameter('sharedSecret'))
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash)
-			->setParameter('sharedSecret', $sharedSecret);
-		$query->executeStatement();
-	}
-
-	/**
-	 * Get shared secret from database
-	 */
-	public function getSharedSecret(string $url): string {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->select('shared_secret')->from($this->dbTable)
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash);
-
-		$statement = $query->executeQuery();
-		$result = $statement->fetch();
-		$statement->closeCursor();
-		return (string)$result['shared_secret'];
-	}
-
-	/**
-	 * Set server status
-	 */
-	public function setServerStatus(string $url, int $status, ?string $token = null): void {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->update($this->dbTable)
-			->set('status', $query->createNamedParameter($status))
-			->where($query->expr()->eq('url_hash', $query->createNamedParameter($hash)));
-		if (!is_null($token)) {
-			$query->set('sync_token', $query->createNamedParameter($token));
-		}
-		$query->executeStatement();
-	}
-
-	/**
-	 * Get server status
-	 */
-	public function getServerStatus(string $url): int {
-		$hash = $this->hash($url);
-		$query = $this->connection->getQueryBuilder();
-		$query->select('status')->from($this->dbTable)
-			->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
-			->setParameter('url_hash', $hash);
-
-		$statement = $query->executeQuery();
-		$result = $statement->fetch();
-		$statement->closeCursor();
-		return (int)$result['status'];
-	}
-
-	/**
-	 * Create hash from URL
-	 */
-	protected function hash(string $url): string {
-		$normalized = $this->normalizeUrl($url);
-		return sha1($normalized);
-	}
-
-	/**
-	 * Normalize URL, used to create the sha1 hash
-	 */
-	protected function normalizeUrl(string $url): string {
-		$normalized = $url;
-
-		if (strpos($url, 'https://') === 0) {
-			$normalized = substr($url, strlen('https://'));
-		} elseif (strpos($url, 'http://') === 0) {
-			$normalized = substr($url, strlen('http://'));
-		}
-
-		$normalized = Filesystem::normalizePath($normalized);
-		$normalized = trim($normalized, '/');
-
-		return $normalized;
-	}
-
-	public function auth(string $username, string $password): bool {
-		if ($username !== 'system') {
-			return false;
-		}
-		$query = $this->connection->getQueryBuilder();
-		$query->select('url')->from($this->dbTable)
-			->where($query->expr()->eq('shared_secret', $query->createNamedParameter($password)));
-
-		$statement = $query->executeQuery();
-		$result = $statement->fetch();
-		$statement->closeCursor();
-		return !empty($result);
-	}
+    private string $dbTable = 'trusted_servers';
+
+    public function __construct(
+        private IDBConnection $connection,
+        private IL10N $IL10N,
+    ) {
+    }
+
+    /**
+     * Add server to the list of trusted servers
+     *
+     * @throws HintException
+     */
+    public function addServer(string $url): int {
+        $hash = $this->hash($url);
+        $url = rtrim($url, '/');
+        $query = $this->connection->getQueryBuilder();
+        $query->insert($this->dbTable)
+            ->values([
+                'url' => $query->createParameter('url'),
+                'url_hash' => $query->createParameter('url_hash'),
+            ])
+            ->setParameter('url', $url)
+            ->setParameter('url_hash', $hash);
+
+        $result = $query->executeStatement();
+
+        if ($result) {
+            return $query->getLastInsertId();
+        }
+
+        $message = 'Internal failure, Could not add trusted server: ' . $url;
+        $message_t = $this->IL10N->t('Could not add server');
+        throw new HintException($message, $message_t);
+        return -1;
+    }
+
+    /**
+     * Remove server from the list of trusted servers
+     */
+    public function removeServer(int $id): void {
+        $query = $this->connection->getQueryBuilder();
+        $query->delete($this->dbTable)
+            ->where($query->expr()->eq('id', $query->createParameter('id')))
+            ->setParameter('id', $id);
+        $query->executeStatement();
+    }
+
+    /**
+     * Get trusted server with given ID
+     *
+     * @return array{id: int, url: string, url_hash: string, token: ?string, shared_secret: ?string, status: int, sync_token: ?string}
+     * @throws \Exception
+     */
+    public function getServerById(int $id): array {
+        $query = $this->connection->getQueryBuilder();
+        $query->select('*')->from($this->dbTable)
+            ->where($query->expr()->eq('id', $query->createParameter('id')))
+            ->setParameter('id', $id, IQueryBuilder::PARAM_INT);
+
+        $qResult = $query->executeQuery();
+        $result = $qResult->fetchAll();
+        $qResult->closeCursor();
+
+        if (empty($result)) {
+            throw new \Exception('No Server found with ID: ' . $id);
+        }
+
+        return $result[0];
+    }
+
+    /**
+     * Get all trusted servers
+     *
+     * @return list<array{id: int, url: string, url_hash: string, shared_secret: ?string, status: int, sync_token: ?string}>
+     * @throws DBException
+     */
+    public function getAllServer(): array {
+        $query = $this->connection->getQueryBuilder();
+        $query->select(['url', 'url_hash', 'id', 'status', 'shared_secret', 'sync_token'])
+            ->from($this->dbTable);
+        $statement = $query->executeQuery();
+        $result = $statement->fetchAll();
+        $statement->closeCursor();
+        return $result;
+    }
+
+    /**
+     * Check if server already exists in the database table
+     */
+    public function serverExists(string $url): bool {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->select('url')
+            ->from($this->dbTable)
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash);
+        $statement = $query->executeQuery();
+        $result = $statement->fetchAll();
+        $statement->closeCursor();
+
+        return !empty($result);
+    }
+
+    /**
+     * Write token to database. Token is used to exchange the secret
+     */
+    public function addToken(string $url, string $token): void {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->update($this->dbTable)
+            ->set('token', $query->createParameter('token'))
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash)
+            ->setParameter('token', $token);
+        $query->executeStatement();
+    }
+
+    /**
+     * Get token stored in database
+     * @throws \Exception
+     */
+    public function getToken(string $url): string {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->select('token')->from($this->dbTable)
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash);
+
+        $statement = $query->executeQuery();
+        $result = $statement->fetch();
+        $statement->closeCursor();
+
+        if (!isset($result['token'])) {
+            throw new \Exception('No token found for: ' . $url);
+        }
+
+        return $result['token'];
+    }
+
+    /**
+     * Add shared Secret to database
+     */
+    public function addSharedSecret(string $url, string $sharedSecret): void {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->update($this->dbTable)
+            ->set('shared_secret', $query->createParameter('sharedSecret'))
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash)
+            ->setParameter('sharedSecret', $sharedSecret);
+        $query->executeStatement();
+    }
+
+    /**
+     * Get shared secret from database
+     */
+    public function getSharedSecret(string $url): string {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->select('shared_secret')->from($this->dbTable)
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash);
+
+        $statement = $query->executeQuery();
+        $result = $statement->fetch();
+        $statement->closeCursor();
+        return (string)$result['shared_secret'];
+    }
+
+    /**
+     * Set server status
+     */
+    public function setServerStatus(string $url, int $status, ?string $token = null): void {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->update($this->dbTable)
+            ->set('status', $query->createNamedParameter($status))
+            ->where($query->expr()->eq('url_hash', $query->createNamedParameter($hash)));
+        if (!is_null($token)) {
+            $query->set('sync_token', $query->createNamedParameter($token));
+        }
+        $query->executeStatement();
+    }
+
+    /**
+     * Get server status
+     */
+    public function getServerStatus(string $url): int {
+        $hash = $this->hash($url);
+        $query = $this->connection->getQueryBuilder();
+        $query->select('status')->from($this->dbTable)
+            ->where($query->expr()->eq('url_hash', $query->createParameter('url_hash')))
+            ->setParameter('url_hash', $hash);
+
+        $statement = $query->executeQuery();
+        $result = $statement->fetch();
+        $statement->closeCursor();
+        return (int)$result['status'];
+    }
+
+    /**
+     * Create hash from URL
+     */
+    protected function hash(string $url): string {
+        $normalized = $this->normalizeUrl($url);
+        return sha1($normalized);
+    }
+
+    /**
+     * Normalize URL, used to create the sha1 hash
+     */
+    protected function normalizeUrl(string $url): string {
+        $normalized = $url;
+
+        if (strpos($url, 'https://') === 0) {
+            $normalized = substr($url, strlen('https://'));
+        } elseif (strpos($url, 'http://') === 0) {
+            $normalized = substr($url, strlen('http://'));
+        }
+
+        $normalized = Filesystem::normalizePath($normalized);
+        $normalized = trim($normalized, '/');
+
+        return $normalized;
+    }
+
+    public function auth(string $username, string $password): bool {
+        if ($username !== 'system') {
+            return false;
+        }
+        $query = $this->connection->getQueryBuilder();
+        $query->select('url')->from($this->dbTable)
+            ->where($query->expr()->eq('shared_secret', $query->createNamedParameter($password)));
+
+        $statement = $query->executeQuery();
+        $result = $statement->fetch();
+        $statement->closeCursor();
+        return !empty($result);
+    }
 }

apps/files_sharing/lib/ExpireSharesJob.php

Spacing +3 added lines, -3 removed lines

@@ -87,13 +87,13 @@
 
 		$shares = $qb->executeQuery();
 		while ($share = $shares->fetch()) {
-			if ((int)$share['share_type'] === IShare::TYPE_LINK) {
+			if ((int) $share['share_type'] === IShare::TYPE_LINK) {
 				$id = 'ocinternal';
-			} elseif ((int)$share['share_type'] === IShare::TYPE_EMAIL) {
+			} elseif ((int) $share['share_type'] === IShare::TYPE_EMAIL) {
 				$id = 'ocMailShare';
 			}
 
-			$id .= ':' . $share['id'];
+			$id .= ':'.$share['id'];
 
 			try {
 				$share = $this->shareManager->getShareById($id);

Indentation +47 added lines, -47 removed lines

@@ -19,60 +19,60 @@
  */
 class ExpireSharesJob extends TimedJob {
 
-	public function __construct(
-		ITimeFactory $time,
-		private IManager $shareManager,
-		private IDBConnection $db,
-	) {
-		parent::__construct($time);
+    public function __construct(
+        ITimeFactory $time,
+        private IManager $shareManager,
+        private IDBConnection $db,
+    ) {
+        parent::__construct($time);
 
-		// Run once a day
-		$this->setInterval(24 * 60 * 60);
-		$this->setTimeSensitivity(self::TIME_INSENSITIVE);
-	}
+        // Run once a day
+        $this->setInterval(24 * 60 * 60);
+        $this->setTimeSensitivity(self::TIME_INSENSITIVE);
+    }
 
 
-	/**
-	 * Makes the background job do its work
-	 *
-	 * @param array $argument unused argument
-	 */
-	public function run($argument) {
-		//Current time
-		$now = new \DateTime();
-		$now = $now->format('Y-m-d H:i:s');
+    /**
+     * Makes the background job do its work
+     *
+     * @param array $argument unused argument
+     */
+    public function run($argument) {
+        //Current time
+        $now = new \DateTime();
+        $now = $now->format('Y-m-d H:i:s');
 
-		/*
+        /*
 		 * Expire file link shares only (for now)
 		 */
-		$qb = $this->db->getQueryBuilder();
-		$qb->select('id', 'share_type')
-			->from('share')
-			->where(
-				$qb->expr()->andX(
-					$qb->expr()->in('share_type', $qb->createNamedParameter([IShare::TYPE_LINK, IShare::TYPE_EMAIL], IQueryBuilder::PARAM_INT_ARRAY)),
-					$qb->expr()->lte('expiration', $qb->expr()->literal($now)),
-					$qb->expr()->in('item_type', $qb->createNamedParameter(['file', 'folder'], IQueryBuilder::PARAM_STR_ARRAY))
-				)
-			);
+        $qb = $this->db->getQueryBuilder();
+        $qb->select('id', 'share_type')
+            ->from('share')
+            ->where(
+                $qb->expr()->andX(
+                    $qb->expr()->in('share_type', $qb->createNamedParameter([IShare::TYPE_LINK, IShare::TYPE_EMAIL], IQueryBuilder::PARAM_INT_ARRAY)),
+                    $qb->expr()->lte('expiration', $qb->expr()->literal($now)),
+                    $qb->expr()->in('item_type', $qb->createNamedParameter(['file', 'folder'], IQueryBuilder::PARAM_STR_ARRAY))
+                )
+            );
 
-		$shares = $qb->executeQuery();
-		while ($share = $shares->fetch()) {
-			if ((int)$share['share_type'] === IShare::TYPE_LINK) {
-				$id = 'ocinternal';
-			} elseif ((int)$share['share_type'] === IShare::TYPE_EMAIL) {
-				$id = 'ocMailShare';
-			}
+        $shares = $qb->executeQuery();
+        while ($share = $shares->fetch()) {
+            if ((int)$share['share_type'] === IShare::TYPE_LINK) {
+                $id = 'ocinternal';
+            } elseif ((int)$share['share_type'] === IShare::TYPE_EMAIL) {
+                $id = 'ocMailShare';
+            }
 
-			$id .= ':' . $share['id'];
+            $id .= ':' . $share['id'];
 
-			try {
-				$share = $this->shareManager->getShareById($id);
-				$this->shareManager->deleteShare($share);
-			} catch (ShareNotFound $e) {
-				// Normally the share gets automatically expired on fetching it
-			}
-		}
-		$shares->closeCursor();
-	}
+            try {
+                $share = $this->shareManager->getShareById($id);
+                $this->shareManager->deleteShare($share);
+            } catch (ShareNotFound $e) {
+                // Normally the share gets automatically expired on fetching it
+            }
+        }
+        $shares->closeCursor();
+    }
 }

apps/files/templates/list.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 	<?php if (isset($_['dirToken'])):?>
 	<input type="hidden" id="publicUploadRequestToken" name="requesttoken" value="<?php p($_['requesttoken']) ?>" />
 	<input type="hidden" id="dirToken" name="dirToken" value="<?php p($_['dirToken']) ?>" />
-	<?php endif;?>
+	<?php endif; ?>
 	<input type="hidden" class="max_human_file_size"
 		   value="(max <?php isset($_['uploadMaxHumanFilesize']) ? p($_['uploadMaxHumanFilesize']) : ''; ?>)">
 </div>
@@ -75,6 +75,6 @@ 
 </div>
 <div id="uploadsize-message" title="<?php p($l->t('Upload too large'))?>">
 	<p>
-	<?php p($l->t('The files you are trying to upload exceed the maximum size for file uploads on this server.'));?>
+	<?php p($l->t('The files you are trying to upload exceed the maximum size for file uploads on this server.')); ?>
 	</p>
 </div>

apps/dav/templates/schedule-response-success.php

Spacing +1 added lines, -1 removed lines

@@ -5,5 +5,5 @@
 ?>
 <div class="guest-box">
 	<div class="icon icon-checkmark"></div>
-	<p class="message"><?php p($l->t('Your attendance was updated successfully.'));?></p>
+	<p class="message"><?php p($l->t('Your attendance was updated successfully.')); ?></p>
 </div>

apps/dav/templates/schedule-response-error.php

Spacing +2 added lines, -2 removed lines

@@ -6,8 +6,8 @@
 ?>
 <div class="guest-box">
 	<div class="notecard error">
-		<p><?php p($l->t('There was an error updating your attendance status.'));?></p>
-		<p><?php p($l->t('Please contact the organizer directly.'));?></p>
+		<p><?php p($l->t('There was an error updating your attendance status.')); ?></p>
+		<p><?php p($l->t('Please contact the organizer directly.')); ?></p>
 		<?php if (isset($_['organizer'])): ?>
 			<p><a href="<?php p($_['organizer']) ?>"><?php p(substr($_['organizer'], 7)) ?></a></p>
 		<?php endif; ?>

apps/dav/lib/Migration/Version1024Date20211221144219.php

Indentation +38 added lines, -38 removed lines

@@ -17,42 +17,42 @@
  */
 class Version1024Date20211221144219 extends SimpleMigrationStep {
 
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 */
-	public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
-	}
-
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 * @return null|ISchemaWrapper
-	 * @throws SchemaException
-	 */
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
-		/** @var ISchemaWrapper $schema */
-		$schema = $schemaClosure();
-		$propertiesTable = $schema->getTable('properties');
-
-		if ($propertiesTable->hasColumn('valuetype')) {
-			return null;
-		}
-		$propertiesTable->addColumn('valuetype', Types::SMALLINT, [
-			'notnull' => false,
-			'default' => CustomPropertiesBackend::PROPERTY_TYPE_STRING
-		]);
-
-		return $schema;
-	}
-
-	/**
-	 * @param IOutput $output
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-	 * @param array $options
-	 */
-	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
-	}
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     */
+    public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
+    }
+
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     * @return null|ISchemaWrapper
+     * @throws SchemaException
+     */
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
+        /** @var ISchemaWrapper $schema */
+        $schema = $schemaClosure();
+        $propertiesTable = $schema->getTable('properties');
+
+        if ($propertiesTable->hasColumn('valuetype')) {
+            return null;
+        }
+        $propertiesTable->addColumn('valuetype', Types::SMALLINT, [
+            'notnull' => false,
+            'default' => CustomPropertiesBackend::PROPERTY_TYPE_STRING
+        ]);
+
+        return $schema;
+    }
+
+    /**
+     * @param IOutput $output
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
+     * @param array $options
+     */
+    public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
+    }
 }

apps/dav/lib/CalDAV/PublicCalendar.php

Indentation +54 added lines, -54 removed lines

@@ -28,63 +28,63 @@
 
 class PublicCalendar extends Calendar {
 
-	/**
-	 * @param string $name
-	 * @throws NotFound
-	 * @return PublicCalendarObject
-	 */
-	public function getChild($name) {
-		$obj = $this->caldavBackend->getCalendarObject($this->calendarInfo['id'], $name);
+    /**
+     * @param string $name
+     * @throws NotFound
+     * @return PublicCalendarObject
+     */
+    public function getChild($name) {
+        $obj = $this->caldavBackend->getCalendarObject($this->calendarInfo['id'], $name);
 
-		if (!$obj) {
-			throw new NotFound('Calendar object not found');
-		}
-		if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
-			throw new NotFound('Calendar object not found');
-		}
-		$obj['acl'] = $this->getChildACL();
+        if (!$obj) {
+            throw new NotFound('Calendar object not found');
+        }
+        if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
+            throw new NotFound('Calendar object not found');
+        }
+        $obj['acl'] = $this->getChildACL();
 
-		return new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
-	}
+        return new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
+    }
 
-	/**
-	 * @return PublicCalendarObject[]
-	 */
-	public function getChildren() {
-		$objs = $this->caldavBackend->getCalendarObjects($this->calendarInfo['id']);
-		$children = [];
-		foreach ($objs as $obj) {
-			if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
-				continue;
-			}
-			$obj['acl'] = $this->getChildACL();
-			$children[] = new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
-		}
-		return $children;
-	}
+    /**
+     * @return PublicCalendarObject[]
+     */
+    public function getChildren() {
+        $objs = $this->caldavBackend->getCalendarObjects($this->calendarInfo['id']);
+        $children = [];
+        foreach ($objs as $obj) {
+            if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
+                continue;
+            }
+            $obj['acl'] = $this->getChildACL();
+            $children[] = new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
+        }
+        return $children;
+    }
 
-	/**
-	 * @param string[] $paths
-	 * @return PublicCalendarObject[]
-	 */
-	public function getMultipleChildren(array $paths) {
-		$objs = $this->caldavBackend->getMultipleCalendarObjects($this->calendarInfo['id'], $paths);
-		$children = [];
-		foreach ($objs as $obj) {
-			if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
-				continue;
-			}
-			$obj['acl'] = $this->getChildACL();
-			$children[] = new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
-		}
-		return $children;
-	}
+    /**
+     * @param string[] $paths
+     * @return PublicCalendarObject[]
+     */
+    public function getMultipleChildren(array $paths) {
+        $objs = $this->caldavBackend->getMultipleCalendarObjects($this->calendarInfo['id'], $paths);
+        $children = [];
+        foreach ($objs as $obj) {
+            if ($obj['classification'] === CalDavBackend::CLASSIFICATION_PRIVATE) {
+                continue;
+            }
+            $obj['acl'] = $this->getChildACL();
+            $children[] = new PublicCalendarObject($this->caldavBackend, $this->l10n, $this->calendarInfo, $obj);
+        }
+        return $children;
+    }
 
-	/**
-	 * public calendars are always shared
-	 * @return bool
-	 */
-	public function isShared() {
-		return true;
-	}
+    /**
+     * public calendars are always shared
+     * @return bool
+     */
+    public function isShared() {
+        return true;
+    }
 }

apps/dav/lib/CalDAV/Reminder/Notifier.php

Spacing +7 added lines, -7 removed lines

@@ -263,10 +263,10 @@ 
 			]);
 
 			return $localeStart
-				. ' (' . $startTimezone . ') '
+				. ' ('.$startTimezone.') '
 				. ' - '
 				. $localeEnd
-				. ' (' . $endTimezone . ')';
+				. ' ('.$endTimezone.')';
 		}
 
 		// Show only the time if the day is the same
@@ -280,7 +280,7 @@ 
 		return $localeStart
 			. ' - '
 			. $localeEnd
-			. ' (' . $startTimezone . ')';
+			. ' ('.$startTimezone.')';
 	}
 
 	/**
@@ -298,7 +298,7 @@ 
 	 * @return string
 	 */
 	private function getWeekDayName(DateTime $dt):string {
-		return (string)$this->l10n->l('weekdayName', $dt, ['width' => 'abbreviated']);
+		return (string) $this->l10n->l('weekdayName', $dt, ['width' => 'abbreviated']);
 	}
 
 	/**
@@ -306,7 +306,7 @@ 
 	 * @return string
 	 */
 	private function getDateString(DateTime $dt):string {
-		return (string)$this->l10n->l('date', $dt, ['width' => 'medium']);
+		return (string) $this->l10n->l('date', $dt, ['width' => 'medium']);
 	}
 
 	/**
@@ -314,7 +314,7 @@ 
 	 * @return string
 	 */
 	private function getDateTimeString(DateTime $dt):string {
-		return (string)$this->l10n->l('datetime', $dt, ['width' => 'medium|short']);
+		return (string) $this->l10n->l('datetime', $dt, ['width' => 'medium|short']);
 	}
 
 	/**
@@ -322,6 +322,6 @@ 
 	 * @return string
 	 */
 	private function getTimeString(DateTime $dt):string {
-		return (string)$this->l10n->l('time', $dt, ['width' => 'short']);
+		return (string) $this->l10n->l('time', $dt, ['width' => 'short']);
 	}
 }

Indentation +282 added lines, -282 removed lines

@@ -26,286 +26,286 @@
  */
 class Notifier implements INotifier {
 
-	/** @var IL10N */
-	private $l10n;
-
-	/**
-	 * Notifier constructor.
-	 *
-	 * @param IFactory $l10nFactory
-	 * @param IURLGenerator $urlGenerator
-	 * @param ITimeFactory $timeFactory
-	 */
-	public function __construct(
-		private IFactory $l10nFactory,
-		private IURLGenerator $urlGenerator,
-		private ITimeFactory $timeFactory,
-	) {
-	}
-
-	/**
-	 * Identifier of the notifier, only use [a-z0-9_]
-	 *
-	 * @return string
-	 * @since 17.0.0
-	 */
-	public function getID():string {
-		return Application::APP_ID;
-	}
-
-	/**
-	 * Human readable name describing the notifier
-	 *
-	 * @return string
-	 * @since 17.0.0
-	 */
-	public function getName():string {
-		return $this->l10nFactory->get('dav')->t('Calendar');
-	}
-
-	/**
-	 * Prepare sending the notification
-	 *
-	 * @param INotification $notification
-	 * @param string $languageCode The code of the language that should be used to prepare the notification
-	 * @return INotification
-	 * @throws UnknownNotificationException
-	 */
-	public function prepare(INotification $notification,
-		string $languageCode):INotification {
-		if ($notification->getApp() !== Application::APP_ID) {
-			throw new UnknownNotificationException('Notification not from this app');
-		}
-
-		// Read the language from the notification
-		$this->l10n = $this->l10nFactory->get('dav', $languageCode);
-
-		// Handle notifier subjects
-		switch ($notification->getSubject()) {
-			case 'calendar_reminder':
-				return $this->prepareReminderNotification($notification);
-
-			default:
-				throw new UnknownNotificationException('Unknown subject');
-
-		}
-	}
-
-	/**
-	 * @param INotification $notification
-	 * @return INotification
-	 */
-	private function prepareReminderNotification(INotification $notification):INotification {
-		$imagePath = $this->urlGenerator->imagePath('core', 'places/calendar.svg');
-		$iconUrl = $this->urlGenerator->getAbsoluteURL($imagePath);
-		$notification->setIcon($iconUrl);
-
-		$this->prepareNotificationSubject($notification);
-		$this->prepareNotificationMessage($notification);
-
-		return $notification;
-	}
-
-	/**
-	 * Sets the notification subject based on the parameters set in PushProvider
-	 *
-	 * @param INotification $notification
-	 */
-	private function prepareNotificationSubject(INotification $notification): void {
-		$parameters = $notification->getSubjectParameters();
-
-		$startTime = \DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['start_atom']);
-		$now = $this->timeFactory->getDateTime();
-		$title = $this->getTitleFromParameters($parameters);
-
-		$diff = $startTime->diff($now);
-		if ($diff === false) {
-			return;
-		}
-
-		$components = [];
-		if ($diff->y) {
-			$components[] = $this->l10n->n('%n year', '%n years', $diff->y);
-		}
-		if ($diff->m) {
-			$components[] = $this->l10n->n('%n month', '%n months', $diff->m);
-		}
-		if ($diff->d) {
-			$components[] = $this->l10n->n('%n day', '%n days', $diff->d);
-		}
-		if ($diff->h) {
-			$components[] = $this->l10n->n('%n hour', '%n hours', $diff->h);
-		}
-		if ($diff->i) {
-			$components[] = $this->l10n->n('%n minute', '%n minutes', $diff->i);
-		}
-
-		if (count($components) > 0 && !$this->hasPhpDatetimeDiffBug()) {
-			// Limiting to the first three components to prevent
-			// the string from getting too long
-			$firstThreeComponents = array_slice($components, 0, 2);
-			$diffLabel = implode(', ', $firstThreeComponents);
-
-			if ($diff->invert) {
-				$title = $this->l10n->t('%s (in %s)', [$title, $diffLabel]);
-			} else {
-				$title = $this->l10n->t('%s (%s ago)', [$title, $diffLabel]);
-			}
-		}
-
-		$notification->setParsedSubject($title);
-	}
-
-	/**
-	 * @see https://github.com/nextcloud/server/issues/41615
-	 * @see https://github.com/php/php-src/issues/9699
-	 */
-	private function hasPhpDatetimeDiffBug(): bool {
-		$d1 = DateTime::createFromFormat(\DateTimeInterface::ATOM, '2023-11-22T11:52:00+01:00');
-		$d2 = new DateTime('2023-11-22T10:52:03', new \DateTimeZone('UTC'));
-
-		// The difference is 3 seconds, not -1year+11months+…
-		return $d1->diff($d2)->y < 0;
-	}
-
-	/**
-	 * Sets the notification message based on the parameters set in PushProvider
-	 *
-	 * @param INotification $notification
-	 */
-	private function prepareNotificationMessage(INotification $notification): void {
-		$parameters = $notification->getMessageParameters();
-
-		$description = [
-			$this->l10n->t('Calendar: %s', $parameters['calendar_displayname']),
-			$this->l10n->t('Date: %s', $this->generateDateString($parameters)),
-		];
-		if ($parameters['description']) {
-			$description[] = $this->l10n->t('Description: %s', $parameters['description']);
-		}
-		if ($parameters['location']) {
-			$description[] = $this->l10n->t('Where: %s', $parameters['location']);
-		}
-
-		$message = implode("\r\n", $description);
-		$notification->setParsedMessage($message);
-	}
-
-	/**
-	 * @param array $parameters
-	 * @return string
-	 */
-	private function getTitleFromParameters(array $parameters):string {
-		return $parameters['title'] ?? $this->l10n->t('Untitled event');
-	}
-
-	/**
-	 * @param array $parameters
-	 * @return string
-	 * @throws \Exception
-	 */
-	private function generateDateString(array $parameters):string {
-		$startDateTime = DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['start_atom']);
-		$endDateTime = DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['end_atom']);
-
-		// If the event has already ended, dismiss the notification
-		if ($endDateTime < $this->timeFactory->getDateTime()) {
-			throw new AlreadyProcessedException();
-		}
-
-		$isAllDay = $parameters['all_day'];
-		$diff = $startDateTime->diff($endDateTime);
-
-		if ($isAllDay) {
-			// One day event
-			if ($diff->days === 1) {
-				return $this->getDateString($startDateTime);
-			}
-
-			return implode(' - ', [
-				$this->getDateString($startDateTime),
-				$this->getDateString($endDateTime),
-			]);
-		}
-
-		$startTimezone = $endTimezone = null;
-		if (!$parameters['start_is_floating']) {
-			$startTimezone = $parameters['start_timezone'];
-			$endTimezone = $parameters['end_timezone'];
-		}
-
-		$localeStart = implode(', ', [
-			$this->getWeekDayName($startDateTime),
-			$this->getDateTimeString($startDateTime)
-		]);
-
-		// always show full date with timezone if timezones are different
-		if ($startTimezone !== $endTimezone) {
-			$localeEnd = implode(', ', [
-				$this->getWeekDayName($endDateTime),
-				$this->getDateTimeString($endDateTime)
-			]);
-
-			return $localeStart
-				. ' (' . $startTimezone . ') '
-				. ' - '
-				. $localeEnd
-				. ' (' . $endTimezone . ')';
-		}
-
-		// Show only the time if the day is the same
-		$localeEnd = $this->isDayEqual($startDateTime, $endDateTime)
-			? $this->getTimeString($endDateTime)
-			: implode(', ', [
-				$this->getWeekDayName($endDateTime),
-				$this->getDateTimeString($endDateTime)
-			]);
-
-		return $localeStart
-			. ' - '
-			. $localeEnd
-			. ' (' . $startTimezone . ')';
-	}
-
-	/**
-	 * @param DateTime $dtStart
-	 * @param DateTime $dtEnd
-	 * @return bool
-	 */
-	private function isDayEqual(DateTime $dtStart,
-		DateTime $dtEnd):bool {
-		return $dtStart->format('Y-m-d') === $dtEnd->format('Y-m-d');
-	}
-
-	/**
-	 * @param DateTime $dt
-	 * @return string
-	 */
-	private function getWeekDayName(DateTime $dt):string {
-		return (string)$this->l10n->l('weekdayName', $dt, ['width' => 'abbreviated']);
-	}
-
-	/**
-	 * @param DateTime $dt
-	 * @return string
-	 */
-	private function getDateString(DateTime $dt):string {
-		return (string)$this->l10n->l('date', $dt, ['width' => 'medium']);
-	}
-
-	/**
-	 * @param DateTime $dt
-	 * @return string
-	 */
-	private function getDateTimeString(DateTime $dt):string {
-		return (string)$this->l10n->l('datetime', $dt, ['width' => 'medium|short']);
-	}
-
-	/**
-	 * @param DateTime $dt
-	 * @return string
-	 */
-	private function getTimeString(DateTime $dt):string {
-		return (string)$this->l10n->l('time', $dt, ['width' => 'short']);
-	}
+    /** @var IL10N */
+    private $l10n;
+
+    /**
+     * Notifier constructor.
+     *
+     * @param IFactory $l10nFactory
+     * @param IURLGenerator $urlGenerator
+     * @param ITimeFactory $timeFactory
+     */
+    public function __construct(
+        private IFactory $l10nFactory,
+        private IURLGenerator $urlGenerator,
+        private ITimeFactory $timeFactory,
+    ) {
+    }
+
+    /**
+     * Identifier of the notifier, only use [a-z0-9_]
+     *
+     * @return string
+     * @since 17.0.0
+     */
+    public function getID():string {
+        return Application::APP_ID;
+    }
+
+    /**
+     * Human readable name describing the notifier
+     *
+     * @return string
+     * @since 17.0.0
+     */
+    public function getName():string {
+        return $this->l10nFactory->get('dav')->t('Calendar');
+    }
+
+    /**
+     * Prepare sending the notification
+     *
+     * @param INotification $notification
+     * @param string $languageCode The code of the language that should be used to prepare the notification
+     * @return INotification
+     * @throws UnknownNotificationException
+     */
+    public function prepare(INotification $notification,
+        string $languageCode):INotification {
+        if ($notification->getApp() !== Application::APP_ID) {
+            throw new UnknownNotificationException('Notification not from this app');
+        }
+
+        // Read the language from the notification
+        $this->l10n = $this->l10nFactory->get('dav', $languageCode);
+
+        // Handle notifier subjects
+        switch ($notification->getSubject()) {
+            case 'calendar_reminder':
+                return $this->prepareReminderNotification($notification);
+
+            default:
+                throw new UnknownNotificationException('Unknown subject');
+
+        }
+    }
+
+    /**
+     * @param INotification $notification
+     * @return INotification
+     */
+    private function prepareReminderNotification(INotification $notification):INotification {
+        $imagePath = $this->urlGenerator->imagePath('core', 'places/calendar.svg');
+        $iconUrl = $this->urlGenerator->getAbsoluteURL($imagePath);
+        $notification->setIcon($iconUrl);
+
+        $this->prepareNotificationSubject($notification);
+        $this->prepareNotificationMessage($notification);
+
+        return $notification;
+    }
+
+    /**
+     * Sets the notification subject based on the parameters set in PushProvider
+     *
+     * @param INotification $notification
+     */
+    private function prepareNotificationSubject(INotification $notification): void {
+        $parameters = $notification->getSubjectParameters();
+
+        $startTime = \DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['start_atom']);
+        $now = $this->timeFactory->getDateTime();
+        $title = $this->getTitleFromParameters($parameters);
+
+        $diff = $startTime->diff($now);
+        if ($diff === false) {
+            return;
+        }
+
+        $components = [];
+        if ($diff->y) {
+            $components[] = $this->l10n->n('%n year', '%n years', $diff->y);
+        }
+        if ($diff->m) {
+            $components[] = $this->l10n->n('%n month', '%n months', $diff->m);
+        }
+        if ($diff->d) {
+            $components[] = $this->l10n->n('%n day', '%n days', $diff->d);
+        }
+        if ($diff->h) {
+            $components[] = $this->l10n->n('%n hour', '%n hours', $diff->h);
+        }
+        if ($diff->i) {
+            $components[] = $this->l10n->n('%n minute', '%n minutes', $diff->i);
+        }
+
+        if (count($components) > 0 && !$this->hasPhpDatetimeDiffBug()) {
+            // Limiting to the first three components to prevent
+            // the string from getting too long
+            $firstThreeComponents = array_slice($components, 0, 2);
+            $diffLabel = implode(', ', $firstThreeComponents);
+
+            if ($diff->invert) {
+                $title = $this->l10n->t('%s (in %s)', [$title, $diffLabel]);
+            } else {
+                $title = $this->l10n->t('%s (%s ago)', [$title, $diffLabel]);
+            }
+        }
+
+        $notification->setParsedSubject($title);
+    }
+
+    /**
+     * @see https://github.com/nextcloud/server/issues/41615
+     * @see https://github.com/php/php-src/issues/9699
+     */
+    private function hasPhpDatetimeDiffBug(): bool {
+        $d1 = DateTime::createFromFormat(\DateTimeInterface::ATOM, '2023-11-22T11:52:00+01:00');
+        $d2 = new DateTime('2023-11-22T10:52:03', new \DateTimeZone('UTC'));
+
+        // The difference is 3 seconds, not -1year+11months+…
+        return $d1->diff($d2)->y < 0;
+    }
+
+    /**
+     * Sets the notification message based on the parameters set in PushProvider
+     *
+     * @param INotification $notification
+     */
+    private function prepareNotificationMessage(INotification $notification): void {
+        $parameters = $notification->getMessageParameters();
+
+        $description = [
+            $this->l10n->t('Calendar: %s', $parameters['calendar_displayname']),
+            $this->l10n->t('Date: %s', $this->generateDateString($parameters)),
+        ];
+        if ($parameters['description']) {
+            $description[] = $this->l10n->t('Description: %s', $parameters['description']);
+        }
+        if ($parameters['location']) {
+            $description[] = $this->l10n->t('Where: %s', $parameters['location']);
+        }
+
+        $message = implode("\r\n", $description);
+        $notification->setParsedMessage($message);
+    }
+
+    /**
+     * @param array $parameters
+     * @return string
+     */
+    private function getTitleFromParameters(array $parameters):string {
+        return $parameters['title'] ?? $this->l10n->t('Untitled event');
+    }
+
+    /**
+     * @param array $parameters
+     * @return string
+     * @throws \Exception
+     */
+    private function generateDateString(array $parameters):string {
+        $startDateTime = DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['start_atom']);
+        $endDateTime = DateTime::createFromFormat(\DateTimeInterface::ATOM, $parameters['end_atom']);
+
+        // If the event has already ended, dismiss the notification
+        if ($endDateTime < $this->timeFactory->getDateTime()) {
+            throw new AlreadyProcessedException();
+        }
+
+        $isAllDay = $parameters['all_day'];
+        $diff = $startDateTime->diff($endDateTime);
+
+        if ($isAllDay) {
+            // One day event
+            if ($diff->days === 1) {
+                return $this->getDateString($startDateTime);
+            }
+
+            return implode(' - ', [
+                $this->getDateString($startDateTime),
+                $this->getDateString($endDateTime),
+            ]);
+        }
+
+        $startTimezone = $endTimezone = null;
+        if (!$parameters['start_is_floating']) {
+            $startTimezone = $parameters['start_timezone'];
+            $endTimezone = $parameters['end_timezone'];
+        }
+
+        $localeStart = implode(', ', [
+            $this->getWeekDayName($startDateTime),
+            $this->getDateTimeString($startDateTime)
+        ]);
+
+        // always show full date with timezone if timezones are different
+        if ($startTimezone !== $endTimezone) {
+            $localeEnd = implode(', ', [
+                $this->getWeekDayName($endDateTime),
+                $this->getDateTimeString($endDateTime)
+            ]);
+
+            return $localeStart
+                . ' (' . $startTimezone . ') '
+                . ' - '
+                . $localeEnd
+                . ' (' . $endTimezone . ')';
+        }
+
+        // Show only the time if the day is the same
+        $localeEnd = $this->isDayEqual($startDateTime, $endDateTime)
+            ? $this->getTimeString($endDateTime)
+            : implode(', ', [
+                $this->getWeekDayName($endDateTime),
+                $this->getDateTimeString($endDateTime)
+            ]);
+
+        return $localeStart
+            . ' - '
+            . $localeEnd
+            . ' (' . $startTimezone . ')';
+    }
+
+    /**
+     * @param DateTime $dtStart
+     * @param DateTime $dtEnd
+     * @return bool
+     */
+    private function isDayEqual(DateTime $dtStart,
+        DateTime $dtEnd):bool {
+        return $dtStart->format('Y-m-d') === $dtEnd->format('Y-m-d');
+    }
+
+    /**
+     * @param DateTime $dt
+     * @return string
+     */
+    private function getWeekDayName(DateTime $dt):string {
+        return (string)$this->l10n->l('weekdayName', $dt, ['width' => 'abbreviated']);
+    }
+
+    /**
+     * @param DateTime $dt
+     * @return string
+     */
+    private function getDateString(DateTime $dt):string {
+        return (string)$this->l10n->l('date', $dt, ['width' => 'medium']);
+    }
+
+    /**
+     * @param DateTime $dt
+     * @return string
+     */
+    private function getDateTimeString(DateTime $dt):string {
+        return (string)$this->l10n->l('datetime', $dt, ['width' => 'medium|short']);
+    }
+
+    /**
+     * @param DateTime $dt
+     * @return string
+     */
+    private function getTimeString(DateTime $dt):string {
+        return (string)$this->l10n->l('time', $dt, ['width' => 'short']);
+    }
 }

apps/dav/lib/Listener/ActivityUpdaterListener.php

Spacing +13 added lines, -13 removed lines

@@ -70,7 +70,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar creation, so we just log it
-				$this->logger->error('Error generating activities for a new calendar: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activities for a new calendar: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -87,7 +87,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activities for changed calendar: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -103,7 +103,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activities for changed calendar: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -119,13 +119,13 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activities for changed calendar: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
 		} elseif ($event instanceof CalendarDeletedEvent) {
 			try {
-				$deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
+				$deletedProp = '{'.Plugin::NS_NEXTCLOUD.'}deleted-at';
 				if (isset($event->getCalendarData()[$deletedProp])) {
 					$this->logger->debug(
 						sprintf('Calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
@@ -142,7 +142,7 @@ 
 				}
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activities for a deleted calendar: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activities for a deleted calendar: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -160,7 +160,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar object creation, so we just log it
-				$this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a new calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -178,7 +178,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a deleted calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -197,7 +197,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a deleted calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -215,7 +215,7 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar object creation, so we just log it
-				$this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a new calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
@@ -233,13 +233,13 @@ 
 				);
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar object restoration, so we just log it
-				$this->logger->error('Error generating activity for a restored calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a restored calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}
 		} elseif ($event instanceof CalendarObjectDeletedEvent) {
 			try {
-				$deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
+				$deletedProp = '{'.Plugin::NS_NEXTCLOUD.'}deleted-at';
 				if (isset($event->getObjectData()[$deletedProp])) {
 					$this->logger->debug(
 						sprintf('Calendar object in calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
@@ -258,7 +258,7 @@ 
 				}
 			} catch (Throwable $e) {
 				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+				$this->logger->error('Error generating activity for a deleted calendar object: '.$e->getMessage(), [
 					'exception' => $e,
 				]);
 			}

Indentation +212 added lines, -212 removed lines

@@ -30,216 +30,216 @@
 /** @template-implements IEventListener<CalendarCreatedEvent|CalendarUpdatedEvent|CalendarMovedToTrashEvent|CalendarRestoredEvent|CalendarDeletedEvent|CalendarObjectCreatedEvent|CalendarObjectUpdatedEvent|CalendarObjectMovedEvent|CalendarObjectMovedToTrashEvent|CalendarObjectRestoredEvent|CalendarObjectDeletedEvent> */
 class ActivityUpdaterListener implements IEventListener {
 
-	public function __construct(
-		private ActivityBackend $activityBackend,
-		private LoggerInterface $logger,
-	) {
-	}
-
-	public function handle(Event $event): void {
-		if ($event instanceof CalendarCreatedEvent) {
-			try {
-				$this->activityBackend->onCalendarAdd(
-					$event->getCalendarData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for new calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar creation, so we just log it
-				$this->logger->error('Error generating activities for a new calendar: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarUpdatedEvent) {
-			try {
-				$this->activityBackend->onCalendarUpdate(
-					$event->getCalendarData(),
-					$event->getShares(),
-					$event->getMutations()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarMovedToTrashEvent) {
-			try {
-				$this->activityBackend->onCalendarMovedToTrash(
-					$event->getCalendarData(),
-					$event->getShares()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarRestoredEvent) {
-			try {
-				$this->activityBackend->onCalendarRestored(
-					$event->getCalendarData(),
-					$event->getShares()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar update, so we just log it
-				$this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarDeletedEvent) {
-			try {
-				$deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
-				if (isset($event->getCalendarData()[$deletedProp])) {
-					$this->logger->debug(
-						sprintf('Calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
-					);
-				} else {
-					$this->activityBackend->onCalendarDelete(
-						$event->getCalendarData(),
-						$event->getShares()
-					);
-
-					$this->logger->debug(
-						sprintf('Activity generated for deleted calendar %d', $event->getCalendarId())
-					);
-				}
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activities for a deleted calendar: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectCreatedEvent) {
-			try {
-				$this->activityBackend->onTouchCalendarObject(
-					\OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_ADD,
-					$event->getCalendarData(),
-					$event->getShares(),
-					$event->getObjectData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for new calendar object in calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar object creation, so we just log it
-				$this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectUpdatedEvent) {
-			try {
-				$this->activityBackend->onTouchCalendarObject(
-					\OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_UPDATE,
-					$event->getCalendarData(),
-					$event->getShares(),
-					$event->getObjectData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for updated calendar object in calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectMovedEvent) {
-			try {
-				$this->activityBackend->onMovedCalendarObject(
-					$event->getSourceCalendarData(),
-					$event->getTargetCalendarData(),
-					$event->getSourceShares(),
-					$event->getTargetShares(),
-					$event->getObjectData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for moved calendar object from calendar %d to calendar %d', $event->getSourceCalendarId(), $event->getTargetCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectMovedToTrashEvent) {
-			try {
-				$this->activityBackend->onTouchCalendarObject(
-					\OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_MOVE_TO_TRASH,
-					$event->getCalendarData(),
-					$event->getShares(),
-					$event->getObjectData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for a calendar object of calendar %d that is moved to trash', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar object creation, so we just log it
-				$this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectRestoredEvent) {
-			try {
-				$this->activityBackend->onTouchCalendarObject(
-					\OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_RESTORE,
-					$event->getCalendarData(),
-					$event->getShares(),
-					$event->getObjectData()
-				);
-
-				$this->logger->debug(
-					sprintf('Activity generated for a restore calendar object of calendar %d', $event->getCalendarId())
-				);
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar object restoration, so we just log it
-				$this->logger->error('Error generating activity for a restored calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		} elseif ($event instanceof CalendarObjectDeletedEvent) {
-			try {
-				$deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
-				if (isset($event->getObjectData()[$deletedProp])) {
-					$this->logger->debug(
-						sprintf('Calendar object in calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
-					);
-				} else {
-					$this->activityBackend->onTouchCalendarObject(
-						\OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_DELETE,
-						$event->getCalendarData(),
-						$event->getShares(),
-						$event->getObjectData()
-					);
-
-					$this->logger->debug(
-						sprintf('Activity generated for deleted calendar object in calendar %d', $event->getCalendarId())
-					);
-				}
-			} catch (Throwable $e) {
-				// Any error with activities shouldn't abort the calendar deletion, so we just log it
-				$this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
-					'exception' => $e,
-				]);
-			}
-		}
-	}
+    public function __construct(
+        private ActivityBackend $activityBackend,
+        private LoggerInterface $logger,
+    ) {
+    }
+
+    public function handle(Event $event): void {
+        if ($event instanceof CalendarCreatedEvent) {
+            try {
+                $this->activityBackend->onCalendarAdd(
+                    $event->getCalendarData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for new calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar creation, so we just log it
+                $this->logger->error('Error generating activities for a new calendar: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarUpdatedEvent) {
+            try {
+                $this->activityBackend->onCalendarUpdate(
+                    $event->getCalendarData(),
+                    $event->getShares(),
+                    $event->getMutations()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar update, so we just log it
+                $this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarMovedToTrashEvent) {
+            try {
+                $this->activityBackend->onCalendarMovedToTrash(
+                    $event->getCalendarData(),
+                    $event->getShares()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar update, so we just log it
+                $this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarRestoredEvent) {
+            try {
+                $this->activityBackend->onCalendarRestored(
+                    $event->getCalendarData(),
+                    $event->getShares()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for changed calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar update, so we just log it
+                $this->logger->error('Error generating activities for changed calendar: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarDeletedEvent) {
+            try {
+                $deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
+                if (isset($event->getCalendarData()[$deletedProp])) {
+                    $this->logger->debug(
+                        sprintf('Calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
+                    );
+                } else {
+                    $this->activityBackend->onCalendarDelete(
+                        $event->getCalendarData(),
+                        $event->getShares()
+                    );
+
+                    $this->logger->debug(
+                        sprintf('Activity generated for deleted calendar %d', $event->getCalendarId())
+                    );
+                }
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar deletion, so we just log it
+                $this->logger->error('Error generating activities for a deleted calendar: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectCreatedEvent) {
+            try {
+                $this->activityBackend->onTouchCalendarObject(
+                    \OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_ADD,
+                    $event->getCalendarData(),
+                    $event->getShares(),
+                    $event->getObjectData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for new calendar object in calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar object creation, so we just log it
+                $this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectUpdatedEvent) {
+            try {
+                $this->activityBackend->onTouchCalendarObject(
+                    \OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_UPDATE,
+                    $event->getCalendarData(),
+                    $event->getShares(),
+                    $event->getObjectData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for updated calendar object in calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar deletion, so we just log it
+                $this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectMovedEvent) {
+            try {
+                $this->activityBackend->onMovedCalendarObject(
+                    $event->getSourceCalendarData(),
+                    $event->getTargetCalendarData(),
+                    $event->getSourceShares(),
+                    $event->getTargetShares(),
+                    $event->getObjectData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for moved calendar object from calendar %d to calendar %d', $event->getSourceCalendarId(), $event->getTargetCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar deletion, so we just log it
+                $this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectMovedToTrashEvent) {
+            try {
+                $this->activityBackend->onTouchCalendarObject(
+                    \OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_MOVE_TO_TRASH,
+                    $event->getCalendarData(),
+                    $event->getShares(),
+                    $event->getObjectData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for a calendar object of calendar %d that is moved to trash', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar object creation, so we just log it
+                $this->logger->error('Error generating activity for a new calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectRestoredEvent) {
+            try {
+                $this->activityBackend->onTouchCalendarObject(
+                    \OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_RESTORE,
+                    $event->getCalendarData(),
+                    $event->getShares(),
+                    $event->getObjectData()
+                );
+
+                $this->logger->debug(
+                    sprintf('Activity generated for a restore calendar object of calendar %d', $event->getCalendarId())
+                );
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar object restoration, so we just log it
+                $this->logger->error('Error generating activity for a restored calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        } elseif ($event instanceof CalendarObjectDeletedEvent) {
+            try {
+                $deletedProp = '{' . Plugin::NS_NEXTCLOUD . '}deleted-at';
+                if (isset($event->getObjectData()[$deletedProp])) {
+                    $this->logger->debug(
+                        sprintf('Calendar object in calendar %d was already in trashbin, skipping deletion activity', $event->getCalendarId())
+                    );
+                } else {
+                    $this->activityBackend->onTouchCalendarObject(
+                        \OCA\DAV\CalDAV\Activity\Provider\Event::SUBJECT_OBJECT_DELETE,
+                        $event->getCalendarData(),
+                        $event->getShares(),
+                        $event->getObjectData()
+                    );
+
+                    $this->logger->debug(
+                        sprintf('Activity generated for deleted calendar object in calendar %d', $event->getCalendarId())
+                    );
+                }
+            } catch (Throwable $e) {
+                // Any error with activities shouldn't abort the calendar deletion, so we just log it
+                $this->logger->error('Error generating activity for a deleted calendar object: ' . $e->getMessage(), [
+                    'exception' => $e,
+                ]);
+            }
+        }
+    }
 }