nextcloud / server

apps/dav/lib/DAV/CustomPropertiesBackend.php

Spacing +12 added lines, -12 removed lines

@@ -261,7 +261,7 @@ 
 	 */
 	#[Override]
 	public function propPatch($path, PropPatch $propPatch): void {
-		$propPatch->handleRemaining(function (array $changedProps) use ($path) {
+		$propPatch->handleRemaining(function(array $changedProps) use ($path) {
 			return $this->updateProperties($path, $changedProps);
 		});
 	}
@@ -357,7 +357,7 @@ 
 	 * Prefetch all user properties in a directory
 	 */
 	private function cacheDirectory(string $path, Directory $node): void {
-		$prefix = ltrim($path . '/', '/');
+		$prefix = ltrim($path.'/', '/');
 		$query = $this->connection->getQueryBuilder();
 		$query->select('name', 'p.propertypath', 'p.propertyname', 'p.propertyvalue', 'p.valuetype')
 			->from('filecache', 'f')
@@ -377,7 +377,7 @@ 
 		$propsByPath = [];
 
 		while ($row = $result->fetchAssociative()) {
-			$childPath = $prefix . $row['name'];
+			$childPath = $prefix.$row['name'];
 			if (!isset($propsByPath[$childPath])) {
 				$propsByPath[$childPath] = [];
 			}
@@ -396,16 +396,16 @@ 
 			if ($calendar instanceof Calendar) {
 				$user = str_replace('principals/users/', '', $calendar->getPrincipalURI());
 				if (!isset($users[$user])) {
-					$users[$user] = ['calendars/' . $user];
+					$users[$user] = ['calendars/'.$user];
 				}
-				$users[$user][] = 'calendars/' . $user . '/' . $calendar->getUri();
+				$users[$user][] = 'calendars/'.$user.'/'.$calendar->getUri();
 			} elseif ($calendar instanceof Inbox || $calendar instanceof Outbox || $calendar instanceof TrashbinHome || $calendar instanceof ExternalCalendar) {
 				if ($calendar->getOwner()) {
 					$user = str_replace('principals/users/', '', $calendar->getOwner());
 					if (!isset($users[$user])) {
-						$users[$user] = ['calendars/' . $user];
+						$users[$user] = ['calendars/'.$user];
 					}
-					$users[$user][] = 'calendars/' . $user . '/' . $calendar->getName();
+					$users[$user][] = 'calendars/'.$user.'/'.$calendar->getName();
 				}
 			}
 		}
@@ -572,7 +572,7 @@ 
 				self::checkIsArrayOfScalar($name, $item);
 			} elseif ($item !== null && !is_scalar($item)) {
 				throw new DavException(
-					"Property \"$name\" has an invalid value of array containing " . gettype($item),
+					"Property \"$name\" has an invalid value of array containing ".gettype($item),
 				);
 			}
 		}
@@ -597,7 +597,7 @@ 
 			$this->validateProperty($path, $name, $value);
 		} catch (DavException $e) {
 			throw new DavException(
-				"Property \"$name\" has an invalid value: " . $e->getMessage(),
+				"Property \"$name\" has an invalid value: ".$e->getMessage(),
 				0,
 				$e,
 			);
@@ -617,7 +617,7 @@ 
 				self::checkIsArrayOfScalar($name, $value);
 			} elseif (!is_object($value)) {
 				throw new DavException(
-					"Property \"$name\" has an invalid value of type " . gettype($value),
+					"Property \"$name\" has an invalid value of type ".gettype($value),
 				);
 			} else {
 				if (!str_starts_with($value::class, 'Sabre\\DAV\\Xml\\Property\\')
@@ -625,7 +625,7 @@ 
 					&& !str_starts_with($value::class, 'Sabre\\CardDAV\\Xml\\Property\\')
 					&& !str_starts_with($value::class, 'OCA\\DAV\\')) {
 					throw new DavException(
-						"Property \"$name\" has an invalid value of class " . $value::class,
+						"Property \"$name\" has an invalid value of class ".$value::class,
 					);
 				}
 			}
@@ -674,7 +674,7 @@ 
 
 		try {
 			// Build path relative to the dav base URI to be used later to find the node
-			$value = new LocalHref($this->server->calculateUri($href) . '/');
+			$value = new LocalHref($this->server->calculateUri($href).'/');
 		} catch (DavException\Forbidden) {
 			// Not existing calendars will be handled later when the value is validated
 		}

apps/dav/lib/BackgroundJob/BuildReminderIndexBackgroundJob.php

Spacing +5 added lines, -5 removed lines

@@ -40,10 +40,10 @@ 
 	}
 
 	public function run($argument) {
-		$offset = (int)$argument['offset'];
-		$stopAt = (int)$argument['stopAt'];
+		$offset = (int) $argument['offset'];
+		$stopAt = (int) $argument['stopAt'];
 
-		$this->logger->info('Building calendar reminder index (' . $offset . '/' . $stopAt . ')');
+		$this->logger->info('Building calendar reminder index ('.$offset.'/'.$stopAt.')');
 
 		$offset = $this->buildIndex($offset, $stopAt);
 
@@ -54,7 +54,7 @@ 
 				'offset' => $offset,
 				'stopAt' => $stopAt
 			]);
-			$this->logger->info('Scheduled a new BuildReminderIndexBackgroundJob with offset ' . $offset);
+			$this->logger->info('Scheduled a new BuildReminderIndexBackgroundJob with offset '.$offset);
 		}
 	}
 
@@ -75,7 +75,7 @@ 
 
 		$result = $query->executeQuery();
 		while ($row = $result->fetchAssociative()) {
-			$offset = (int)$row['id'];
+			$offset = (int) $row['id'];
 			if (is_resource($row['calendardata'])) {
 				$row['calendardata'] = stream_get_contents($row['calendardata']);
 			}

apps/dav/lib/CardDAV/CardDavBackend.php

Spacing +63 added lines, -63 removed lines

@@ -77,7 +77,7 @@ 
 			->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri)));
 
 		$result = $query->executeQuery();
-		$column = (int)$result->fetchOne();
+		$column = (int) $result->fetchOne();
 		$result->closeCursor();
 		return $column;
 	}
@@ -100,7 +100,7 @@ 
 	 * @return array
 	 */
 	public function getAddressBooksForUser($principalUri) {
-		return $this->atomic(function () use ($principalUri) {
+		return $this->atomic(function() use ($principalUri) {
 			$principalUriOriginal = $principalUri;
 			$principalUri = $this->convertPrincipal($principalUri, true);
 			$select = $this->db->getQueryBuilder();
@@ -117,7 +117,7 @@ 
 					'uri' => $row['uri'],
 					'principaluri' => $this->convertPrincipal($row['principaluri'], false),
 					'{DAV:}displayname' => $row['displayname'],
-					'{' . Plugin::NS_CARDDAV . '}addressbook-description' => $row['description'],
+					'{'.Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
 					'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
 					'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ?: '0',
 				];
@@ -148,13 +148,13 @@ 
 				->andWhere($select->expr()->notIn('s.id', $select->createFunction($subSelect->getSQL()), IQueryBuilder::PARAM_INT_ARRAY));
 			$result = $select->executeQuery();
 
-			$readOnlyPropertyName = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
+			$readOnlyPropertyName = '{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}read-only';
 			while ($row = $result->fetchAssociative()) {
 				if ($row['principaluri'] === $principalUri) {
 					continue;
 				}
 
-				$readOnly = (int)$row['access'] === Backend::ACCESS_READ;
+				$readOnly = (int) $row['access'] === Backend::ACCESS_READ;
 				if (isset($addressBooks[$row['id']])) {
 					if ($readOnly) {
 						// New share can not have more permissions then the old one.
@@ -168,18 +168,18 @@ 
 				}
 
 				[, $name] = \Sabre\Uri\split($row['principaluri']);
-				$uri = $row['uri'] . '_shared_by_' . $name;
-				$displayName = $row['displayname'] . ' (' . ($this->userManager->getDisplayName($name) ?? $name ?? '') . ')';
+				$uri = $row['uri'].'_shared_by_'.$name;
+				$displayName = $row['displayname'].' ('.($this->userManager->getDisplayName($name) ?? $name ?? '').')';
 
 				$addressBooks[$row['id']] = [
 					'id' => $row['id'],
 					'uri' => $uri,
 					'principaluri' => $principalUriOriginal,
 					'{DAV:}displayname' => $displayName,
-					'{' . Plugin::NS_CARDDAV . '}addressbook-description' => $row['description'],
+					'{'.Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
 					'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
 					'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ?: '0',
-					'{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal' => $row['principaluri'],
+					'{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}owner-principal' => $row['principaluri'],
 					$readOnlyPropertyName => $readOnly,
 				];
 
@@ -207,7 +207,7 @@ 
 				'uri' => $row['uri'],
 				'principaluri' => $this->convertPrincipal($row['principaluri'], false),
 				'{DAV:}displayname' => $row['displayname'],
-				'{' . Plugin::NS_CARDDAV . '}addressbook-description' => $row['description'],
+				'{'.Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
 				'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
 				'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ?: '0',
 			];
@@ -239,7 +239,7 @@ 
 			'uri' => $row['uri'],
 			'principaluri' => $row['principaluri'],
 			'{DAV:}displayname' => $row['displayname'],
-			'{' . Plugin::NS_CARDDAV . '}addressbook-description' => $row['description'],
+			'{'.Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
 			'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
 			'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ?: '0',
 		];
@@ -269,7 +269,7 @@ 
 			'uri' => $row['uri'],
 			'principaluri' => $row['principaluri'],
 			'{DAV:}displayname' => $row['displayname'],
-			'{' . Plugin::NS_CARDDAV . '}addressbook-description' => $row['description'],
+			'{'.Plugin::NS_CARDDAV.'}addressbook-description' => $row['description'],
 			'{http://calendarserver.org/ns/}getctag' => $row['synctoken'],
 			'{http://sabredav.org/ns}sync-token' => $row['synctoken'] ?: '0',
 
@@ -277,8 +277,8 @@ 
 
 		// system address books are always read only
 		if ($principal === 'principals/system/system') {
-			$addressBook['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal'] = $row['principaluri'];
-			$addressBook['{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only'] = true;
+			$addressBook['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}owner-principal'] = $row['principaluri'];
+			$addressBook['{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}read-only'] = true;
 		}
 
 		$this->addOwnerPrincipal($addressBook);
@@ -305,22 +305,22 @@ 
 	public function updateAddressBook($addressBookId, \Sabre\DAV\PropPatch $propPatch) {
 		$supportedProperties = [
 			'{DAV:}displayname',
-			'{' . Plugin::NS_CARDDAV . '}addressbook-description',
+			'{'.Plugin::NS_CARDDAV.'}addressbook-description',
 		];
 
-		$propPatch->handle($supportedProperties, function ($mutations) use ($addressBookId) {
+		$propPatch->handle($supportedProperties, function($mutations) use ($addressBookId) {
 			$updates = [];
 			foreach ($mutations as $property => $newValue) {
 				switch ($property) {
 					case '{DAV:}displayname':
 						$updates['displayname'] = $newValue;
 						break;
-					case '{' . Plugin::NS_CARDDAV . '}addressbook-description':
+					case '{'.Plugin::NS_CARDDAV.'}addressbook-description':
 						$updates['description'] = $newValue;
 						break;
 				}
 			}
-			[$addressBookRow, $shares] = $this->atomic(function () use ($addressBookId, $updates) {
+			[$addressBookRow, $shares] = $this->atomic(function() use ($addressBookId, $updates) {
 				$query = $this->db->getQueryBuilder();
 				$query->update('addressbooks');
 
@@ -332,12 +332,12 @@ 
 
 				$this->addChange($addressBookId, '', 2);
 
-				$addressBookRow = $this->getAddressBookById((int)$addressBookId);
-				$shares = $this->getShares((int)$addressBookId);
+				$addressBookRow = $this->getAddressBookById((int) $addressBookId);
+				$shares = $this->getShares((int) $addressBookId);
 				return [$addressBookRow, $shares];
 			}, $this->db);
 
-			$this->dispatcher->dispatchTyped(new AddressBookUpdatedEvent((int)$addressBookId, $addressBookRow, $shares, $mutations));
+			$this->dispatcher->dispatchTyped(new AddressBookUpdatedEvent((int) $addressBookId, $addressBookRow, $shares, $mutations));
 
 			return true;
 		});
@@ -371,11 +371,11 @@ 
 				case '{DAV:}displayname':
 					$values['displayname'] = $newValue;
 					break;
-				case '{' . Plugin::NS_CARDDAV . '}addressbook-description':
+				case '{'.Plugin::NS_CARDDAV.'}addressbook-description':
 					$values['description'] = $newValue;
 					break;
 				default:
-					throw new BadRequest('Unknown property: ' . $property);
+					throw new BadRequest('Unknown property: '.$property);
 			}
 		}
 
@@ -385,7 +385,7 @@ 
 			$values['displayname'] = $url;
 		}
 
-		[$addressBookId, $addressBookRow] = $this->atomic(function () use ($values) {
+		[$addressBookId, $addressBookRow] = $this->atomic(function() use ($values) {
 			$query = $this->db->getQueryBuilder();
 			$query->insert('addressbooks')
 				->values([
@@ -417,8 +417,8 @@ 
 	 * @return void
 	 */
 	public function deleteAddressBook($addressBookId) {
-		$this->atomic(function () use ($addressBookId): void {
-			$addressBookId = (int)$addressBookId;
+		$this->atomic(function() use ($addressBookId): void {
+			$addressBookId = (int) $addressBookId;
 			$addressBookData = $this->getAddressBookById($addressBookId);
 			$shares = $this->getShares($addressBookId);
 
@@ -482,7 +482,7 @@ 
 
 		$result = $query->executeQuery();
 		while ($row = $result->fetchAssociative()) {
-			$row['etag'] = '"' . $row['etag'] . '"';
+			$row['etag'] = '"'.$row['etag'].'"';
 
 			$modified = false;
 			$row['carddata'] = $this->readBlob($row['carddata'], $modified);
@@ -522,7 +522,7 @@ 
 		if (!$row) {
 			return false;
 		}
-		$row['etag'] = '"' . $row['etag'] . '"';
+		$row['etag'] = '"'.$row['etag'].'"';
 
 		$modified = false;
 		$row['carddata'] = $this->readBlob($row['carddata'], $modified);
@@ -564,7 +564,7 @@ 
 			$result = $query->executeQuery();
 
 			while ($row = $result->fetchAssociative()) {
-				$row['etag'] = '"' . $row['etag'] . '"';
+				$row['etag'] = '"'.$row['etag'].'"';
 
 				$modified = false;
 				$row['carddata'] = $this->readBlob($row['carddata'], $modified);
@@ -608,7 +608,7 @@ 
 	public function createCard($addressBookId, $cardUri, $cardData, bool $checkAlreadyExists = true) {
 		$etag = md5($cardData);
 		$uid = $this->getUID($cardData);
-		return $this->atomic(function () use ($addressBookId, $cardUri, $cardData, $checkAlreadyExists, $etag, $uid) {
+		return $this->atomic(function() use ($addressBookId, $cardUri, $cardData, $checkAlreadyExists, $etag, $uid) {
 			if ($checkAlreadyExists) {
 				$q = $this->db->getQueryBuilder();
 				$q->select('uid')
@@ -617,7 +617,7 @@ 
 					->andWhere($q->expr()->eq('uid', $q->createNamedParameter($uid)))
 					->setMaxResults(1);
 				$result = $q->executeQuery();
-				$count = (bool)$result->fetchOne();
+				$count = (bool) $result->fetchOne();
 				$result->closeCursor();
 				if ($count) {
 					throw new \Sabre\DAV\Exception\BadRequest('VCard object with uid already exists in this addressbook collection.');
@@ -648,7 +648,7 @@ 
 			$objectRow = $this->getCard($addressBookId, $cardUri);
 			$this->dispatcher->dispatchTyped(new CardCreatedEvent($addressBookId, $addressBookData, $shares, $objectRow));
 
-			return '"' . $etag . '"';
+			return '"'.$etag.'"';
 		}, $this->db);
 	}
 
@@ -681,13 +681,13 @@ 
 		$uid = $this->getUID($cardData);
 		$etag = md5($cardData);
 
-		return $this->atomic(function () use ($addressBookId, $cardUri, $cardData, $uid, $etag) {
+		return $this->atomic(function() use ($addressBookId, $cardUri, $cardData, $uid, $etag) {
 			$query = $this->db->getQueryBuilder();
 
 			// check for recently stored etag and stop if it is the same
 			$etagCacheKey = "$addressBookId#$cardUri";
 			if (isset($this->etagCache[$etagCacheKey]) && $this->etagCache[$etagCacheKey] === $etag) {
-				return '"' . $etag . '"';
+				return '"'.$etag.'"';
 			}
 
 			$query->update($this->dbCardsTable)
@@ -709,7 +709,7 @@ 
 			$shares = $this->getShares($addressBookId);
 			$objectRow = $this->getCard($addressBookId, $cardUri);
 			$this->dispatcher->dispatchTyped(new CardUpdatedEvent($addressBookId, $addressBookData, $shares, $objectRow));
-			return '"' . $etag . '"';
+			return '"'.$etag.'"';
 		}, $this->db);
 	}
 
@@ -717,12 +717,12 @@ 
 	 * @throws Exception
 	 */
 	public function moveCard(int $sourceAddressBookId, string $sourceObjectUri, int $targetAddressBookId, string $tragetObjectUri): bool {
-		return $this->atomic(function () use ($sourceAddressBookId, $sourceObjectUri, $targetAddressBookId, $tragetObjectUri) {
+		return $this->atomic(function() use ($sourceAddressBookId, $sourceObjectUri, $targetAddressBookId, $tragetObjectUri) {
 			$card = $this->getCard($sourceAddressBookId, $sourceObjectUri);
 			if (empty($card)) {
 				return false;
 			}
-			$sourceObjectId = (int)$card['id'];
+			$sourceObjectId = (int) $card['id'];
 
 			$query = $this->db->getQueryBuilder();
 			$query->update('cards')
@@ -765,7 +765,7 @@ 
 	 * @return bool
 	 */
 	public function deleteCard($addressBookId, $cardUri) {
-		return $this->atomic(function () use ($addressBookId, $cardUri) {
+		return $this->atomic(function() use ($addressBookId, $cardUri) {
 			$addressBookData = $this->getAddressBookById($addressBookId);
 			$shares = $this->getShares($addressBookId);
 			$objectRow = $this->getCard($addressBookId, $cardUri);
@@ -855,7 +855,7 @@ 
 		$maxLimit = $this->config->getSystemValueInt('carddav_sync_request_truncation', 2500);
 		$limit = ($limit === null) ? $maxLimit : min($limit, $maxLimit);
 		// Current synctoken
-		return $this->atomic(function () use ($addressBookId, $syncToken, $syncLevel, $limit) {
+		return $this->atomic(function() use ($addressBookId, $syncToken, $syncLevel, $limit) {
 			$qb = $this->db->getQueryBuilder();
 			$qb->select('synctoken')
 				->from('addressbooks')
@@ -899,7 +899,7 @@ 
 				} else {
 					$lastID = $values[array_key_last($values)]['id'];
 					$result['added'] = array_column($values, 'uri');
-					$result['syncToken'] = count($result['added']) >= $limit ? "init_{$lastID}_$initialSyncToken" : $initialSyncToken ;
+					$result['syncToken'] = count($result['added']) >= $limit ? "init_{$lastID}_$initialSyncToken" : $initialSyncToken;
 					$result['result_truncated'] = count($result['added']) >= $limit;
 				}
 			} elseif ($syncToken) {
@@ -983,7 +983,7 @@ 
 				}
 				$lastID = $values[array_key_last($values)]['id'];
 				if (count($values) >= $limit) {
-					$result['syncToken'] = 'init_' . $lastID . '_' . $currentToken;
+					$result['syncToken'] = 'init_'.$lastID.'_'.$currentToken;
 					$result['result_truncated'] = true;
 				}
 
@@ -1004,13 +1004,13 @@ 
 	 * @return void
 	 */
 	protected function addChange(int $addressBookId, string $objectUri, int $operation): void {
-		$this->atomic(function () use ($addressBookId, $objectUri, $operation): void {
+		$this->atomic(function() use ($addressBookId, $objectUri, $operation): void {
 			$query = $this->db->getQueryBuilder();
 			$query->select('synctoken')
 				->from('addressbooks')
 				->where($query->expr()->eq('id', $query->createNamedParameter($addressBookId)));
 			$result = $query->executeQuery();
-			$syncToken = (int)$result->fetchOne();
+			$syncToken = (int) $result->fetchOne();
 			$result->closeCursor();
 
 			$query = $this->db->getQueryBuilder();
@@ -1079,7 +1079,7 @@ 
 	 * @param list<string> $remove
 	 */
 	public function updateShares(IShareable $shareable, array $add, array $remove): void {
-		$this->atomic(function () use ($shareable, $add, $remove): void {
+		$this->atomic(function() use ($shareable, $add, $remove): void {
 			$addressBookId = $shareable->getResourceId();
 			$addressBookData = $this->getAddressBookById($addressBookId);
 			$oldShares = $this->getShares($addressBookId);
@@ -1113,7 +1113,7 @@ 
 	 * @return array an array of contacts which are arrays of key-value-pairs
 	 */
 	public function search($addressBookId, $pattern, $searchProperties, $options = []): array {
-		return $this->atomic(function () use ($addressBookId, $pattern, $searchProperties, $options) {
+		return $this->atomic(function() use ($addressBookId, $pattern, $searchProperties, $options) {
 			return $this->searchByAddressBookIds([$addressBookId], $pattern, $searchProperties, $options);
 		}, $this->db);
 	}
@@ -1131,9 +1131,9 @@ 
 		string $pattern,
 		array $searchProperties,
 		array $options = []): array {
-		return $this->atomic(function () use ($principalUri, $pattern, $searchProperties, $options) {
-			$addressBookIds = array_map(static function ($row):int {
-				return (int)$row['id'];
+		return $this->atomic(function() use ($principalUri, $pattern, $searchProperties, $options) {
+			$addressBookIds = array_map(static function($row):int {
+				return (int) $row['id'];
 			}, $this->getAddressBooksForUser($principalUri));
 
 			return $this->searchByAddressBookIds($addressBookIds, $pattern, $searchProperties, $options);
@@ -1168,7 +1168,7 @@ 
 		$useWildcards = !\array_key_exists('wildcard', $options) || $options['wildcard'] !== false;
 
 		if ($escapePattern) {
-			$searchProperties = array_filter($searchProperties, function ($property) use ($pattern) {
+			$searchProperties = array_filter($searchProperties, function($property) use ($pattern) {
 				if ($property === 'EMAIL' && str_contains($pattern, ' ')) {
 					// There can be no spaces in emails
 					return false;
@@ -1201,7 +1201,7 @@ 
 			} elseif (!$escapePattern) {
 				$query2->andWhere($query2->expr()->ilike('cp.value', $query2->createNamedParameter($pattern)));
 			} else {
-				$query2->andWhere($query2->expr()->ilike('cp.value', $query2->createNamedParameter('%' . $this->db->escapeLikeParameter($pattern) . '%')));
+				$query2->andWhere($query2->expr()->ilike('cp.value', $query2->createNamedParameter('%'.$this->db->escapeLikeParameter($pattern).'%')));
 			}
 		}
 		if (isset($options['limit'])) {
@@ -1212,7 +1212,7 @@ 
 		}
 
 		if (isset($options['person'])) {
-			$query2->andWhere($query2->expr()->ilike('cp.value', $query2->createNamedParameter('%' . $this->db->escapeLikeParameter($options['person']) . '%')));
+			$query2->andWhere($query2->expr()->ilike('cp.value', $query2->createNamedParameter('%'.$this->db->escapeLikeParameter($options['person']).'%')));
 		}
 		if (isset($options['since']) || isset($options['until'])) {
 			$query2->join('cp', $this->dbCardsPropertiesTable, 'cp_bday', 'cp.cardid = cp_bday.cardid');
@@ -1240,8 +1240,8 @@ 
 		$result = $query2->executeQuery();
 		$matches = $result->fetchAllAssociative();
 		$result->closeCursor();
-		$matches = array_map(function ($match) {
-			return (int)$match['cardid'];
+		$matches = array_map(function($match) {
+			return (int) $match['cardid'];
 		}, $matches);
 
 		$cardResults = [];
@@ -1258,8 +1258,8 @@ 
 		}
 
 		$cards = array_merge(...$cardResults);
-		return array_map(function ($array) {
-			$array['addressbookid'] = (int)$array['addressbookid'];
+		return array_map(function($array) {
+			$array['addressbookid'] = (int) $array['addressbookid'];
 			$modified = false;
 			$array['carddata'] = $this->readBlob($array['carddata'], $modified);
 			if ($modified) {
@@ -1305,7 +1305,7 @@ 
 		$result->closeCursor();
 
 		if (!isset($uri['uri'])) {
-			throw new \InvalidArgumentException('Card does not exists: ' . $id);
+			throw new \InvalidArgumentException('Card does not exists: '.$id);
 		}
 
 		return $uri['uri'];
@@ -1330,7 +1330,7 @@ 
 
 		if (is_array($contact)) {
 			$modified = false;
-			$contact['etag'] = '"' . $contact['etag'] . '"';
+			$contact['etag'] = '"'.$contact['etag'].'"';
 			$contact['carddata'] = $this->readBlob($contact['carddata'], $modified);
 			if ($modified) {
 				$contact['size'] = strlen($contact['carddata']);
@@ -1365,7 +1365,7 @@ 
 	 * @param string $vCardSerialized
 	 */
 	protected function updateProperties($addressBookId, $cardUri, $vCardSerialized) {
-		$this->atomic(function () use ($addressBookId, $cardUri, $vCardSerialized): void {
+		$this->atomic(function() use ($addressBookId, $cardUri, $vCardSerialized): void {
 			$cardId = $this->getCardId($addressBookId, $cardUri);
 			$vCard = $this->readCard($vCardSerialized);
 
@@ -1440,10 +1440,10 @@ 
 		$result->closeCursor();
 
 		if (!isset($cardIds['id'])) {
-			throw new \InvalidArgumentException('Card does not exists: ' . $uri);
+			throw new \InvalidArgumentException('Card does not exists: '.$uri);
 		}
 
-		return (int)$cardIds['id'];
+		return (int) $cardIds['id'];
 	}
 
 	/**
@@ -1471,7 +1471,7 @@ 
 			->from('addressbookchanges');
 
 		$result = $query->executeQuery();
-		$maxId = (int)$result->fetchOne();
+		$maxId = (int) $result->fetchOne();
 		$result->closeCursor();
 		if (!$maxId || $maxId < $keep) {
 			return 0;
@@ -1498,8 +1498,8 @@ 
 	}
 
 	private function addOwnerPrincipal(array &$addressbookInfo): void {
-		$ownerPrincipalKey = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}owner-principal';
-		$displaynameKey = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD . '}owner-displayname';
+		$ownerPrincipalKey = '{'.\OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD.'}owner-principal';
+		$displaynameKey = '{'.\OCA\DAV\DAV\Sharing\Plugin::NS_NEXTCLOUD.'}owner-displayname';
 		if (isset($addressbookInfo[$ownerPrincipalKey])) {
 			$uri = $addressbookInfo[$ownerPrincipalKey];
 		} else {

apps/dav/lib/Migration/RemoveDeletedUsersCalendarSubscriptions.php

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
 		$result->closeCursor();
 
 		if ($count !== false) {
-			$count = (int)$count;
+			$count = (int) $count;
 		} else {
 			$count = 0;
 		}
@@ -91,7 +91,7 @@ 
 		while ($row = $result->fetchAssociative()) {
 			$username = $this->getPrincipal($row['principaluri']);
 			if (!$this->userManager->userExists($username)) {
-				$this->orphanSubscriptionIds[] = (int)$row['id'];
+				$this->orphanSubscriptionIds[] = (int) $row['id'];
 			}
 		}
 		$result->closeCursor();

apps/dav/lib/Migration/BuildSocialSearchIndexBackgroundJob.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 		$offset = $argument['offset'];
 		$stopAt = $argument['stopAt'];
 
-		$this->logger->info('Indexing social profile data (' . $offset . '/' . $stopAt . ')');
+		$this->logger->info('Indexing social profile data ('.$offset.'/'.$stopAt.')');
 
 		$offset = $this->buildIndex($offset, $stopAt);
 
@@ -42,7 +42,7 @@ 
 				'offset' => $offset,
 				'stopAt' => $stopAt
 			]);
-			$this->logger->info('New social profile indexing job scheduled with offset ' . $offset);
+			$this->logger->info('New social profile indexing job scheduled with offset '.$offset);
 		}
 	}
 
@@ -60,7 +60,7 @@ 
 			->from('cards', 'c')
 			->orderBy('id', 'ASC')
 			->where($query->expr()->like('carddata', $query->createNamedParameter('%SOCIALPROFILE%')))
-			->andWhere($query->expr()->gt('id', $query->createNamedParameter((int)$offset, IQueryBuilder::PARAM_INT)))
+			->andWhere($query->expr()->gt('id', $query->createNamedParameter((int) $offset, IQueryBuilder::PARAM_INT)))
 			->setMaxResults(100);
 		$social_cards = $query->executeQuery()->fetchAllAssociative();
 

apps/user_ldap/lib/Mapping/AbstractMapping.php

Spacing +13 added lines, -13 removed lines

@@ -78,7 +78,7 @@ 
 		}
 
 		$section = \str_contains($this->getTableName(), 'user') ? 'u/' : 'g/';
-		$this->localNameToDnCache = $this->cacheFactory->createLocal('ldap/map/' . $section);
+		$this->localNameToDnCache = $this->cacheFactory->createLocal('ldap/map/'.$section);
 
 		// We use the cache as well to store whether it shall be used. If the
 		// answer was no, we unset it again.
@@ -101,7 +101,7 @@ 
 		$row = $result->fetchAssociative();
 		$result->closeCursor();
 
-		$use = (int)$row['count'] <= self::LOCAL_CACHE_OBJECT_THRESHOLD;
+		$use = (int) $row['count'] <= self::LOCAL_CACHE_OBJECT_THRESHOLD;
 		$this->localNameToDnCache->set('use', $use, self::LOCAL_USER_COUNT_TTL);
 		return $use;
 	}
@@ -175,7 +175,7 @@ 
 	/**
 	 * Gets the LDAP DN based on the provided name.
 	 */
-	public function getDNByName(string $name): string|false {
+	public function getDNByName(string $name): string | false {
 		$dn = array_search($name, $this->cache, true);
 		if ($dn === false) {
 			$dn = $this->localNameToDnCache?->get($name);
@@ -200,7 +200,7 @@ 
 	public function setDNbyUUID($fdn, $uuid) {
 		$oldDn = $this->getDnByUUID($uuid);
 		$statement = $this->dbc->prepare('
-			UPDATE `' . $this->getTableName() . '`
+			UPDATE `' . $this->getTableName().'`
 			SET `ldap_dn_hash` = ?, `ldap_dn` = ?
 			WHERE `directory_uuid` = ?
 		');
@@ -232,7 +232,7 @@ 
 	 */
 	public function setUUIDbyDN($uuid, $fdn): bool {
 		$statement = $this->dbc->prepare('
-			UPDATE `' . $this->getTableName() . '`
+			UPDATE `' . $this->getTableName().'`
 			SET `directory_uuid` = ?
 			WHERE `ldap_dn_hash` = ?
 		');
@@ -335,12 +335,12 @@ 
 	public function getNamesBySearch(string $search, string $prefixMatch = '', string $postfixMatch = ''): array {
 		$statement = $this->dbc->prepare('
 			SELECT `owncloud_name`
-			FROM `' . $this->getTableName() . '`
+			FROM `' . $this->getTableName().'`
 			WHERE `owncloud_name` LIKE ?
 		');
 
 		try {
-			$res = $statement->execute([$prefixMatch . $this->dbc->escapeLikeParameter($search) . $postfixMatch]);
+			$res = $statement->execute([$prefixMatch.$this->dbc->escapeLikeParameter($search).$postfixMatch]);
 		} catch (Exception $e) {
 			return [];
 		}
@@ -425,12 +425,12 @@ 
 
 		try {
 			$result = $this->dbc->insertIfNotExist($this->getTableName(), $row);
-			if ((bool)$result === true) {
+			if ((bool) $result === true) {
 				$this->cache[$fdn] = $name;
 				$this->localNameToDnCache?->set($name, $fdn, self::LOCAL_CACHE_TTL);
 			}
 			// insertIfNotExist returns values as int
-			return (bool)$result;
+			return (bool) $result;
 		} catch (\Exception $e) {
 			return false;
 		}
@@ -444,7 +444,7 @@ 
 	 */
 	public function unmap($name) {
 		$statement = $this->dbc->prepare('
-			DELETE FROM `' . $this->getTableName() . '`
+			DELETE FROM `' . $this->getTableName().'`
 			WHERE `owncloud_name` = ?');
 
 		$dn = array_search($name, $this->cache);
@@ -464,7 +464,7 @@ 
 	public function clear() {
 		$sql = $this->dbc
 			->getDatabasePlatform()
-			->getTruncateTableSQL('`' . $this->getTableName() . '`');
+			->getTruncateTableSQL('`'.$this->getTableName().'`');
 		try {
 			$this->dbc->executeQuery($sql);
 			$this->localNameToDnCache?->clear();
@@ -513,7 +513,7 @@ 
 		$res = $query->executeQuery();
 		$count = $res->fetchOne();
 		$res->closeCursor();
-		return (int)$count;
+		return (int) $count;
 	}
 
 	public function countInvalidated(): int {
@@ -524,6 +524,6 @@ 
 		$res = $query->executeQuery();
 		$count = $res->fetchOne();
 		$res->closeCursor();
-		return (int)$count;
+		return (int) $count;
 	}
 }

apps/federatedfilesharing/lib/FederatedShareProvider.php

Spacing +13 added lines, -13 removed lines

@@ -140,7 +140,7 @@ 
 		if ($remoteShare) {
 			try {
 				$ownerCloudId = $this->cloudIdManager->getCloudId($remoteShare['owner'], $remoteShare['remote']);
-				$shareId = $this->addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $ownerCloudId->getId(), $permissions, 'tmp_token_' . time(), $shareType, $expirationDate);
+				$shareId = $this->addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $ownerCloudId->getId(), $permissions, 'tmp_token_'.time(), $shareType, $expirationDate);
 				$share->setId($shareId);
 				[$token, $remoteId] = $this->askOwnerToReShare($shareWith, $share, $shareId);
 				// remote share was create successfully if we get a valid token as return
@@ -274,7 +274,7 @@ 
 		$result = $qResult->fetchAllAssociative();
 		$qResult->closeCursor();
 
-		if (isset($result[0]) && (int)$result[0]['remote_id'] > 0) {
+		if (isset($result[0]) && (int) $result[0]['remote_id'] > 0) {
 			return $result[0];
 		}
 
@@ -408,7 +408,7 @@ 
 	public function getRemoteId(IShare $share): string {
 		$query = $this->dbConnection->getQueryBuilder();
 		$query->select('remote_id')->from('federated_reshares')
-			->where($query->expr()->eq('share_id', $query->createNamedParameter((int)$share->getId())));
+			->where($query->expr()->eq('share_id', $query->createNamedParameter((int) $share->getId())));
 		$result = $query->executeQuery();
 		$data = $result->fetchAssociative();
 		$result->closeCursor();
@@ -417,7 +417,7 @@ 
 			throw new ShareNotFound();
 		}
 
-		return (string)$data['remote_id'];
+		return (string) $data['remote_id'];
 	}
 
 	/**
@@ -671,7 +671,7 @@ 
 		$cursor->closeCursor();
 
 		if ($data === false) {
-			throw new ShareNotFound('Can not find share with ID: ' . $id);
+			throw new ShareNotFound('Can not find share with ID: '.$id);
 		}
 
 		try {
@@ -814,16 +814,16 @@ 
 	 */
 	private function createShareObject($data) {
 		$share = new Share($this->rootFolder, $this->userManager);
-		$share->setId((int)$data['id'])
-			->setShareType((int)$data['share_type'])
-			->setPermissions((int)$data['permissions'])
+		$share->setId((int) $data['id'])
+			->setShareType((int) $data['share_type'])
+			->setPermissions((int) $data['permissions'])
 			->setTarget($data['file_target'])
-			->setMailSend((bool)$data['mail_send'])
-			->setStatus((int)$data['accepted'])
+			->setMailSend((bool) $data['mail_send'])
+			->setStatus((int) $data['accepted'])
 			->setToken($data['token']);
 
 		$shareTime = new \DateTime();
-		$shareTime->setTimestamp((int)$data['stime']);
+		$shareTime->setTimestamp((int) $data['stime']);
 		$share->setShareTime($shareTime);
 		$share->setSharedWith($data['share_with']);
 
@@ -833,13 +833,13 @@ 
 		} else {
 			//OLD SHARE
 			$share->setSharedBy($data['uid_owner']);
-			$path = $this->getNode($share->getSharedBy(), (int)$data['file_source']);
+			$path = $this->getNode($share->getSharedBy(), (int) $data['file_source']);
 
 			$owner = $path->getOwner();
 			$share->setShareOwner($owner->getUID());
 		}
 
-		$share->setNodeId((int)$data['file_source']);
+		$share->setNodeId((int) $data['file_source']);
 		$share->setNodeType($data['item_type']);
 
 		$share->setProviderId($this->identifier());

apps/files_external/lib/Migration/Version1015Date20211104103506.php

Spacing +5 added lines, -5 removed lines

@@ -38,19 +38,19 @@
 		}
 
 		while ($mount = $mounts->fetchAssociative()) {
-			$config = $this->getStorageConfig((int)$mount['mount_id']);
+			$config = $this->getStorageConfig((int) $mount['mount_id']);
 			$hostname = $config['hostname'];
 			$bucket = $config['bucket'];
 			$key = $config['key'];
-			$oldId = Storage::adjustStorageId('amazon::' . $bucket);
-			$newId = Storage::adjustStorageId('amazon::external::' . md5($hostname . ':' . $bucket . ':' . $key));
+			$oldId = Storage::adjustStorageId('amazon::'.$bucket);
+			$newId = Storage::adjustStorageId('amazon::external::'.md5($hostname.':'.$bucket.':'.$key));
 			try {
 				$qb->setParameter('oldId', $oldId);
 				$qb->setParameter('newId', $newId);
 				$qb->executeStatement();
-				$this->logger->info('Migrated s3 storage id for mount with id ' . $mount['mount_id'] . ' to ' . $newId);
+				$this->logger->info('Migrated s3 storage id for mount with id '.$mount['mount_id'].' to '.$newId);
 			} catch (Exception $e) {
-				$this->logger->error('Failed to migrate external s3 storage id for mount with id ' . $mount['mount_id'], [
+				$this->logger->error('Failed to migrate external s3 storage id for mount with id '.$mount['mount_id'], [
 					'exception' => $e
 				]);
 			}

apps/files/lib/Command/DeleteOrphanedFiles.php

Spacing +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 			$result = $query->executeQuery();
 			while ($row = $result->fetchAssociative()) {
 				$deletedInLastChunk++;
-				$deletedEntries += $deleteQuery->setParameter('storageid', (int)$row['storage_id'])
+				$deletedEntries += $deleteQuery->setParameter('storageid', (int) $row['storage_id'])
 					->executeStatement();
 			}
 			$result->closeCursor();