Passed
Push — master ( 7ef10b...59c1ff )
by Joas
11:15 queued 11s
created
lib/private/Preview/ProviderV1Adapter.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,11 +36,11 @@
 block discarded – undo
36 36
 	}
37 37
 
38 38
 	public function getMimeType(): string {
39
-		return (string)$this->providerV1->getMimeType();
39
+		return (string) $this->providerV1->getMimeType();
40 40
 	}
41 41
 
42 42
 	public function isAvailable(FileInfo $file): bool {
43
-		return (bool)$this->providerV1->isAvailable($file);
43
+		return (bool) $this->providerV1->isAvailable($file);
44 44
 	}
45 45
 
46 46
 	public function getThumbnail(File $file, int $maxX, int $maxY): ?IImage {
Please login to merge, or discard this patch.
lib/private/Security/RateLimiting/Limiter.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
 										string $ip): void {
81 81
 		$ipSubnet = (new IpAddress($ip))->getSubnet();
82 82
 
83
-		$anonHashIdentifier = hash('sha512', 'anon::' . $identifier . $ipSubnet);
83
+		$anonHashIdentifier = hash('sha512', 'anon::'.$identifier.$ipSubnet);
84 84
 		$this->register($identifier, $anonHashIdentifier, $anonPeriod, $anonLimit);
85 85
 	}
86 86
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 										int $userLimit,
98 98
 										int $userPeriod,
99 99
 										IUser $user): void {
100
-		$userHashIdentifier = hash('sha512', 'user::' . $identifier . $user->getUID());
100
+		$userHashIdentifier = hash('sha512', 'user::'.$identifier.$user->getUID());
101 101
 		$this->register($identifier, $userHashIdentifier, $userPeriod, $userLimit);
102 102
 	}
103 103
 }
Please login to merge, or discard this patch.
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.