Passed
Push — master ( c78bd4...b125e1 )
by Joas
13:11 queued 11s
created
core/Command/Group/Add.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 		$gid = $input->getArgument('groupid');
66 66
 		$group = $this->groupManager->get($gid);
67 67
 		if ($group) {
68
-			$output->writeln('<error>Group "' . $gid . '" already exists.</error>');
68
+			$output->writeln('<error>Group "'.$gid.'" already exists.</error>');
69 69
 			return 1;
70 70
 		} else {
71 71
 			$group = $this->groupManager->createGroup($gid);
@@ -73,9 +73,9 @@  discard block
 block discarded – undo
73 73
 				$output->writeln('<error>Could not create group</error>');
74 74
 				return 2;
75 75
 			}
76
-			$output->writeln('Created group "' . $group->getGID() . '"');
76
+			$output->writeln('Created group "'.$group->getGID().'"');
77 77
 
78
-			$displayName = trim((string)$input->getOption('display-name'));
78
+			$displayName = trim((string) $input->getOption('display-name'));
79 79
 			if ($displayName !== '') {
80 80
 				$group->setDisplayName($displayName);
81 81
 			}
Please login to merge, or discard this patch.
apps/files/lib/BackgroundJob/TransferOwnership.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 		$destinationUserObject = $this->userManager->get($destinationUser);
99 99
 
100 100
 		if (!$sourceUserObject instanceof IUser) {
101
-			$this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser);
101
+			$this->logger->alert('Could not transfer ownership: Unknown source user '.$sourceUser);
102 102
 			$this->failedNotication($transfer);
103 103
 			return;
104 104
 		}
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
 				'targetUser' => $transfer->getTargetUser(),
137 137
 				'nodeName' => $transfer->getNodeName(),
138 138
 			])
139
-			->setObject('transfer', (string)$transfer->getId());
139
+			->setObject('transfer', (string) $transfer->getId());
140 140
 		$this->notificationManager->notify($notification);
141 141
 
142 142
 		// Send notification to source user
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 				'targetUser' => $transfer->getTargetUser(),
150 150
 				'nodeName' => $transfer->getNodeName(),
151 151
 			])
152
-			->setObject('transfer', (string)$transfer->getId());
152
+			->setObject('transfer', (string) $transfer->getId());
153 153
 		$this->notificationManager->notify($notification);
154 154
 	}
155 155
 
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 				'targetUser' => $transfer->getTargetUser(),
165 165
 				'nodeName' => $transfer->getNodeName(),
166 166
 			])
167
-			->setObject('transfer', (string)$transfer->getId());
167
+			->setObject('transfer', (string) $transfer->getId());
168 168
 		$this->notificationManager->notify($notification);
169 169
 
170 170
 		// Send notification to source user
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
 				'targetUser' => $transfer->getTargetUser(),
178 178
 				'nodeName' => $transfer->getNodeName(),
179 179
 			])
180
-			->setObject('transfer', (string)$transfer->getId());
180
+			->setObject('transfer', (string) $transfer->getId());
181 181
 		$this->notificationManager->notify($notification);
182 182
 	}
183 183
 }
Please login to merge, or discard this patch.
apps/comments/lib/Search/Result.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -105,7 +105,7 @@
 block discarded – undo
105 105
 			$suffix = '…';
106 106
 		}
107 107
 
108
-		return $prefix . mb_substr($message, $start, $end - $start) . $suffix;
108
+		return $prefix.mb_substr($message, $start, $end - $start).$suffix;
109 109
 	}
110 110
 
111 111
 }
Please login to merge, or discard this patch.
lib/public/AppFramework/Http/Template/ExternalShareMenuAction.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -66,16 +66,16 @@
 block discarded – undo
66 66
 	 * @since 14.0.0
67 67
 	 */
68 68
 	public function render(): string {
69
-		return '<li>' .
70
-			'    <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>' .
71
-			'</li>' .
72
-			'<li id="external-share-menu-item" class="hidden">' .
73
-			'    <span class="menuitem">' .
74
-			'        <form class="save-form" action="#">' .
75
-			'            <input type="text" id="remote_address" placeholder="[email protected]">' .
76
-			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>' .
77
-			'        </form>' .
78
-			'    </span>' .
69
+		return '<li>'.
70
+			'    <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>'.
71
+			'</li>'.
72
+			'<li id="external-share-menu-item" class="hidden">'.
73
+			'    <span class="menuitem">'.
74
+			'        <form class="save-form" action="#">'.
75
+			'            <input type="text" id="remote_address" placeholder="[email protected]">'.
76
+			'            <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>'.
77
+			'        </form>'.
78
+			'    </span>'.
79 79
 			'</li>';
80 80
 	}
81 81
 }
Please login to merge, or discard this patch.
apps/workflowengine/lib/Check/FileMimeType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -155,7 +155,7 @@
 block discarded – undo
155 155
 	}
156 156
 
157 157
 	public function supportedEntities(): array {
158
-		return [ File::class ];
158
+		return [File::class];
159 159
 	}
160 160
 
161 161
 	public function isAvailableForScope(int $scope): bool {
Please login to merge, or discard this patch.
apps/settings/lib/Settings/Personal/Security/TwoFactor.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -106,12 +106,12 @@
 block discarded – undo
106 106
 		}
107 107
 
108 108
 		return [
109
-			'providers' => array_map(function (IProvidesPersonalSettings $provider) use ($user) {
109
+			'providers' => array_map(function(IProvidesPersonalSettings $provider) use ($user) {
110 110
 				return [
111 111
 					'provider' => $provider,
112 112
 					'settings' => $provider->getPersonalSettings($user)
113 113
 				];
114
-			}, array_filter($this->providerLoader->getProviders($user), function (IProvider $provider) {
114
+			}, array_filter($this->providerLoader->getProviders($user), function(IProvider $provider) {
115 115
 				return $provider instanceof IProvidesPersonalSettings;
116 116
 			}))
117 117
 		];
Please login to merge, or discard this patch.
lib/private/BackgroundJob/Job.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -57,17 +57,17 @@
 block discarded – undo
57 57
 
58 58
 		try {
59 59
 			$jobStartTime = time();
60
-			$logger->debug('Run ' . get_class($this) . ' job with ID ' . $this->getId(), ['app' => 'cron']);
60
+			$logger->debug('Run '.get_class($this).' job with ID '.$this->getId(), ['app' => 'cron']);
61 61
 			$this->run($this->argument);
62 62
 			$timeTaken = time() - $jobStartTime;
63 63
 
64
-			$logger->debug('Finished ' . get_class($this) . ' job with ID ' . $this->getId() . ' in ' . $timeTaken . ' seconds', ['app' => 'cron']);
64
+			$logger->debug('Finished '.get_class($this).' job with ID '.$this->getId().' in '.$timeTaken.' seconds', ['app' => 'cron']);
65 65
 			$jobList->setExecutionTime($this, $timeTaken);
66 66
 		} catch (\Throwable $e) {
67 67
 			if ($logger) {
68 68
 				$logger->logException($e, [
69 69
 					'app' => 'core',
70
-					'message' => 'Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . ')'
70
+					'message' => 'Error while running background job (class: '.get_class($this).', arguments: '.print_r($this->argument, true).')'
71 71
 				]);
72 72
 			}
73 73
 		}
Please login to merge, or discard this patch.
apps/files/lib/Controller/TransferOwnershipController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 				'targetUser' => $recipient,
113 113
 				'nodeName' => $node->getName(),
114 114
 			])
115
-			->setObject('transfer', (string)$transferOwnership->getId());
115
+			->setObject('transfer', (string) $transferOwnership->getId());
116 116
 
117 117
 		$this->notificationManager->notify($notification);
118 118
 
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
 
136 136
 		$notification = $this->notificationManager->createNotification();
137 137
 		$notification->setApp('files')
138
-			->setObject('transfer', (string)$id);
138
+			->setObject('transfer', (string) $id);
139 139
 		$this->notificationManager->markProcessed($notification);
140 140
 
141 141
 		$newTransferOwnership = new TransferOwnershipEntity();
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
 
169 169
 		$notification = $this->notificationManager->createNotification();
170 170
 		$notification->setApp('files')
171
-			->setObject('transfer', (string)$id);
171
+			->setObject('transfer', (string) $id);
172 172
 		$this->notificationManager->markProcessed($notification);
173 173
 
174 174
 		$notification = $this->notificationManager->createNotification();
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 				'targetUser' => $transferOwnership->getTargetUser(),
181 181
 				'nodeName' => $transferOwnership->getNodeName()
182 182
 			])
183
-			->setObject('transfer', (string)$transferOwnership->getId());
183
+			->setObject('transfer', (string) $transferOwnership->getId());
184 184
 		$this->notificationManager->notify($notification);
185 185
 
186 186
 		$this->mapper->delete($transferOwnership);
Please login to merge, or discard this patch.
lib/private/Files/SimpleFS/NewSimpleFile.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -212,7 +212,7 @@
 block discarded – undo
212 212
 			return $this->file->fopen('w');
213 213
 		} else {
214 214
 			$source = fopen('php://temp', 'w+');
215
-			return CallbackWrapper::wrap($source, null, null, null, null, function () use ($source) {
215
+			return CallbackWrapper::wrap($source, null, null, null, null, function() use ($source) {
216 216
 				rewind($source);
217 217
 				$this->putContent($source);
218 218
 			});
Please login to merge, or discard this patch.