Passed
Push — master ( 0ecef7...7d2f5a )
by Blizzz
13:12 queued 11s
created
apps/dav/lib/Migration/RemoveOrphanEventsAndContacts.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -48,23 +48,23 @@  discard block
 block discarded – undo
48 48
 	 * @inheritdoc
49 49
 	 */
50 50
 	public function run(IOutput $output) {
51
-		$orphanItems = $this->removeOrphanChildren('calendarobjects', 'calendars',  'calendarid');
51
+		$orphanItems = $this->removeOrphanChildren('calendarobjects', 'calendars', 'calendarid');
52 52
 		$output->info(sprintf('%d events without a calendar have been cleaned up', $orphanItems));
53
-		$orphanItems = $this->removeOrphanChildren('calendarobjects_props', 'calendarobjects',  'objectid');
53
+		$orphanItems = $this->removeOrphanChildren('calendarobjects_props', 'calendarobjects', 'objectid');
54 54
 		$output->info(sprintf('%d properties without an events have been cleaned up', $orphanItems));
55
-		$orphanItems = $this->removeOrphanChildren('calendarchanges', 'calendars',  'calendarid');
55
+		$orphanItems = $this->removeOrphanChildren('calendarchanges', 'calendars', 'calendarid');
56 56
 		$output->info(sprintf('%d changes without a calendar have been cleaned up', $orphanItems));
57 57
 
58
-		$orphanItems = $this->removeOrphanChildren('calendarobjects', 'calendarsubscriptions',  'calendarid');
58
+		$orphanItems = $this->removeOrphanChildren('calendarobjects', 'calendarsubscriptions', 'calendarid');
59 59
 		$output->info(sprintf('%d cached events without a calendar subscription have been cleaned up', $orphanItems));
60
-		$orphanItems = $this->removeOrphanChildren('calendarchanges', 'calendarsubscriptions',  'calendarid');
60
+		$orphanItems = $this->removeOrphanChildren('calendarchanges', 'calendarsubscriptions', 'calendarid');
61 61
 		$output->info(sprintf('%d changes without a calendar subscription have been cleaned up', $orphanItems));
62 62
 
63
-		$orphanItems = $this->removeOrphanChildren('cards', 'addressbooks',  'addressbookid');
63
+		$orphanItems = $this->removeOrphanChildren('cards', 'addressbooks', 'addressbookid');
64 64
 		$output->info(sprintf('%d contacts without an addressbook have been cleaned up', $orphanItems));
65
-		$orphanItems = $this->removeOrphanChildren('cards_properties', 'cards',  'cardid');
65
+		$orphanItems = $this->removeOrphanChildren('cards_properties', 'cards', 'cardid');
66 66
 		$output->info(sprintf('%d properties without a contact have been cleaned up', $orphanItems));
67
-		$orphanItems = $this->removeOrphanChildren('addressbookchanges', 'addressbooks',  'addressbookid');
67
+		$orphanItems = $this->removeOrphanChildren('addressbookchanges', 'addressbooks', 'addressbookid');
68 68
 		$output->info(sprintf('%d changes without an addressbook have been cleaned up', $orphanItems));
69 69
 	}
70 70
 
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 
74 74
 		$qb->select('c.id')
75 75
 			->from($childTable, 'c')
76
-			->leftJoin('c', $parentTable, 'p', $qb->expr()->eq('c.' . $parentId, 'p.id'))
76
+			->leftJoin('c', $parentTable, 'p', $qb->expr()->eq('c.'.$parentId, 'p.id'))
77 77
 			->where($qb->expr()->isNull('p.id'));
78 78
 
79 79
 		if (\in_array($parentTable, ['calendars', 'calendarsubscriptions'], true)) {
Please login to merge, or discard this patch.
lib/private/Files/Storage/Wrapper/Jail.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -58,12 +58,12 @@
 block discarded – undo
58 58
 		if ($path === '') {
59 59
 			return $this->rootPath;
60 60
 		} else {
61
-			return Filesystem::normalizePath($this->rootPath . '/' . $path);
61
+			return Filesystem::normalizePath($this->rootPath.'/'.$path);
62 62
 		}
63 63
 	}
64 64
 
65 65
 	public function getJailedPath($path) {
66
-		$root = rtrim($this->rootPath, '/') . '/';
66
+		$root = rtrim($this->rootPath, '/').'/';
67 67
 
68 68
 		if ($path !== $this->rootPath && strpos($path, $root) !== 0) {
69 69
 			return null;
Please login to merge, or discard this patch.
apps/files_trashbin/lib/Storage.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 		} catch (GenericEncryptionException $e) {
100 100
 			// in case of a encryption exception we delete the file right away
101 101
 			$this->logger->info(
102
-				"Can't move file" . $path .
102
+				"Can't move file".$path.
103 103
 				"to the trash bin, therefore it was deleted right away");
104 104
 
105 105
 			return $this->storage->unlink($path);
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 	 * @return bool
126 126
 	 */
127 127
 	protected function shouldMoveToTrash($path) {
128
-		$normalized = Filesystem::normalizePath($this->mountPoint . '/' . $path);
128
+		$normalized = Filesystem::normalizePath($this->mountPoint.'/'.$path);
129 129
 		$parts = explode('/', $normalized);
130 130
 		if (count($parts) < 4) {
131 131
 			return false;
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
 	 * Setup the storate wrapper callback
201 201
 	 */
202 202
 	public static function setupStorage() {
203
-		\OC\Files\Filesystem::addStorageWrapper('oc_trashbin', function ($mountPoint, $storage) {
203
+		\OC\Files\Filesystem::addStorageWrapper('oc_trashbin', function($mountPoint, $storage) {
204 204
 			return new \OCA\Files_Trashbin\Storage(
205 205
 				['storage' => $storage, 'mountPoint' => $mountPoint],
206 206
 				\OC::$server->query(ITrashManager::class),
Please login to merge, or discard this patch.
apps/workflowengine/lib/Check/FileSystemTags.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.
lib/private/Files/Node/Node.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 		$dispatcher = \OC::$server->getEventDispatcher();
111 111
 		foreach ($hooks as $hook) {
112 112
 			$this->root->emit('\OC\Files', $hook, $args);
113
-			$dispatcher->dispatch('\OCP\Files::' . $hook, new GenericEvent($args));
113
+			$dispatcher->dispatch('\OCP\Files::'.$hook, new GenericEvent($args));
114 114
 		}
115 115
 	}
116 116
 
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
 		$path = str_replace('\\', '/', $path);
298 298
 		//add leading slash
299 299
 		if ($path[0] !== '/') {
300
-			$path = '/' . $path;
300
+			$path = '/'.$path;
301 301
 		}
302 302
 		//remove duplicate slashes
303 303
 		while (strpos($path, '//') !== false) {
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
 	 */
318 318
 	public function isValidPath($path) {
319 319
 		if (!$path || $path[0] !== '/') {
320
-			$path = '/' . $path;
320
+			$path = '/'.$path;
321 321
 		}
322 322
 		if (strstr($path, '/../') || strrchr($path, '/') === '/..') {
323 323
 			return false;
@@ -401,14 +401,14 @@  discard block
 block discarded – undo
401 401
 			$this->sendHooks(['preCopy'], [$this, $nonExisting]);
402 402
 			$this->sendHooks(['preWrite'], [$nonExisting]);
403 403
 			if (!$this->view->copy($this->path, $targetPath)) {
404
-				throw new NotPermittedException('Could not copy ' . $this->path . ' to ' . $targetPath);
404
+				throw new NotPermittedException('Could not copy '.$this->path.' to '.$targetPath);
405 405
 			}
406 406
 			$targetNode = $this->root->get($targetPath);
407 407
 			$this->sendHooks(['postCopy'], [$this, $targetNode]);
408 408
 			$this->sendHooks(['postWrite'], [$targetNode]);
409 409
 			return $targetNode;
410 410
 		} else {
411
-			throw new NotPermittedException('No permission to copy to path ' . $targetPath);
411
+			throw new NotPermittedException('No permission to copy to path '.$targetPath);
412 412
 		}
413 413
 	}
414 414
 
@@ -432,7 +432,7 @@  discard block
 block discarded – undo
432 432
 			$this->sendHooks(['preRename'], [$this, $nonExisting]);
433 433
 			$this->sendHooks(['preWrite'], [$nonExisting]);
434 434
 			if (!$this->view->rename($this->path, $targetPath)) {
435
-				throw new NotPermittedException('Could not move ' . $this->path . ' to ' . $targetPath);
435
+				throw new NotPermittedException('Could not move '.$this->path.' to '.$targetPath);
436 436
 			}
437 437
 			$targetNode = $this->root->get($targetPath);
438 438
 			$this->sendHooks(['postRename'], [$this, $targetNode]);
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
 			$this->path = $targetPath;
441 441
 			return $targetNode;
442 442
 		} else {
443
-			throw new NotPermittedException('No permission to move to path ' . $targetPath);
443
+			throw new NotPermittedException('No permission to move to path '.$targetPath);
444 444
 		}
445 445
 	}
446 446
 
Please login to merge, or discard this patch.
apps/workflowengine/lib/Check/FileSize.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/workflowengine/lib/Check/FileName.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/encryption/lib/Hooks/UserHooks.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 			// Save private key
267 267
 			if ($encryptedPrivateKey) {
268 268
 				$this->keyManager->setPrivateKey($user->getUID(),
269
-					$this->crypt->generateHeader() . $encryptedPrivateKey);
269
+					$this->crypt->generateHeader().$encryptedPrivateKey);
270 270
 			} else {
271 271
 				$this->logger->error('Encryption could not update users encryption password');
272 272
 			}
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
 				$encryptedKey = $this->crypt->encryptPrivateKey($keyPair['privateKey'], $newUserPassword, $userId);
316 316
 
317 317
 				if ($encryptedKey) {
318
-					$this->keyManager->setPrivateKey($userId, $this->crypt->generateHeader() . $encryptedKey);
318
+					$this->keyManager->setPrivateKey($userId, $this->crypt->generateHeader().$encryptedKey);
319 319
 
320 320
 					if ($recoveryPassword) { // if recovery key is set we can re-encrypt the key files
321 321
 						$this->recovery->recoverUsersFiles($recoveryPassword, $userId);
Please login to merge, or discard this patch.
apps/settings/templates/settings/admin/additional-mail.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -56,9 +56,9 @@  discard block
 block discarded – undo
56 56
 
57 57
 <div class="section" id="mail_general_settings">
58 58
 	<form id="mail_general_settings_form" class="mail_settings">
59
-		<h2><?php p($l->t('Email server'));?></h2>
59
+		<h2><?php p($l->t('Email server')); ?></h2>
60 60
 		<a target="_blank" rel="noreferrer noopener" class="icon-info"
61
-		   title="<?php p($l->t('Open documentation'));?>"
61
+		   title="<?php p($l->t('Open documentation')); ?>"
62 62
 		   href="<?php p(link_to_docs('admin-email')); ?>"></a>
63 63
 		<p class="settings-hint"><?php p($l->t('It is important to set up this server to be able to send emails, like for password reset and notifications.')); ?></p>
64 64
 		<p><span id="mail_settings_msg" class="msg"></span></p>
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 						$selected = 'selected="selected"';
73 73
 					endif; ?>
74 74
 					<option value="<?php p($smtpmode[0])?>" <?php p($selected) ?>><?php p($smtpmode[1]) ?></option>
75
-				<?php endforeach;?>
75
+				<?php endforeach; ?>
76 76
 			</select>
77 77
 
78 78
 			<label id="mail_smtpsecure_label" for="mail_smtpsecure"
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 						$selected = 'selected="selected"';
88 88
 					endif; ?>
89 89
 					<option value="<?php p($secure)?>" <?php p($selected) ?>><?php p($name) ?></option>
90
-				<?php endforeach;?>
90
+				<?php endforeach; ?>
91 91
 			</select>
92 92
 
93 93
 			<label id="mail_sendmailmode_label" for="mail_sendmailmode" class="<?= $_['mail_smtpmode'] !== 'sendmail' ? 'hidden' : '' ?>">
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 			<select name="mail_sendmailmode" id="mail_sendmailmode" class="<?= $_['mail_smtpmode'] !== 'sendmail' ? 'hidden' : '' ?>">
97 97
 				<?php foreach ($mail_sendmailmode as $sendmailmodeValue => $sendmailmodeLabel): ?>
98 98
 					<option value="<?php p($sendmailmodeValue)?>" <?= $sendmailmodeValue === $_['mail_sendmailmode'] ? 'selected="selected"' : '' ?>><?php p($sendmailmodeLabel) ?></option>
99
-				<?php endforeach;?>
99
+				<?php endforeach; ?>
100 100
 			</select>
101 101
 		</p>
102 102
 
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 						$selected = 'selected="selected"';
118 118
 					endif; ?>
119 119
 					<option value="<?php p($authtype)?>" <?php p($selected) ?>><?php p($name) ?></option>
120
-				<?php endforeach;?>
120
+				<?php endforeach; ?>
121 121
 			</select>
122 122
 
123 123
 			<input type="checkbox" name="mail_smtpauth" id="mail_smtpauth" class="checkbox" value="1"
Please login to merge, or discard this patch.