Passed
Push — master ( bed972...c9ea23 )
by Joas
15:02 queued 16s
created
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.
apps/files/lib/Activity/Provider.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -124,9 +124,9 @@  discard block
 block discarded – undo
124 124
 
125 125
 	protected function setIcon(IEvent $event, string $icon, string $app = 'files') {
126 126
 		if ($this->activityManager->getRequirePNG()) {
127
-			$event->setIcon($this->url->getAbsoluteURL($this->url->imagePath($app, $icon . '.png')));
127
+			$event->setIcon($this->url->getAbsoluteURL($this->url->imagePath($app, $icon.'.png')));
128 128
 		} else {
129
-			$event->setIcon($this->url->getAbsoluteURL($this->url->imagePath($app, $icon . '.svg')));
129
+			$event->setIcon($this->url->getAbsoluteURL($this->url->imagePath($app, $icon.'.svg')));
130 130
 		}
131 131
 	}
132 132
 
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
 		}
246 246
 
247 247
 		if ($this->fileIsEncrypted) {
248
-			$event->setSubject($event->getSubject() . '_enc', $event->getSubjectParameters());
248
+			$event->setSubject($event->getSubject().'_enc', $event->getSubjectParameters());
249 249
 		}
250 250
 
251 251
 		if (!isset($parsedParameters['user'])) {
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
 	protected function setSubjects(IEvent $event, $subject, array $parameters) {
273 273
 		$placeholders = $replacements = [];
274 274
 		foreach ($parameters as $placeholder => $parameter) {
275
-			$placeholders[] = '{' . $placeholder . '}';
275
+			$placeholders[] = '{'.$placeholder.'}';
276 276
 			if ($parameter['type'] === 'file') {
277 277
 				$replacements[] = $parameter['path'];
278 278
 			} else {
@@ -538,7 +538,7 @@  discard block
 block discarded – undo
538 538
 				$lowerSearch = strtolower($search);
539 539
 				foreach ($cloudIds as $cloudId) {
540 540
 					if (strtolower($cloudId) === $lowerSearch) {
541
-						$this->displayNames[$search] = $contact['FN'] . " ($cloudId)";
541
+						$this->displayNames[$search] = $contact['FN']." ($cloudId)";
542 542
 						return $this->displayNames[$search];
543 543
 					}
544 544
 				}
Please login to merge, or discard this patch.
apps/files_sharing/lib/Activity/Providers/Base.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 	protected function setSubjects(IEvent $event, $subject, array $parameters) {
126 126
 		$placeholders = $replacements = [];
127 127
 		foreach ($parameters as $placeholder => $parameter) {
128
-			$placeholders[] = '{' . $placeholder . '}';
128
+			$placeholders[] = '{'.$placeholder.'}';
129 129
 			if ($parameter['type'] === 'file') {
130 130
 				$replacements[] = $parameter['path'];
131 131
 			} else {
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
 				$lowerSearch = strtolower($search);
219 219
 				foreach ($cloudIds as $cloudId) {
220 220
 					if (strtolower($cloudId) === $lowerSearch) {
221
-						$this->displayNames[$search] = $contact['FN'] . " ($cloudId)";
221
+						$this->displayNames[$search] = $contact['FN']." ($cloudId)";
222 222
 						return $this->displayNames[$search];
223 223
 					}
224 224
 				}
Please login to merge, or discard this patch.
apps/files_sharing/lib/Controller/RemoteController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 			return new DataResponse();
84 84
 		}
85 85
 
86
-		$this->logger->error('Could not accept federated share with id: ' . $id,
86
+		$this->logger->error('Could not accept federated share with id: '.$id,
87 87
 			['app' => 'files_sharing']);
88 88
 
89 89
 		throw new OCSNotFoundException('wrong share ID, share doesn\'t exist.');
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 	 * @return array enriched share info with data from the filecache
115 115
 	 */
116 116
 	private static function extendShareInfo($share) {
117
-		$view = new \OC\Files\View('/' . \OC_User::getUser() . '/files/');
117
+		$view = new \OC\Files\View('/'.\OC_User::getUser().'/files/');
118 118
 		$info = $view->getFileInfo($share['mountpoint']);
119 119
 
120 120
 		if ($info === false) {
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 			throw new OCSNotFoundException('Share does not exist');
182 182
 		}
183 183
 
184
-		$mountPoint = '/' . \OC_User::getUser() . '/files' . $shareInfo['mountpoint'];
184
+		$mountPoint = '/'.\OC_User::getUser().'/files'.$shareInfo['mountpoint'];
185 185
 
186 186
 		if ($this->externalManager->removeShare($mountPoint) === true) {
187 187
 			return new DataResponse();
Please login to merge, or discard this patch.