nextcloud / server

apps/theming/templates/settings-admin.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 ?>
 <div id="theming" class="section">
 	<h2 class="inlineblock"><?php p($l->t('Theming')); ?></h2>
-	<a target="_blank" rel="noreferrer" class="icon-info" title="<?php p($l->t('Open documentation'));?>" href="<?php p(link_to_docs('admin-theming')); ?>"></a>
+	<a target="_blank" rel="noreferrer" class="icon-info" title="<?php p($l->t('Open documentation')); ?>" href="<?php p(link_to_docs('admin-theming')); ?>"></a>
         <p class="settings-hint"><?php p($l->t('Theming makes it possible to easily customize the look and feel of your instance and supported clients. This will be visible for all users.')); ?></p>
 		<div id="theming_settings_status">
 			<div id="theming_settings_loading" class="icon-loading-small" style="display: none;"></div>

apps/twofactor_backupcodes/lib/Service/BackupCodeStorage.php

Spacing +3 added lines, -3 removed lines

@@ -105,8 +105,8 @@ 
 		$codes = $this->mapper->getBackupCodes($user);
 		$total = count($codes);
 		$used = 0;
-		array_walk($codes, function (BackupCode $code) use (&$used) {
-			if (1 === (int)$code->getUsed()) {
+		array_walk($codes, function(BackupCode $code) use (&$used) {
+			if (1 === (int) $code->getUsed()) {
 				$used++;
 			}
 		});
@@ -126,7 +126,7 @@ 
 		$dbCodes = $this->mapper->getBackupCodes($user);
 
 		foreach ($dbCodes as $dbCode) {
-			if (0 === (int)$dbCode->getUsed() && $this->hasher->verify($code, $dbCode->getCode())) {
+			if (0 === (int) $dbCode->getUsed() && $this->hasher->verify($code, $dbCode->getCode())) {
 				$dbCode->setUsed(1);
 				$this->mapper->update($dbCode);
 				return true;

apps/files_sharing/lib/Command/ExiprationNotification.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 		//Current time
 		$minTime = $this->time->getDateTime();
 		$minTime->add(new \DateInterval('P1D'));
-		$minTime->setTime(0,0,0);
+		$minTime->setTime(0, 0, 0);
 
 		$maxTime = clone $minTime;
 		$maxTime->setTime(23, 59, 59);

apps/files_sharing/lib/Notification/Notifier.php

Spacing +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 
 		$placeholders = $replacements = [];
 		foreach ($subjectParameters as $placeholder => $parameter) {
-			$placeholders[] = '{' . $placeholder . '}';
+			$placeholders[] = '{'.$placeholder.'}';
 			$replacements[] = $parameter['name'];
 		}
 

core/Command/Group/Add.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
 		$gid = $input->getArgument('groupid');
 		$group = $this->groupManager->get($gid);
 		if ($group) {
-			$output->writeln('<error>Group "' . $gid . '" already exists.</error>');
+			$output->writeln('<error>Group "'.$gid.'" already exists.</error>');
 			return 1;
 		} else {
 			$group = $this->groupManager->createGroup($gid);
@@ -73,9 +73,9 @@ 
 				$output->writeln('<error>Could not create group</error>');
 				return 2;
 			}
-			$output->writeln('Created group "' . $group->getGID() . '"');
+			$output->writeln('Created group "'.$group->getGID().'"');
 
-			$displayName = trim((string)$input->getOption('display-name'));
+			$displayName = trim((string) $input->getOption('display-name'));
 			if ($displayName !== '') {
 				$group->setDisplayName($displayName);
 			}

apps/files/lib/BackgroundJob/TransferOwnership.php

Spacing +5 added lines, -5 removed lines

@@ -98,7 +98,7 @@ 
 		$destinationUserObject = $this->userManager->get($destinationUser);
 
 		if (!$sourceUserObject instanceof IUser) {
-			$this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
+			$this->logger->alert('Could not transfer ownership: Unknown source user '.$sourceUser);
 			$this->failedNotication($transfer);
 			return;
 		}
@@ -136,7 +136,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -149,7 +149,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 
@@ -164,7 +164,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 
 		// Send notification to source user
@@ -177,7 +177,7 @@ 
 				'targetUser' => $transfer->getTargetUser(),
 				'nodeName' => $transfer->getNodeName(),
 			])
-			->setObject('transfer', (string)$transfer->getId());
+			->setObject('transfer', (string) $transfer->getId());
 		$this->notificationManager->notify($notification);
 	}
 }

apps/comments/lib/Search/Result.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 			$suffix = '…';
 		}
 
-		return $prefix . mb_substr($message, $start, $end - $start) . $suffix;
+		return $prefix.mb_substr($message, $start, $end - $start).$suffix;
 	}
 
 }

apps/files/lib/Notification/Notifier.php

Spacing +13 added lines, -13 removed lines

@@ -120,7 +120,7 @@ 
 				$this->urlGenerator->getAbsoluteURL(
 					$this->urlGenerator->linkTo(
 						'',
-						'ocs/v2.php/apps/files/api/v1/transferownership/' . $id
+						'ocs/v2.php/apps/files/api/v1/transferownership/'.$id
 					)
 				),
 				IAction::TYPE_POST
@@ -133,7 +133,7 @@ 
 				$this->urlGenerator->getAbsoluteURL(
 					$this->urlGenerator->linkTo(
 						'',
-						'ocs/v2.php/apps/files/api/v1/transferownership/' . $id
+						'ocs/v2.php/apps/files/api/v1/transferownership/'.$id
 					)
 				),
 				IAction::TYPE_DELETE
@@ -156,7 +156,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					]
 				])
@@ -165,7 +165,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipFailedSource(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipFailedSource(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -177,7 +177,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -190,7 +190,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipFailedTarget(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipFailedTarget(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -202,7 +202,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['sourceUser'] . '::' . $param['nodeName'],
+						'id' => $param['sourceUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -216,7 +216,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipDoneSource(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipDoneSource(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -228,7 +228,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['targetUser'] . '::' . $param['nodeName'],
+						'id' => $param['targetUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -242,7 +242,7 @@ 
 		return $notification;
 	}
 
-	public function handleTransferOwnershipDoneTarget(INotification $notification,  string $languageCode): INotification {
+	public function handleTransferOwnershipDoneTarget(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('files', $languageCode);
 		$param = $notification->getSubjectParameters();
 
@@ -254,7 +254,7 @@ 
 				[
 					'path' => [
 						'type' => 'highlight',
-						'id' => $param['sourceUser'] . '::' . $param['nodeName'],
+						'id' => $param['sourceUser'].'::'.$param['nodeName'],
 						'name' => $param['nodeName'],
 					],
 					'user' => [
@@ -275,7 +275,7 @@ 
 
 		// TODO: This should all be moved to a service that also the transferownershipContoller uses.
 		try {
-			$transferOwnership = $this->mapper->getById((int)$notification->getObjectId());
+			$transferOwnership = $this->mapper->getById((int) $notification->getObjectId());
 		} catch (DoesNotExistException $e) {
 			return;
 		}
@@ -289,7 +289,7 @@ 
 				'targetUser' => $transferOwnership->getTargetUser(),
 				'nodeName' => $transferOwnership->getNodeName()
 			])
-			->setObject('transfer', (string)$transferOwnership->getId());
+			->setObject('transfer', (string) $transferOwnership->getId());
 		$this->notificationManager->notify($notification);
 
 		$this->mapper->delete($transferOwnership);

lib/public/AppFramework/Http/Template/ExternalShareMenuAction.php

Spacing +10 added lines, -10 removed lines

@@ -66,16 +66,16 @@
 	 * @since 14.0.0
 	 */
 	public function render(): string {
-		return '<li>' .
-			'    <button id="save-external-share" class="icon ' . Util::sanitizeHTML($this->getIcon()) . '" data-protected="false" data-owner-display-name="' . Util::sanitizeHTML($this->displayname) . '" data-owner="' . Util::sanitizeHTML($this->owner) . '" data-name="' . Util::sanitizeHTML($this->shareName) . '">' . Util::sanitizeHTML($this->getLabel()) . '</button>' .
-			'</li>' .
-			'<li id="external-share-menu-item" class="hidden">' .
-			'    <span class="menuitem">' .
-			'        <form class="save-form" action="#">' .
-			'            <input type="text" id="remote_address" placeholder="user@yourNextcloud.org">' .
-			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>' .
-			'        </form>' .
-			'    </span>' .
+		return '<li>'.
+			'    <button id="save-external-share" class="icon '.Util::sanitizeHTML($this->getIcon()).'" data-protected="false" data-owner-display-name="'.Util::sanitizeHTML($this->displayname).'" data-owner="'.Util::sanitizeHTML($this->owner).'" data-name="'.Util::sanitizeHTML($this->shareName).'">'.Util::sanitizeHTML($this->getLabel()).'</button>'.
+			'</li>'.
+			'<li id="external-share-menu-item" class="hidden">'.
+			'    <span class="menuitem">'.
+			'        <form class="save-form" action="#">'.
+			'            <input type="text" id="remote_address" placeholder="user@yourNextcloud.org">'.
+			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>'.
+			'        </form>'.
+			'    </span>'.
 			'</li>';
 	}
 }