Passed
Push — master ( 557066...971e22 )
by John
15:32 queued 12s
created
lib/private/Security/IdentityProof/Manager.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 	 */
145 145
 	public function getKey(IUser $user): Key {
146 146
 		$uid = $user->getUID();
147
-		return $this->retrieveKey('user-' . $uid);
147
+		return $this->retrieveKey('user-'.$uid);
148 148
 	}
149 149
 
150 150
 	/**
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
 		if ($instanceId === null) {
159 159
 			throw new \RuntimeException('no instance id!');
160 160
 		}
161
-		return $this->retrieveKey('system-' . $instanceId);
161
+		return $this->retrieveKey('system-'.$instanceId);
162 162
 	}
163 163
 
164 164
 	private function logOpensslError(): void {
@@ -166,6 +166,6 @@  discard block
 block discarded – undo
166 166
 		while ($error = openssl_error_string()) {
167 167
 			$errors[] = $error;
168 168
 		}
169
-		$this->logger->critical('Something is wrong with your openssl setup: ' . implode(', ', $errors));
169
+		$this->logger->critical('Something is wrong with your openssl setup: '.implode(', ', $errors));
170 170
 	}
171 171
 }
Please login to merge, or discard this patch.
lib/private/Security/CSRF/CsrfToken.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
 	public function getEncryptedValue(): string {
58 58
 		if ($this->encryptedValue === '') {
59 59
 			$sharedSecret = random_bytes(\strlen($this->value));
60
-			$this->encryptedValue = base64_encode($this->value ^ $sharedSecret) . ':' . base64_encode($sharedSecret);
60
+			$this->encryptedValue = base64_encode($this->value ^ $sharedSecret).':'.base64_encode($sharedSecret);
61 61
 		}
62 62
 
63 63
 		return $this->encryptedValue;
Please login to merge, or discard this patch.
lib/private/Security/CSP/ContentSecurityPolicyManager.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
 	 */
75 75
 	public function mergePolicies(ContentSecurityPolicy $defaultPolicy,
76 76
 								  EmptyContentSecurityPolicy $originalPolicy): ContentSecurityPolicy {
77
-		foreach ((object)(array)$originalPolicy as $name => $value) {
77
+		foreach ((object) (array) $originalPolicy as $name => $value) {
78 78
 			$setter = 'set'.ucfirst($name);
79 79
 			if (\is_array($value)) {
80 80
 				$getter = 'get'.ucfirst($name);
Please login to merge, or discard this patch.
lib/private/Security/Hasher.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -96,14 +96,14 @@  discard block
 block discarded – undo
96 96
 		$alg = $this->getPrefferedAlgorithm();
97 97
 
98 98
 		if (\defined('PASSWORD_ARGON2ID') && $alg === PASSWORD_ARGON2ID) {
99
-			return 3 . '|' . password_hash($message, PASSWORD_ARGON2ID, $this->options);
99
+			return 3.'|'.password_hash($message, PASSWORD_ARGON2ID, $this->options);
100 100
 		}
101 101
 
102 102
 		if (\defined('PASSWORD_ARGON2I') && $alg === PASSWORD_ARGON2I) {
103
-			return 2 . '|' . password_hash($message, PASSWORD_ARGON2I, $this->options);
103
+			return 2.'|'.password_hash($message, PASSWORD_ARGON2I, $this->options);
104 104
 		}
105 105
 
106
-		return 1 . '|' . password_hash($message, PASSWORD_BCRYPT, $this->options);
106
+		return 1.'|'.password_hash($message, PASSWORD_BCRYPT, $this->options);
107 107
 	}
108 108
 
109 109
 	/**
@@ -114,8 +114,8 @@  discard block
 block discarded – undo
114 114
 	protected function splitHash(string $prefixedHash) {
115 115
 		$explodedString = explode('|', $prefixedHash, 2);
116 116
 		if (\count($explodedString) === 2) {
117
-			if ((int)$explodedString[0] > 0) {
118
-				return ['version' => (int)$explodedString[0], 'hash' => $explodedString[1]];
117
+			if ((int) $explodedString[0] > 0) {
118
+				return ['version' => (int) $explodedString[0], 'hash' => $explodedString[1]];
119 119
 			}
120 120
 		}
121 121
 
Please login to merge, or discard this patch.
lib/private/NavigationManager.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 
121 121
 		$result = $this->entries;
122 122
 		if ($type !== 'all') {
123
-			$result = array_filter($this->entries, function ($entry) use ($type) {
123
+			$result = array_filter($this->entries, function($entry) use ($type) {
124 124
 				return $entry['type'] === $type;
125 125
 			});
126 126
 		}
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
 	 * @return array
136 136
 	 */
137 137
 	private function proceedNavigation(array $list): array {
138
-		uasort($list, function ($a, $b) {
138
+		uasort($list, function($a, $b) {
139 139
 			if (isset($a['order']) && isset($b['order'])) {
140 140
 				return ($a['order'] < $b['order']) ? -1 : 1;
141 141
 			} elseif (isset($a['order']) || isset($b['order'])) {
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
 					continue;
289 289
 				}
290 290
 				$l = $this->l10nFac->get($app);
291
-				$id = $nav['id'] ?? $app . ($key === 0 ? '' : $key);
291
+				$id = $nav['id'] ?? $app.($key === 0 ? '' : $key);
292 292
 				$order = isset($nav['order']) ? $nav['order'] : 100;
293 293
 				$type = isset($nav['type']) ? $nav['type'] : 'link';
294 294
 				$route = $nav['route'] !== '' ? $this->urlGenerator->linkToRoute($nav['route']) : '';
Please login to merge, or discard this patch.
lib/private/Collaboration/Collaborators/MailPlugin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
 						$emailAddressType = $emailAddressData['type'];
100 100
 					}
101 101
 					if (isset($contact['FN'])) {
102
-						$displayName = $contact['FN'] . ' (' . $emailAddress . ')';
102
+						$displayName = $contact['FN'].' ('.$emailAddress.')';
103 103
 					}
104 104
 					$exactEmailMatch = strtolower($emailAddress) === $lowerSearch;
105 105
 
Please login to merge, or discard this patch.
lib/private/Files/ObjectStore/S3Signature.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 		$request = $this->prepareRequest($request, $credentials);
65 65
 		$stringToSign = $this->createCanonicalizedString($request);
66 66
 		$auth = 'AWS '
67
-			. $credentials->getAccessKeyId() . ':'
67
+			. $credentials->getAccessKeyId().':'
68 68
 			. $this->signString($stringToSign, $credentials);
69 69
 
70 70
 		return $request->withHeader('Authorization', $auth);
@@ -148,11 +148,11 @@  discard block
 block discarded – undo
148 148
 		RequestInterface $request,
149 149
 		$expires = null
150 150
 	) {
151
-		$buffer = $request->getMethod() . "\n";
151
+		$buffer = $request->getMethod()."\n";
152 152
 
153 153
 		// Add the interesting headers
154 154
 		foreach ($this->signableHeaders as $header) {
155
-			$buffer .= $request->getHeaderLine($header) . "\n";
155
+			$buffer .= $request->getHeaderLine($header)."\n";
156 156
 		}
157 157
 
158 158
 		$date = $expires ?: $request->getHeaderLine('date');
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 			if (strpos($name, 'x-amz-') === 0) {
171 171
 				$value = implode(',', $header);
172 172
 				if (strlen($value) > 0) {
173
-					$headers[$name] = $name . ':' . $value;
173
+					$headers[$name] = $name.':'.$value;
174 174
 				}
175 175
 			}
176 176
 		}
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
 
182 182
 		ksort($headers);
183 183
 
184
-		return implode("\n", $headers) . "\n";
184
+		return implode("\n", $headers)."\n";
185 185
 	}
186 186
 
187 187
 	private function createCanonicalizedResource(RequestInterface $request) {
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
 		if ($data['bucket']) {
192 192
 			$buffer .= $data['bucket'];
193 193
 			if (!empty($data['key']) || !$data['path_style']) {
194
-				$buffer .= '/' . $data['key'];
194
+				$buffer .= '/'.$data['key'];
195 195
 			}
196 196
 		}
197 197
 
Please login to merge, or discard this patch.
lib/private/Group/Group.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 			return;
174 174
 		}
175 175
 
176
-		$this->dispatcher->dispatch(IGroup::class . '::preAddUser', new GenericEvent($this, [
176
+		$this->dispatcher->dispatch(IGroup::class.'::preAddUser', new GenericEvent($this, [
177 177
 			'user' => $user,
178 178
 		]));
179 179
 
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 					$this->users[$user->getUID()] = $user;
188 188
 				}
189 189
 
190
-				$this->dispatcher->dispatch(IGroup::class . '::postAddUser', new GenericEvent($this, [
190
+				$this->dispatcher->dispatch(IGroup::class.'::postAddUser', new GenericEvent($this, [
191 191
 					'user' => $user,
192 192
 				]));
193 193
 
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 	 */
207 207
 	public function removeUser($user) {
208 208
 		$result = false;
209
-		$this->dispatcher->dispatch(IGroup::class . '::preRemoveUser', new GenericEvent($this, [
209
+		$this->dispatcher->dispatch(IGroup::class.'::preRemoveUser', new GenericEvent($this, [
210 210
 			'user' => $user,
211 211
 		]));
212 212
 		if ($this->emitter) {
@@ -219,7 +219,7 @@  discard block
 block discarded – undo
219 219
 			}
220 220
 		}
221 221
 		if ($result) {
222
-			$this->dispatcher->dispatch(IGroup::class . '::postRemoveUser', new GenericEvent($this, [
222
+			$this->dispatcher->dispatch(IGroup::class.'::postRemoveUser', new GenericEvent($this, [
223 223
 				'user' => $user,
224 224
 			]));
225 225
 			if ($this->emitter) {
@@ -329,7 +329,7 @@  discard block
 block discarded – undo
329 329
 		}
330 330
 
331 331
 		$result = false;
332
-		$this->dispatcher->dispatch(IGroup::class . '::preDelete', new GenericEvent($this));
332
+		$this->dispatcher->dispatch(IGroup::class.'::preDelete', new GenericEvent($this));
333 333
 		if ($this->emitter) {
334 334
 			$this->emitter->emit('\OC\Group', 'preDelete', [$this]);
335 335
 		}
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
 			}
341 341
 		}
342 342
 		if ($result) {
343
-			$this->dispatcher->dispatch(IGroup::class . '::postDelete', new GenericEvent($this));
343
+			$this->dispatcher->dispatch(IGroup::class.'::postDelete', new GenericEvent($this));
344 344
 			if ($this->emitter) {
345 345
 				$this->emitter->emit('\OC\Group', 'postDelete', [$this]);
346 346
 			}
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
 	 * @since 16.0.0
399 399
 	 */
400 400
 	public function hideFromCollaboration(): bool {
401
-		return array_reduce($this->backends, function (bool $hide, GroupInterface $backend) {
401
+		return array_reduce($this->backends, function(bool $hide, GroupInterface $backend) {
402 402
 			return $hide | ($backend instanceof IHideFromCollaborationBackend && $backend->hideGroup($this->gid));
403 403
 		}, false);
404 404
 	}
Please login to merge, or discard this patch.
lib/private/Repair/RemoveLinkShares.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
 
113 113
 		$subQuery = $this->connection->getQueryBuilder();
114 114
 		$subQuery->select('s1.id')
115
-			->from($subQuery->createFunction('(' . $subSubQuery->getSQL() . ')'), 's1')
115
+			->from($subQuery->createFunction('('.$subSubQuery->getSQL().')'), 's1')
116 116
 			->join(
117 117
 				's1', 'share', 's2',
118 118
 				$subQuery->expr()->eq('s1.parent', 's2.id')
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 		$query = $this->connection->getQueryBuilder();
127 127
 		$query->select($query->func()->count('*', 'total'))
128 128
 			->from('share')
129
-			->where($query->expr()->in('id', $query->createFunction('(' . $subQuery->getSQL() . ')')));
129
+			->where($query->expr()->in('id', $query->createFunction('('.$subQuery->getSQL().')')));
130 130
 
131 131
 		$result = $query->execute();
132 132
 		$data = $result->fetch();
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 
150 150
 		$query = $this->connection->getQueryBuilder();
151 151
 		$query->select('s1.id', 's1.uid_owner', 's1.uid_initiator')
152
-			->from($query->createFunction('(' . $subQuery->getSQL() . ')'), 's1')
152
+			->from($query->createFunction('('.$subQuery->getSQL().')'), 's1')
153 153
 			->join(
154 154
 				's1', 'share', 's2',
155 155
 				$query->expr()->eq('s1.parent', 's2.id')
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
 		$this->addToNotify($data['uid_owner']);
174 174
 		$this->addToNotify($data['uid_initiator']);
175 175
 
176
-		$this->deleteShare((int)$id);
176
+		$this->deleteShare((int) $id);
177 177
 	}
178 178
 
179 179
 	/**
Please login to merge, or discard this patch.