Completed
Pull Request — master (#4212)
by Individual IT
13:52
created
apps/files_external/lib/Service/GlobalStoragesService.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -172,11 +172,11 @@
 block discarded – undo
172 172
 	public function getStorageForAllUsers() {
173 173
 		$mounts = $this->dbConfig->getAllMounts();
174 174
 		$configs = array_map([$this, 'getStorageConfigFromDBMount'], $mounts);
175
-		$configs = array_filter($configs, function ($config) {
175
+		$configs = array_filter($configs, function($config) {
176 176
 			return $config instanceof StorageConfig;
177 177
 		});
178 178
 
179
-		$keys = array_map(function (StorageConfig $config) {
179
+		$keys = array_map(function(StorageConfig $config) {
180 180
 			return $config->getId();
181 181
 		}, $configs);
182 182
 
Please login to merge, or discard this patch.
apps/files_external/lib/Service/DBConfigService.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
 	 */
179 179
 	public function getAdminMountsForMultiple($type, array $values) {
180 180
 		$builder = $this->connection->getQueryBuilder();
181
-		$params = array_map(function ($value) use ($builder) {
181
+		$params = array_map(function($value) use ($builder) {
182 182
 			return $builder->createNamedParameter($value, IQueryBuilder::PARAM_STR);
183 183
 		}, $values);
184 184
 
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
 				'type' => $builder->createNamedParameter($type, IQueryBuilder::PARAM_INT)
232 232
 			]);
233 233
 		$query->execute();
234
-		return (int)$this->connection->lastInsertId('*PREFIX*external_mounts');
234
+		return (int) $this->connection->lastInsertId('*PREFIX*external_mounts');
235 235
 	}
236 236
 
237 237
 	/**
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
 		}
368 368
 		$uniqueMounts = array_values($uniqueMounts);
369 369
 
370
-		$mountIds = array_map(function ($mount) {
370
+		$mountIds = array_map(function($mount) {
371 371
 			return $mount['mount_id'];
372 372
 		}, $uniqueMounts);
373 373
 		$mountIds = array_values(array_unique($mountIds));
@@ -376,9 +376,9 @@  discard block
 block discarded – undo
376 376
 		$config = $this->getConfigForMounts($mountIds);
377 377
 		$options = $this->getOptionsForMounts($mountIds);
378 378
 
379
-		return array_map(function ($mount, $applicable, $config, $options) {
380
-			$mount['type'] = (int)$mount['type'];
381
-			$mount['priority'] = (int)$mount['priority'];
379
+		return array_map(function($mount, $applicable, $config, $options) {
380
+			$mount['type'] = (int) $mount['type'];
381
+			$mount['priority'] = (int) $mount['priority'];
382 382
 			$mount['applicable'] = $applicable;
383 383
 			$mount['config'] = $config;
384 384
 			$mount['options'] = $options;
@@ -400,7 +400,7 @@  discard block
 block discarded – undo
400 400
 		}
401 401
 		$builder = $this->connection->getQueryBuilder();
402 402
 		$fields[] = 'mount_id';
403
-		$placeHolders = array_map(function ($id) use ($builder) {
403
+		$placeHolders = array_map(function($id) use ($builder) {
404 404
 			return $builder->createPositionalParameter($id, IQueryBuilder::PARAM_INT);
405 405
 		}, $mountIds);
406 406
 		$query = $builder->select($fields)
@@ -414,7 +414,7 @@  discard block
 block discarded – undo
414 414
 		}
415 415
 		foreach ($rows as $row) {
416 416
 			if (isset($row['type'])) {
417
-				$row['type'] = (int)$row['type'];
417
+				$row['type'] = (int) $row['type'];
418 418
 			}
419 419
 			$result[$row['mount_id']][] = $row;
420 420
 		}
@@ -445,8 +445,8 @@  discard block
 block discarded – undo
445 445
 	public function getOptionsForMounts($mountIds) {
446 446
 		$mountOptions = $this->selectForMounts('external_options', ['key', 'value'], $mountIds);
447 447
 		$optionsMap = array_map([$this, 'createKeyValueMap'], $mountOptions);
448
-		return array_map(function (array $options) {
449
-			return array_map(function ($option) {
448
+		return array_map(function(array $options) {
449
+			return array_map(function($option) {
450 450
 				return json_decode($option);
451 451
 			}, $options);
452 452
 		}, $optionsMap);
@@ -457,16 +457,16 @@  discard block
 block discarded – undo
457 457
 	 * @return array ['key1' => $value1, ...]
458 458
 	 */
459 459
 	private function createKeyValueMap(array $keyValuePairs) {
460
-		$decryptedPairts = array_map(function ($pair) {
460
+		$decryptedPairts = array_map(function($pair) {
461 461
 			if ($pair['key'] === 'password') {
462 462
 				$pair['value'] = $this->decryptValue($pair['value']);
463 463
 			}
464 464
 			return $pair;
465 465
 		}, $keyValuePairs);
466
-		$keys = array_map(function ($pair) {
466
+		$keys = array_map(function($pair) {
467 467
 			return $pair['key'];
468 468
 		}, $decryptedPairts);
469
-		$values = array_map(function ($pair) {
469
+		$values = array_map(function($pair) {
470 470
 			return $pair['value'];
471 471
 		}, $decryptedPairts);
472 472
 
Please login to merge, or discard this patch.
apps/files_external/lib/Service/LegacyStoragesService.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	) {
59 59
 		$backend = $this->backendService->getBackend($storageOptions['backend']);
60 60
 		if (!$backend) {
61
-			throw new \UnexpectedValueException('Invalid backend ' . $storageOptions['backend']);
61
+			throw new \UnexpectedValueException('Invalid backend '.$storageOptions['backend']);
62 62
 		}
63 63
 		$storageConfig->setBackend($backend);
64 64
 		if (isset($storageOptions['authMechanism']) && $storageOptions['authMechanism'] !== 'builtin::builtin') {
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 			$storageOptions['authMechanism'] = 'null'; // to make error handling easier
69 69
 		}
70 70
 		if (!$authMechanism) {
71
-			throw new \UnexpectedValueException('Invalid authentication mechanism ' . $storageOptions['authMechanism']);
71
+			throw new \UnexpectedValueException('Invalid authentication mechanism '.$storageOptions['authMechanism']);
72 72
 		}
73 73
 		$storageConfig->setAuthMechanism($authMechanism);
74 74
 		$storageConfig->setBackendOptions($storageOptions['options']);
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 						// something went wrong, skip
143 143
 						\OCP\Util::writeLog(
144 144
 							'files_external',
145
-							'Could not parse mount point "' . $rootMountPath . '"',
145
+							'Could not parse mount point "'.$rootMountPath.'"',
146 146
 							\OCP\Util::ERROR
147 147
 						);
148 148
 						continue;
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
 						$storageOptions['authMechanism'] = null; // ensure config hash works
159 159
 					}
160 160
 					if (isset($storageOptions['id'])) {
161
-						$configId = (int)$storageOptions['id'];
161
+						$configId = (int) $storageOptions['id'];
162 162
 						if (isset($storages[$configId])) {
163 163
 							$currentStorage = $storages[$configId];
164 164
 						}
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
 						// don't die if a storage backend doesn't exist
195 195
 						\OCP\Util::writeLog(
196 196
 							'files_external',
197
-							'Could not load storage: "' . $e->getMessage() . '"',
197
+							'Could not load storage: "'.$e->getMessage().'"',
198 198
 							\OCP\Util::ERROR
199 199
 						);
200 200
 					}
Please login to merge, or discard this patch.
apps/files_external/lib/Service/StoragesService.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -70,17 +70,17 @@  discard block
 block discarded – undo
70 70
 	}
71 71
 
72 72
 	protected function getStorageConfigFromDBMount(array $mount) {
73
-		$applicableUsers = array_filter($mount['applicable'], function ($applicable) {
73
+		$applicableUsers = array_filter($mount['applicable'], function($applicable) {
74 74
 			return $applicable['type'] === DBConfigService::APPLICABLE_TYPE_USER;
75 75
 		});
76
-		$applicableUsers = array_map(function ($applicable) {
76
+		$applicableUsers = array_map(function($applicable) {
77 77
 			return $applicable['value'];
78 78
 		}, $applicableUsers);
79 79
 
80
-		$applicableGroups = array_filter($mount['applicable'], function ($applicable) {
80
+		$applicableGroups = array_filter($mount['applicable'], function($applicable) {
81 81
 			return $applicable['type'] === DBConfigService::APPLICABLE_TYPE_GROUP;
82 82
 		});
83
-		$applicableGroups = array_map(function ($applicable) {
83
+		$applicableGroups = array_map(function($applicable) {
84 84
 			return $applicable['value'];
85 85
 		}, $applicableGroups);
86 86
 
@@ -96,20 +96,20 @@  discard block
 block discarded – undo
96 96
 				$mount['priority']
97 97
 			);
98 98
 			$config->setType($mount['type']);
99
-			$config->setId((int)$mount['mount_id']);
99
+			$config->setId((int) $mount['mount_id']);
100 100
 			return $config;
101 101
 		} catch (\UnexpectedValueException $e) {
102 102
 			// don't die if a storage backend doesn't exist
103 103
 			\OCP\Util::writeLog(
104 104
 				'files_external',
105
-				'Could not load storage: "' . $e->getMessage() . '"',
105
+				'Could not load storage: "'.$e->getMessage().'"',
106 106
 				\OCP\Util::ERROR
107 107
 			);
108 108
 			return null;
109 109
 		} catch (\InvalidArgumentException $e) {
110 110
 			\OCP\Util::writeLog(
111 111
 				'files_external',
112
-				'Could not load storage: "' . $e->getMessage() . '"',
112
+				'Could not load storage: "'.$e->getMessage().'"',
113 113
 				\OCP\Util::ERROR
114 114
 			);
115 115
 			return null;
@@ -124,11 +124,11 @@  discard block
 block discarded – undo
124 124
 	protected function readConfig() {
125 125
 		$mounts = $this->readDBConfig();
126 126
 		$configs = array_map([$this, 'getStorageConfigFromDBMount'], $mounts);
127
-		$configs = array_filter($configs, function ($config) {
127
+		$configs = array_filter($configs, function($config) {
128 128
 			return $config instanceof StorageConfig;
129 129
 		});
130 130
 
131
-		$keys = array_map(function (StorageConfig $config) {
131
+		$keys = array_map(function(StorageConfig $config) {
132 132
 			return $config->getId();
133 133
 		}, $configs);
134 134
 
@@ -147,14 +147,14 @@  discard block
 block discarded – undo
147 147
 		$mount = $this->dbConfig->getMountById($id);
148 148
 
149 149
 		if (!is_array($mount)) {
150
-			throw new NotFoundException('Storage with id "' . $id . '" not found');
150
+			throw new NotFoundException('Storage with id "'.$id.'" not found');
151 151
 		}
152 152
 
153 153
 		$config = $this->getStorageConfigFromDBMount($mount);
154 154
 		if ($this->isApplicable($config)) {
155 155
 			return $config;
156 156
 		} else {
157
-			throw new NotFoundException('Storage with id "' . $id . '" not found');
157
+			throw new NotFoundException('Storage with id "'.$id.'" not found');
158 158
 		}
159 159
 	}
160 160
 
@@ -295,11 +295,11 @@  discard block
 block discarded – undo
295 295
 	) {
296 296
 		$backend = $this->backendService->getBackend($backendIdentifier);
297 297
 		if (!$backend) {
298
-			throw new \InvalidArgumentException('Unable to get backend for ' . $backendIdentifier);
298
+			throw new \InvalidArgumentException('Unable to get backend for '.$backendIdentifier);
299 299
 		}
300 300
 		$authMechanism = $this->backendService->getAuthMechanism($authMechanismIdentifier);
301 301
 		if (!$authMechanism) {
302
-			throw new \InvalidArgumentException('Unable to get authentication mechanism for ' . $authMechanismIdentifier);
302
+			throw new \InvalidArgumentException('Unable to get authentication mechanism for '.$authMechanismIdentifier);
303 303
 		}
304 304
 		$newStorage = new StorageConfig();
305 305
 		$newStorage->setMountPoint($mountPoint);
@@ -377,7 +377,7 @@  discard block
 block discarded – undo
377 377
 		$existingMount = $this->dbConfig->getMountById($id);
378 378
 
379 379
 		if (!is_array($existingMount)) {
380
-			throw new NotFoundException('Storage with id "' . $id . '" not found while updating storage');
380
+			throw new NotFoundException('Storage with id "'.$id.'" not found while updating storage');
381 381
 		}
382 382
 
383 383
 		$oldStorage = $this->getStorageConfigFromDBMount($existingMount);
@@ -456,7 +456,7 @@  discard block
 block discarded – undo
456 456
 		$existingMount = $this->dbConfig->getMountById($id);
457 457
 
458 458
 		if (!is_array($existingMount)) {
459
-			throw new NotFoundException('Storage with id "' . $id . '" not found');
459
+			throw new NotFoundException('Storage with id "'.$id.'" not found');
460 460
 		}
461 461
 
462 462
 		$this->dbConfig->removeMount($id);
@@ -474,7 +474,7 @@  discard block
 block discarded – undo
474 474
 			// the storage id could not be computed
475 475
 			\OCP\Util::writeLog(
476 476
 				'files_external',
477
-				'Exception: "' . $e->getMessage() . '"',
477
+				'Exception: "'.$e->getMessage().'"',
478 478
 				\OCP\Util::ERROR
479 479
 			);
480 480
 		}
Please login to merge, or discard this patch.
apps/files_external/lib/Controller/UserStoragesController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -198,7 +198,7 @@
 block discarded – undo
198 198
 		} catch (NotFoundException $e) {
199 199
 			return new DataResponse(
200 200
 				[
201
-					'message' => (string)$this->l10n->t('Storage with id "%i" not found', array($id))
201
+					'message' => (string) $this->l10n->t('Storage with id "%i" not found', array($id))
202 202
 				],
203 203
 				Http::STATUS_NOT_FOUND
204 204
 			);
Please login to merge, or discard this patch.
apps/files_external/lib/Controller/UserGlobalStoragesController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 		} catch (NotFoundException $e) {
123 123
 			return new DataResponse(
124 124
 				[
125
-					'message' => (string)$this->l10n->t('Storage with id "%i" not found', array($id))
125
+					'message' => (string) $this->l10n->t('Storage with id "%i" not found', array($id))
126 126
 				],
127 127
 				Http::STATUS_NOT_FOUND
128 128
 			);
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 			} else {
163 163
 				return new DataResponse(
164 164
 					[
165
-						'message' => (string)$this->l10n->t('Storage with id "%i" is not user editable', array($id))
165
+						'message' => (string) $this->l10n->t('Storage with id "%i" is not user editable', array($id))
166 166
 					],
167 167
 					Http::STATUS_FORBIDDEN
168 168
 				);
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 		} catch (NotFoundException $e) {
171 171
 			return new DataResponse(
172 172
 				[
173
-					'message' => (string)$this->l10n->t('Storage with id "%i" not found', array($id))
173
+					'message' => (string) $this->l10n->t('Storage with id "%i" not found', array($id))
174 174
 				],
175 175
 				Http::STATUS_NOT_FOUND
176 176
 			);
Please login to merge, or discard this patch.
apps/files_external/lib/AppInfo/Application.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
 
47 47
 		$container = $this->getContainer();
48 48
 
49
-		$container->registerService('OCP\Files\Config\IUserMountCache', function (IAppContainer $c) {
49
+		$container->registerService('OCP\Files\Config\IUserMountCache', function(IAppContainer $c) {
50 50
 			return $c->getServer()->query('UserMountCache');
51 51
 		});
52 52
 
Please login to merge, or discard this patch.
apps/files_external/lib/Config/ConfigAdapter.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 	 * @param StorageConfig $storage
75 75
 	 * @param IUser $user
76 76
 	 */
77
-	private function prepareStorageConfig(StorageConfig &$storage, IUser $user) {
77
+	private function prepareStorageConfig(StorageConfig & $storage, IUser $user) {
78 78
 		foreach ($storage->getBackendOptions() as $option => $value) {
79 79
 			$storage->setBackendOption($option, \OC_Mount_Config::setUserVars(
80 80
 				$user->getUID(), $value
@@ -141,12 +141,12 @@  discard block
 block discarded – undo
141 141
 			return $storage->getId();
142 142
 		}, $storages));
143 143
 
144
-		$availableStorages = array_map(function (Storage\IStorage $storage, StorageConfig $storageConfig) {
144
+		$availableStorages = array_map(function(Storage\IStorage $storage, StorageConfig $storageConfig) {
145 145
 			try {
146 146
 				$availability = $storage->getAvailability();
147 147
 				if (!$availability['available'] && !Availability::shouldRecheck($availability)) {
148 148
 					$storage = new FailedStorage([
149
-						'exception' => new StorageNotAvailableException('Storage with mount id ' . $storageConfig->getId() . ' is not available')
149
+						'exception' => new StorageNotAvailableException('Storage with mount id '.$storageConfig->getId().' is not available')
150 150
 					]);
151 151
 				}
152 152
 			} catch (\Exception $e) {
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 					$this->userStoragesService,
163 163
 					$storageConfig->getId(),
164 164
 					$storage,
165
-					'/' . $user->getUID() . '/files' . $storageConfig->getMountPoint(),
165
+					'/'.$user->getUID().'/files'.$storageConfig->getMountPoint(),
166 166
 					null,
167 167
 					$loader,
168 168
 					$storageConfig->getMountOptions()
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 			} else {
171 171
 				return new MountPoint(
172 172
 					$storage,
173
-					'/' . $user->getUID() . '/files' . $storageConfig->getMountPoint(),
173
+					'/'.$user->getUID().'/files'.$storageConfig->getMountPoint(),
174 174
 					null,
175 175
 					$loader,
176 176
 					$storageConfig->getMountOptions(),
Please login to merge, or discard this patch.
apps/files_external/lib/config.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -255,11 +255,11 @@  discard block
 block discarded – undo
255 255
 	 */
256 256
 	public static function readData($user = null) {
257 257
 		if (isset($user)) {
258
-			$jsonFile = \OC::$server->getUserManager()->get($user)->getHome() . '/mount.json';
258
+			$jsonFile = \OC::$server->getUserManager()->get($user)->getHome().'/mount.json';
259 259
 		} else {
260 260
 			$config = \OC::$server->getConfig();
261
-			$datadir = $config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data/');
262
-			$jsonFile = $config->getSystemValue('mount_file', $datadir . '/mount.json');
261
+			$datadir = $config->getSystemValue('datadirectory', \OC::$SERVERROOT.'/data/');
262
+			$jsonFile = $config->getSystemValue('mount_file', $datadir.'/mount.json');
263 263
 		}
264 264
 		if (is_file($jsonFile)) {
265 265
 			$mountPoints = json_decode(file_get_contents($jsonFile), true);
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
 		foreach ($backends as $backend) {
286 286
 			foreach ($backend->checkDependencies() as $dependency) {
287 287
 				if ($message = $dependency->getMessage()) {
288
-					$message .= '<p>' . $message . '</p>';
288
+					$message .= '<p>'.$message.'</p>';
289 289
 				} else {
290 290
 					$dependencyGroups[$dependency->getDependency()][] = $backend;
291 291
 				}
@@ -294,9 +294,9 @@  discard block
 block discarded – undo
294 294
 
295 295
 		foreach ($dependencyGroups as $module => $dependants) {
296 296
 			$backends = implode(', ', array_map(function($backend) {
297
-				return '"' . $backend->getText() . '"';
297
+				return '"'.$backend->getText().'"';
298 298
 			}, $dependants));
299
-			$message .= '<p>' . OC_Mount_Config::getSingleDependencyMessage($l, $module, $backends) . '</p>';
299
+			$message .= '<p>'.OC_Mount_Config::getSingleDependencyMessage($l, $module, $backends).'</p>';
300 300
 		}
301 301
 
302 302
 		return $message;
@@ -313,11 +313,11 @@  discard block
 block discarded – undo
313 313
 	private static function getSingleDependencyMessage(\OCP\IL10N $l, $module, $backend) {
314 314
 		switch (strtolower($module)) {
315 315
 			case 'curl':
316
-				return (string)$l->t('The cURL support in PHP is not enabled or installed. Mounting of %s is not possible. Please ask your system administrator to install it.', $backend);
316
+				return (string) $l->t('The cURL support in PHP is not enabled or installed. Mounting of %s is not possible. Please ask your system administrator to install it.', $backend);
317 317
 			case 'ftp':
318
-				return (string)$l->t('The FTP support in PHP is not enabled or installed. Mounting of %s is not possible. Please ask your system administrator to install it.', $backend);
318
+				return (string) $l->t('The FTP support in PHP is not enabled or installed. Mounting of %s is not possible. Please ask your system administrator to install it.', $backend);
319 319
 			default:
320
-				return (string)$l->t('"%s" is not installed. Mounting of %s is not possible. Please ask your system administrator to install it.', array($module, $backend));
320
+				return (string) $l->t('"%s" is not installed. Mounting of %s is not possible. Please ask your system administrator to install it.', array($module, $backend));
321 321
 		}
322 322
 	}
323 323
 
@@ -362,7 +362,7 @@  discard block
 block discarded – undo
362 362
 		$cipher = self::getCipher();
363 363
 		$iv = \OCP\Util::generateRandomBytes(16);
364 364
 		$cipher->setIV($iv);
365
-		return base64_encode($iv . $cipher->encrypt($password));
365
+		return base64_encode($iv.$cipher->encrypt($password));
366 366
 	}
367 367
 
368 368
 	/**
Please login to merge, or discard this patch.