Passed
Push — master ( c233ac...ceefe0 )
by Joas
14:23 queued 13s
created
apps/files_external/lib/Service/UserGlobalStoragesService.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
 
116 116
 		$result = [];
117 117
 		foreach ($storagesByMountpoint as $storageList) {
118
-			$storage = array_reduce($storageList, function ($carry, $item) {
118
+			$storage = array_reduce($storageList, function($carry, $item) {
119 119
 				if (isset($carry)) {
120 120
 					$carryPriorityType = $this->getPriorityType($carry);
121 121
 					$itemPriorityType = $this->getPriorityType($item);
@@ -191,11 +191,11 @@  discard block
 block discarded – undo
191 191
 		$groupIds = $this->groupManager->getUserGroupIds($user);
192 192
 		$mounts = $this->dbConfig->getMountsForUser($user->getUID(), $groupIds);
193 193
 		$configs = array_map([$this, 'getStorageConfigFromDBMount'], $mounts);
194
-		$configs = array_filter($configs, function ($config) {
194
+		$configs = array_filter($configs, function($config) {
195 195
 			return $config instanceof StorageConfig;
196 196
 		});
197 197
 
198
-		$keys = array_map(function (StorageConfig $config) {
198
+		$keys = array_map(function(StorageConfig $config) {
199 199
 			return $config->getId();
200 200
 		}, $configs);
201 201
 
Please login to merge, or discard this patch.
apps/files_external/lib/BackgroundJob/CredentialsCleanup.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -54,10 +54,10 @@
 block discarded – undo
54 54
 	}
55 55
 
56 56
 	protected function run($argument) {
57
-		$this->userManager->callForSeenUsers(function (IUser $user) {
57
+		$this->userManager->callForSeenUsers(function(IUser $user) {
58 58
 			$storages = $this->userGlobalStoragesService->getAllStoragesForUser($user);
59 59
 
60
-			$usesLoginCredentials = array_reduce($storages, function (bool $uses, StorageConfig $storage) {
60
+			$usesLoginCredentials = array_reduce($storages, function(bool $uses, StorageConfig $storage) {
61 61
 				return $uses || $storage->getAuthMechanism() instanceof LoginCredentials;
62 62
 			}, false);
63 63
 
Please login to merge, or discard this patch.
core/Command/App/Update.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 			try {
94 94
 				$this->manager->getAppPath($singleAppId);
95 95
 			} catch (\OCP\App\AppPathNotFoundException $e) {
96
-				$output->writeln($singleAppId . ' not installed');
96
+				$output->writeln($singleAppId.' not installed');
97 97
 				return 1;
98 98
 			}
99 99
 		} elseif ($input->getOption('all') || $input->getOption('showonly')) {
@@ -107,22 +107,22 @@  discard block
 block discarded – undo
107 107
 		foreach ($apps as $appId) {
108 108
 			$newVersion = $this->installer->isUpdateAvailable($appId, $input->getOption('allow-unstable'));
109 109
 			if ($newVersion) {
110
-				$output->writeln($appId . ' new version available: ' . $newVersion);
110
+				$output->writeln($appId.' new version available: '.$newVersion);
111 111
 
112 112
 				if (!$input->getOption('showonly')) {
113 113
 					try {
114 114
 						$result = $this->installer->updateAppstoreApp($appId, $input->getOption('allow-unstable'));
115 115
 					} catch (\Exception $e) {
116
-						$this->logger->logException($e, ['message' => 'Failure during update of app "' . $appId . '"','app' => 'app:update']);
117
-						$output->writeln('Error: ' . $e->getMessage());
116
+						$this->logger->logException($e, ['message' => 'Failure during update of app "'.$appId.'"', 'app' => 'app:update']);
117
+						$output->writeln('Error: '.$e->getMessage());
118 118
 						$return = 1;
119 119
 					}
120 120
 
121 121
 					if ($result === false) {
122
-						$output->writeln($appId . ' couldn\'t be updated');
122
+						$output->writeln($appId.' couldn\'t be updated');
123 123
 						$return = 1;
124 124
 					} elseif ($result === true) {
125
-						$output->writeln($appId . ' updated');
125
+						$output->writeln($appId.' updated');
126 126
 					}
127 127
 				}
128 128
 			}
Please login to merge, or discard this patch.
core/Command/App/Disable.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -72,14 +72,14 @@
 block discarded – undo
72 72
 
73 73
 	private function disableApp(string $appId, OutputInterface $output): void {
74 74
 		if ($this->appManager->isInstalled($appId) === false) {
75
-			$output->writeln('No such app enabled: ' . $appId);
75
+			$output->writeln('No such app enabled: '.$appId);
76 76
 			return;
77 77
 		}
78 78
 
79 79
 		try {
80 80
 			$this->appManager->disableApp($appId);
81 81
 			$appVersion = \OC_App::getAppVersion($appId);
82
-			$output->writeln($appId . ' ' . $appVersion . ' disabled');
82
+			$output->writeln($appId.' '.$appVersion.' disabled');
83 83
 		} catch (\Exception $e) {
84 84
 			$output->writeln($e->getMessage());
85 85
 			$this->exitCode = 2;
Please login to merge, or discard this patch.
core/Command/App/Enable.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -102,12 +102,12 @@  discard block
 block discarded – undo
102 102
 	 * @param OutputInterface $output
103 103
 	 */
104 104
 	private function enableApp(string $appId, array $groupIds, bool $forceEnable, OutputInterface $output): void {
105
-		$groupNames = array_map(function (IGroup $group) {
105
+		$groupNames = array_map(function(IGroup $group) {
106 106
 			return $group->getDisplayName();
107 107
 		}, $groupIds);
108 108
 
109 109
 		if ($this->appManager->isInstalled($appId) && $groupIds === []) {
110
-			$output->writeln($appId . ' already enabled');
110
+			$output->writeln($appId.' already enabled');
111 111
 			return;
112 112
 		}
113 113
 
@@ -124,13 +124,13 @@  discard block
 block discarded – undo
124 124
 
125 125
 			if ($groupIds === []) {
126 126
 				$this->appManager->enableApp($appId, $forceEnable);
127
-				$output->writeln($appId . ' ' . $appVersion . ' enabled');
127
+				$output->writeln($appId.' '.$appVersion.' enabled');
128 128
 			} else {
129 129
 				$this->appManager->enableAppForGroups($appId, $groupIds, $forceEnable);
130
-				$output->writeln($appId . ' ' . $appVersion . ' enabled for groups: ' . implode(', ', $groupNames));
130
+				$output->writeln($appId.' '.$appVersion.' enabled for groups: '.implode(', ', $groupNames));
131 131
 			}
132 132
 		} catch (AppPathNotFoundException $e) {
133
-			$output->writeln($appId . ' not found');
133
+			$output->writeln($appId.' not found');
134 134
 			$this->exitCode = 1;
135 135
 		} catch (\Exception $e) {
136 136
 			$output->writeln($e->getMessage());
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	 */
161 161
 	public function completeOptionValues($optionName, CompletionContext $context) {
162 162
 		if ($optionName === 'groups') {
163
-			return array_map(function (IGroup $group) {
163
+			return array_map(function(IGroup $group) {
164 164
 				return $group->getGID();
165 165
 			}, $this->groupManager->search($context->getCurrentWord()));
166 166
 		}
Please login to merge, or discard this patch.
core/Command/App/Remove.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -80,14 +80,14 @@  discard block
 block discarded – undo
80 80
 
81 81
 		// Check if the app is installed
82 82
 		if (!\OC_App::getAppPath($appId)) {
83
-			$output->writeln($appId . ' is not installed');
83
+			$output->writeln($appId.' is not installed');
84 84
 			return 1;
85 85
 		}
86 86
 
87 87
 		// Removing shipped apps is not possible, therefore we pre-check that
88 88
 		// before trying to remove it
89 89
 		if ($this->manager->isShipped($appId)) {
90
-			$output->writeln($appId . ' could not be removed as it is a shipped app');
90
+			$output->writeln($appId.' could not be removed as it is a shipped app');
91 91
 			return 1;
92 92
 		}
93 93
 
@@ -96,9 +96,9 @@  discard block
 block discarded – undo
96 96
 		if (!$input->getOption('keep-data')) {
97 97
 			try {
98 98
 				$this->manager->disableApp($appId);
99
-				$output->writeln($appId . ' disabled');
99
+				$output->writeln($appId.' disabled');
100 100
 			} catch (Throwable $e) {
101
-				$output->writeln('<error>Error: ' . $e->getMessage() . '</error>');
101
+				$output->writeln('<error>Error: '.$e->getMessage().'</error>');
102 102
 				$this->logger->logException($e, [
103 103
 					'app' => 'CLI',
104 104
 					'level' => ILogger::ERROR
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 		try {
112 112
 			$result = $this->installer->removeApp($appId);
113 113
 		} catch (Throwable $e) {
114
-			$output->writeln('<error>Error: ' . $e->getMessage() . '</error>');
114
+			$output->writeln('<error>Error: '.$e->getMessage().'</error>');
115 115
 			$this->logger->logException($e, [
116 116
 				'app' => 'CLI',
117 117
 				'level' => ILogger::ERROR
@@ -120,12 +120,12 @@  discard block
 block discarded – undo
120 120
 		}
121 121
 
122 122
 		if ($result === false) {
123
-			$output->writeln($appId . ' could not be removed');
123
+			$output->writeln($appId.' could not be removed');
124 124
 			return 1;
125 125
 		}
126 126
 
127 127
 		$appVersion = \OC_App::getAppVersion($appId);
128
-		$output->writeln($appId . ' ' . $appVersion . ' removed');
128
+		$output->writeln($appId.' '.$appVersion.' removed');
129 129
 
130 130
 		return 0;
131 131
 	}
Please login to merge, or discard this patch.
lib/private/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDao.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 		$result = $query->execute();
61 61
 		$providers = [];
62 62
 		foreach ($result->fetchAll() as $row) {
63
-			$providers[$row['provider_id']] = 1 === (int)$row['enabled'];
63
+			$providers[$row['provider_id']] = 1 === (int) $row['enabled'];
64 64
 		}
65 65
 		$result->closeCursor();
66 66
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 			->where($qb2->expr()->eq('uid', $qb2->createNamedParameter($uid)));
115 115
 		$deleteQuery->execute();
116 116
 
117
-		return array_map(function (array $row) {
117
+		return array_map(function(array $row) {
118 118
 			return [
119 119
 				'provider_id' => $row['provider_id'],
120 120
 				'uid' => $row['uid'],
Please login to merge, or discard this patch.
lib/private/Search.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 		$this->initProviders();
53 53
 		$results = [];
54 54
 		foreach ($this->providers as $provider) {
55
-			if (! $provider->providesResultsFor($inApps)) {
55
+			if (!$provider->providesResultsFor($inApps)) {
56 56
 				continue;
57 57
 			}
58 58
 			if ($provider instanceof PagedProvider) {
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 	public function removeProvider($provider) {
88 88
 		$this->registeredProviders = array_filter(
89 89
 			$this->registeredProviders,
90
-			function ($element) use ($provider) {
90
+			function($element) use ($provider) {
91 91
 				return ($element['class'] != $provider);
92 92
 			}
93 93
 		);
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
 	 * Create instances of all the registered search providers
109 109
 	 */
110 110
 	private function initProviders() {
111
-		if (! empty($this->providers)) {
111
+		if (!empty($this->providers)) {
112 112
 			return;
113 113
 		}
114 114
 		foreach ($this->registeredProviders as $provider) {
Please login to merge, or discard this patch.
apps/files_sharing/lib/ShareBackend/File.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 	 */
98 98
 	public function generateTarget($itemSource, $shareWith, $exclude = null) {
99 99
 		$shareFolder = \OCA\Files_Sharing\Helper::getShareFolder();
100
-		$target = \OC\Files\Filesystem::normalizePath($shareFolder . '/' . basename($itemSource));
100
+		$target = \OC\Files\Filesystem::normalizePath($shareFolder.'/'.basename($itemSource));
101 101
 
102 102
 		// for group shares we return the target right away
103 103
 		if ($shareWith === false) {
@@ -105,13 +105,13 @@  discard block
 block discarded – undo
105 105
 		}
106 106
 
107 107
 		\OC\Files\Filesystem::initMountPoints($shareWith);
108
-		$view = new \OC\Files\View('/' . $shareWith . '/files');
108
+		$view = new \OC\Files\View('/'.$shareWith.'/files');
109 109
 
110 110
 		if (!$view->is_dir($shareFolder)) {
111 111
 			$dir = '';
112 112
 			$subdirs = explode('/', $shareFolder);
113 113
 			foreach ($subdirs as $subdir) {
114
-				$dir = $dir . '/' . $subdir;
114
+				$dir = $dir.'/'.$subdir;
115 115
 				if (!$view->is_dir($dir)) {
116 116
 					$view->mkdir($dir);
117 117
 				}
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
 		if ($share['item_type'] === 'folder' && $target !== '') {
250 250
 			// note: in case of ext storage mount points the path might be empty
251 251
 			// which would cause a leading slash to appear
252
-			$share['path'] = ltrim($share['path'] . '/' . $target, '/');
252
+			$share['path'] = ltrim($share['path'].'/'.$target, '/');
253 253
 		}
254 254
 		return self::resolveReshares($share);
255 255
 	}
Please login to merge, or discard this patch.