Passed
Push — master ( b794fa...797527 )
by Christoph
82:28 queued 66:03
created
lib/private/Authentication/Token/RemoteWipe.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 		$tokens = $this->tokenProvider->getTokenByUser($user->getUID());
86 86
 
87 87
 		/** @var IWipeableToken[] $wipeable */
88
-		$wipeable = array_filter($tokens, function (IToken $token) {
88
+		$wipeable = array_filter($tokens, function(IToken $token) {
89 89
 			return $token instanceof IWipeableToken;
90 90
 		});
91 91
 
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
 		$dbToken = $e->getToken();
123 123
 
124
-		$this->logger->info("user " . $dbToken->getUID() . " started a remote wipe");
124
+		$this->logger->info("user ".$dbToken->getUID()." started a remote wipe");
125 125
 
126 126
 		$this->eventDispatcher->dispatch(RemoteWipeStarted::class, new RemoteWipeStarted($dbToken));
127 127
 
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 
150 150
 		$this->tokenProvider->invalidateToken($token);
151 151
 
152
-		$this->logger->info("user " . $dbToken->getUID() . " finished a remote wipe");
152
+		$this->logger->info("user ".$dbToken->getUID()." finished a remote wipe");
153 153
 		$this->eventDispatcher->dispatch(RemoteWipeFinished::class, new RemoteWipeFinished($dbToken));
154 154
 
155 155
 		return true;
Please login to merge, or discard this patch.
lib/private/Notification/Manager.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 	public function registerNotifier(\Closure $service, \Closure $info) {
85 85
 		$infoData = $info();
86 86
 		$this->logger->logException(new \InvalidArgumentException(
87
-			'Notifier ' . $infoData['name'] . ' (id: ' . $infoData['id'] . ') is not considered because it is using the old way to register.'
87
+			'Notifier '.$infoData['name'].' (id: '.$infoData['id'].') is not considered because it is using the old way to register.'
88 88
 		));
89 89
 	}
90 90
 
@@ -110,14 +110,14 @@  discard block
 block discarded – undo
110 110
 				$app = \OC::$server->query($appClass);
111 111
 			} catch (QueryException $e) {
112 112
 				$this->logger->logException($e, [
113
-					'message' => 'Failed to load notification app class: ' . $appClass,
113
+					'message' => 'Failed to load notification app class: '.$appClass,
114 114
 					'app' => 'notifications',
115 115
 				]);
116 116
 				continue;
117 117
 			}
118 118
 
119 119
 			if (!($app instanceof IApp)) {
120
-				$this->logger->error('Notification app class ' . $appClass . ' is not implementing ' . IApp::class, [
120
+				$this->logger->error('Notification app class '.$appClass.' is not implementing '.IApp::class, [
121 121
 					'app' => 'notifications',
122 122
 				]);
123 123
 				continue;
@@ -144,14 +144,14 @@  discard block
 block discarded – undo
144 144
 				$notifier = \OC::$server->query($notifierClass);
145 145
 			} catch (QueryException $e) {
146 146
 				$this->logger->logException($e, [
147
-					'message' => 'Failed to load notification notifier class: ' . $notifierClass,
147
+					'message' => 'Failed to load notification notifier class: '.$notifierClass,
148 148
 					'app' => 'notifications',
149 149
 				]);
150 150
 				continue;
151 151
 			}
152 152
 
153 153
 			if (!($notifier instanceof INotifier)) {
154
-				$this->logger->error('Notification notifier class ' . $notifierClass . ' is not implementing ' . INotifier::class, [
154
+				$this->logger->error('Notification notifier class '.$notifierClass.' is not implementing '.INotifier::class, [
155 155
 					'app' => 'notifications',
156 156
 				]);
157 157
 				continue;
Please login to merge, or discard this patch.
lib/public/Calendar/IMetadataProvider.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,5 +61,5 @@
 block discarded – undo
61 61
 	 * @return string|null - The value stored for the key, null if no value stored
62 62
 	 * @since 17.0.0
63 63
 	 */
64
-	public function getMetadataForKey(string $key):?string;
64
+	public function getMetadataForKey(string $key): ?string;
65 65
 }
Please login to merge, or discard this patch.
lib/private/Security/FeaturePolicy/FeaturePolicyManager.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -60,10 +60,10 @@
 block discarded – undo
60 60
 	 */
61 61
 	public function mergePolicies(FeaturePolicy $defaultPolicy,
62 62
 								  EmptyFeaturePolicy $originalPolicy): FeaturePolicy {
63
-		foreach ((object)(array)$originalPolicy as $name => $value) {
64
-			$setter = 'set' . ucfirst($name);
63
+		foreach ((object) (array) $originalPolicy as $name => $value) {
64
+			$setter = 'set'.ucfirst($name);
65 65
 			if (\is_array($value)) {
66
-				$getter = 'get' . ucfirst($name);
66
+				$getter = 'get'.ucfirst($name);
67 67
 				$currentValues = \is_array($defaultPolicy->$getter()) ? $defaultPolicy->$getter() : [];
68 68
 				$defaultPolicy->$setter(\array_values(\array_unique(\array_merge($currentValues, $value))));
69 69
 			} elseif (\is_bool($value)) {
Please login to merge, or discard this patch.
lib/public/AppFramework/Http/EmptyFeaturePolicy.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -139,42 +139,42 @@
 block discarded – undo
139 139
 		if (empty($this->autoplayDomains)) {
140 140
 			$policy .= "autoplay 'none';";
141 141
 		} else {
142
-			$policy .= 'autoplay ' . implode(' ', $this->autoplayDomains);
142
+			$policy .= 'autoplay '.implode(' ', $this->autoplayDomains);
143 143
 			$policy .= ';';
144 144
 		}
145 145
 
146 146
 		if (empty($this->cameraDomains)) {
147 147
 			$policy .= "camera 'none';";
148 148
 		} else {
149
-			$policy .= 'camera ' . implode(' ', $this->cameraDomains);
149
+			$policy .= 'camera '.implode(' ', $this->cameraDomains);
150 150
 			$policy .= ';';
151 151
 		}
152 152
 
153 153
 		if (empty($this->fullscreenDomains)) {
154 154
 			$policy .= "fullscreen 'none';";
155 155
 		} else {
156
-			$policy .= 'fullscreen ' . implode(' ', $this->fullscreenDomains);
156
+			$policy .= 'fullscreen '.implode(' ', $this->fullscreenDomains);
157 157
 			$policy .= ';';
158 158
 		}
159 159
 
160 160
 		if (empty($this->geolocationDomains)) {
161 161
 			$policy .= "geolocation 'none';";
162 162
 		} else {
163
-			$policy .= 'geolocation ' . implode(' ', $this->geolocationDomains);
163
+			$policy .= 'geolocation '.implode(' ', $this->geolocationDomains);
164 164
 			$policy .= ';';
165 165
 		}
166 166
 
167 167
 		if (empty($this->microphoneDomains)) {
168 168
 			$policy .= "microphone 'none';";
169 169
 		} else {
170
-			$policy .= 'microphone ' . implode(' ', $this->microphoneDomains);
170
+			$policy .= 'microphone '.implode(' ', $this->microphoneDomains);
171 171
 			$policy .= ';';
172 172
 		}
173 173
 
174 174
 		if (empty($this->paymentDomains)) {
175 175
 			$policy .= "payment 'none';";
176 176
 		} else {
177
-			$policy .= 'payment ' . implode(' ', $this->paymentDomains);
177
+			$policy .= 'payment '.implode(' ', $this->paymentDomains);
178 178
 			$policy .= ';';
179 179
 		}
180 180
 
Please login to merge, or discard this patch.
apps/dav/lib/Migration/Version1012Date20190808122342.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
 	 */
43 43
 	public function changeSchema(IOutput $output,
44 44
 								 \Closure $schemaClosure,
45
-								 array $options):?ISchemaWrapper {
45
+								 array $options): ?ISchemaWrapper {
46 46
 		/** @var ISchemaWrapper $schema */
47 47
 		$schema = $schemaClosure();
48 48
 
Please login to merge, or discard this patch.
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.