Passed
Push — master ( 348454...c8160a )
by Joas
15:07 queued 14s
created
apps/workflowengine/lib/Entity/File.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -106,12 +106,12 @@  discard block
 block discarded – undo
106 106
 
107 107
 	public function getEvents(): array {
108 108
 		return [
109
-			new GenericEntityEvent($this->l10n->t('File created'), self::EVENT_NAMESPACE . 'postCreate'),
110
-			new GenericEntityEvent($this->l10n->t('File updated'), self::EVENT_NAMESPACE . 'postWrite'),
111
-			new GenericEntityEvent($this->l10n->t('File renamed'), self::EVENT_NAMESPACE . 'postRename'),
112
-			new GenericEntityEvent($this->l10n->t('File deleted'), self::EVENT_NAMESPACE . 'postDelete'),
113
-			new GenericEntityEvent($this->l10n->t('File accessed'), self::EVENT_NAMESPACE . 'postTouch'),
114
-			new GenericEntityEvent($this->l10n->t('File copied'), self::EVENT_NAMESPACE . 'postCopy'),
109
+			new GenericEntityEvent($this->l10n->t('File created'), self::EVENT_NAMESPACE.'postCreate'),
110
+			new GenericEntityEvent($this->l10n->t('File updated'), self::EVENT_NAMESPACE.'postWrite'),
111
+			new GenericEntityEvent($this->l10n->t('File renamed'), self::EVENT_NAMESPACE.'postRename'),
112
+			new GenericEntityEvent($this->l10n->t('File deleted'), self::EVENT_NAMESPACE.'postDelete'),
113
+			new GenericEntityEvent($this->l10n->t('File accessed'), self::EVENT_NAMESPACE.'postTouch'),
114
+			new GenericEntityEvent($this->l10n->t('File copied'), self::EVENT_NAMESPACE.'postCopy'),
115 115
 			new GenericEntityEvent($this->l10n->t('Tag assigned'), MapperEvent::EVENT_ASSIGN),
116 116
 		];
117 117
 	}
@@ -156,19 +156,19 @@  discard block
 block discarded – undo
156 156
 			throw new NotFoundException();
157 157
 		}
158 158
 		switch ($this->eventName) {
159
-			case self::EVENT_NAMESPACE . 'postCreate':
160
-			case self::EVENT_NAMESPACE . 'postWrite':
161
-			case self::EVENT_NAMESPACE . 'postDelete':
162
-			case self::EVENT_NAMESPACE . 'postTouch':
159
+			case self::EVENT_NAMESPACE.'postCreate':
160
+			case self::EVENT_NAMESPACE.'postWrite':
161
+			case self::EVENT_NAMESPACE.'postDelete':
162
+			case self::EVENT_NAMESPACE.'postTouch':
163 163
 				return $this->event->getSubject();
164
-			case self::EVENT_NAMESPACE . 'postRename':
165
-			case self::EVENT_NAMESPACE . 'postCopy':
164
+			case self::EVENT_NAMESPACE.'postRename':
165
+			case self::EVENT_NAMESPACE.'postCopy':
166 166
 				return $this->event->getSubject()[1];
167 167
 			case MapperEvent::EVENT_ASSIGN:
168 168
 				if (!$this->event instanceof MapperEvent || $this->event->getObjectType() !== 'files') {
169 169
 					throw new NotFoundException();
170 170
 				}
171
-				$nodes = $this->root->getById((int)$this->event->getObjectId());
171
+				$nodes = $this->root->getById((int) $this->event->getObjectId());
172 172
 				if (is_array($nodes) && isset($nodes[0])) {
173 173
 					$this->node = $nodes[0];
174 174
 					return $this->node;
@@ -191,17 +191,17 @@  discard block
 block discarded – undo
191 191
 		];
192 192
 
193 193
 		switch ($this->eventName) {
194
-			case self::EVENT_NAMESPACE . 'postCreate':
194
+			case self::EVENT_NAMESPACE.'postCreate':
195 195
 				return $this->l10n->t('%s created %s', $options);
196
-			case self::EVENT_NAMESPACE . 'postWrite':
196
+			case self::EVENT_NAMESPACE.'postWrite':
197 197
 				return $this->l10n->t('%s modified %s', $options);
198
-			case self::EVENT_NAMESPACE . 'postDelete':
198
+			case self::EVENT_NAMESPACE.'postDelete':
199 199
 				return $this->l10n->t('%s deleted %s', $options);
200
-			case self::EVENT_NAMESPACE . 'postTouch':
200
+			case self::EVENT_NAMESPACE.'postTouch':
201 201
 				return $this->l10n->t('%s accessed %s', $options);
202
-			case self::EVENT_NAMESPACE . 'postRename':
202
+			case self::EVENT_NAMESPACE.'postRename':
203 203
 				return $this->l10n->t('%s renamed %s', $options);
204
-			case self::EVENT_NAMESPACE . 'postCopy':
204
+			case self::EVENT_NAMESPACE.'postCopy':
205 205
 				return $this->l10n->t('%s copied %s', $options);
206 206
 			case MapperEvent::EVENT_ASSIGN:
207 207
 				$tagNames = [];
Please login to merge, or discard this patch.
lib/private/Files/ObjectStore/AppdataPreviewObjectStoreStorage.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,11 +34,11 @@
 block discarded – undo
34 34
 		if (!isset($params['internal-id'])) {
35 35
 			throw new \Exception('missing id in parameters');
36 36
 		}
37
-		$this->internalId = (string)$params['internal-id'];
37
+		$this->internalId = (string) $params['internal-id'];
38 38
 		parent::__construct($params);
39 39
 	}
40 40
 
41 41
 	public function getId() {
42
-		return 'object::appdata::preview:' . $this->internalId;
42
+		return 'object::appdata::preview:'.$this->internalId;
43 43
 	}
44 44
 }
Please login to merge, or discard this patch.
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/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.
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.
lib/private/SystemTag/SystemTagManager.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -141,14 +141,14 @@  discard block
 block discarded – undo
141 141
 			->from(self::TAG_TABLE);
142 142
 
143 143
 		if (!\is_null($visibilityFilter)) {
144
-			$query->andWhere($query->expr()->eq('visibility', $query->createNamedParameter((int)$visibilityFilter)));
144
+			$query->andWhere($query->expr()->eq('visibility', $query->createNamedParameter((int) $visibilityFilter)));
145 145
 		}
146 146
 
147 147
 		if (!empty($nameSearchPattern)) {
148 148
 			$query->andWhere(
149 149
 				$query->expr()->like(
150 150
 					'name',
151
-					$query->createNamedParameter('%' . $this->connection->escapeLikeParameter($nameSearchPattern). '%')
151
+					$query->createNamedParameter('%'.$this->connection->escapeLikeParameter($nameSearchPattern).'%')
152 152
 				)
153 153
 			);
154 154
 		}
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
 		$result->closeCursor();
183 183
 		if (!$row) {
184 184
 			throw new TagNotFoundException(
185
-				'Tag ("' . $tagName . '", '. $userVisible . ', ' . $userAssignable . ') does not exist'
185
+				'Tag ("'.$tagName.'", '.$userVisible.', '.$userAssignable.') does not exist'
186 186
 			);
187 187
 		}
188 188
 
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 			$query->execute();
206 206
 		} catch (UniqueConstraintViolationException $e) {
207 207
 			throw new TagAlreadyExistsException(
208
-				'Tag ("' . $tagName . '", '. $userVisible . ', ' . $userAssignable . ') already exists',
208
+				'Tag ("'.$tagName.'", '.$userVisible.', '.$userAssignable.') already exists',
209 209
 				0,
210 210
 				$e
211 211
 			);
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 		$tagId = $query->getLastInsertId();
215 215
 
216 216
 		$tag = new SystemTag(
217
-			(string)$tagId,
217
+			(string) $tagId,
218 218
 			$tagName,
219 219
 			$userVisible,
220 220
 			$userAssignable
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
 			}
267 267
 		} catch (UniqueConstraintViolationException $e) {
268 268
 			throw new TagAlreadyExistsException(
269
-				'Tag ("' . $newName . '", '. $userVisible . ', ' . $userAssignable . ') already exists',
269
+				'Tag ("'.$newName.'", '.$userVisible.', '.$userAssignable.') already exists',
270 270
 				0,
271 271
 				$e
272 272
 			);
@@ -373,7 +373,7 @@  discard block
 block discarded – undo
373 373
 	}
374 374
 
375 375
 	private function createSystemTagFromRow($row) {
376
-		return new SystemTag((string)$row['id'], $row['name'], (bool)$row['visibility'], (bool)$row['editable']);
376
+		return new SystemTag((string) $row['id'], $row['name'], (bool) $row['visibility'], (bool) $row['editable']);
377 377
 	}
378 378
 
379 379
 	/**
Please login to merge, or discard this patch.