Completed
Pull Request — master (#936)
by Maxence
15s
created
lib/Service/IndexService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -155,7 +155,7 @@
 block discarded – undo
155 155
 		IFullTextSearchPlatform $platform, IFullTextSearchProvider $provider, string $userId,
156 156
 		IndexOptions $options
157 157
 	) {
158
-		$this->updateRunnerAction('generateIndex' . $provider->getName());
158
+		$this->updateRunnerAction('generateIndex'.$provider->getName());
159 159
 		$this->updateRunnerInfoArray(
160 160
 			[
161 161
 				'userId' => $userId,
Please login to merge, or discard this patch.
lib/Service/CollectionService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@
 block discarded – undo
127 127
 		}
128 128
 
129 129
 		return array_map(
130
-			function (Index $index): array {
130
+			function(Index $index): array {
131 131
 				return $index->asSitemap($this->urlGenerator);
132 132
 			},
133 133
 			$this->indexesRequest->getQueuedIndexes($collection, false, $length)
Please login to merge, or discard this patch.
lib/Command/CollectionInit.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -118,7 +118,7 @@
 block discarded – undo
118 118
 
119 119
 		$this->cliService->createPanel(
120 120
 			'collection', [
121
-							'┌─ Collection ' . $collection . ' ────',
121
+							'┌─ Collection '.$collection.' ────',
122 122
 							'│ ProviderId, UserId: <info>%providerId%</info> / <info>%userId%</info>',
123 123
 							'│ Chunk: <info>%chunkCurr:3s%</info>/<info>%chunkTotal%</info>',
124 124
 							'│ Document: <info>%documentCurr:6s%</info>/<info>%documentChunk%</info>',
Please login to merge, or discard this patch.
lib/Service/PlatformService.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 			try {
118 118
 				$platform = \OCP\Server::get((string)$class);
119 119
 				if (!($platform instanceof IFullTextSearchPlatform)) {
120
-					$this->logger->warning($class . ' does not implement ' . IFullTextSearchPlatform::class);
120
+					$this->logger->warning($class.' does not implement '.IFullTextSearchPlatform::class);
121 121
 					continue;
122 122
 				}
123 123
 				
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 		$platform = OC::$server->query((string)$selected->getClass());
176 176
 		if (!($platform instanceof IFullTextSearchPlatform)) {
177 177
 			throw new PlatformIsNotCompatibleException(
178
-				$selected->getClass() . ' is not a compatible FullTextSearchPlatform'
178
+				$selected->getClass().' is not a compatible FullTextSearchPlatform'
179 179
 			);
180 180
 		}
181 181
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 		}
208 208
 
209 209
 		throw new PlatformDoesNotExistException(
210
-			'FullTextSearchPlatform ' . $selected . ' is not available'
210
+			'FullTextSearchPlatform '.$selected.' is not available'
211 211
 		);
212 212
 	}
213 213
 
Please login to merge, or discard this patch.
lib/Tools/Traits/TArrayTools.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -400,7 +400,7 @@
 block discarded – undo
400 400
 		foreach ($keys as $key) {
401 401
 			if (!array_key_exists($key, $arr)) {
402 402
 				throw new MalformedArrayException(
403
-					'source: ' . json_encode($arr) . ' - missing key: ' . $key
403
+					'source: '.json_encode($arr).' - missing key: '.$key
404 404
 				);
405 405
 			}
406 406
 		}
Please login to merge, or discard this patch.
lib/Cron/Index.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -124,8 +124,8 @@
 block discarded – undo
124 124
 			} catch (PlatformTemporaryException $e) {
125 125
 				$this->logger->warning('platform seems down. we will update index next cron tick');
126 126
 				return;
127
-			} catch (Throwable|Exception $e) {
128
-				$this->runner->exception(get_class($e) . ' - ' . $e->getMessage(), false);
127
+			} catch (Throwable | Exception $e) {
128
+				$this->runner->exception(get_class($e).' - '.$e->getMessage(), false);
129 129
 				$this->logger->notice(
130 130
 					'exception encountered while running fulltextsearch/lib/Cron/Index.php',
131 131
 					['exception' => $e]
Please login to merge, or discard this patch.
lib/Command/CollectionList.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -62,10 +62,10 @@
 block discarded – undo
62 62
 	 */
63 63
 	protected function execute(InputInterface $input, OutputInterface $output) {
64 64
 		$collections = $this->collectionService->getCollections();
65
-		$output->writeln('found ' . sizeof($collections) . ' collection(s)');
65
+		$output->writeln('found '.sizeof($collections).' collection(s)');
66 66
 
67 67
 		foreach ($this->collectionService->getCollections() as $collection) {
68
-			$output->writeln('- ' . (($collection === $this->configService->getInternalCollection()) ? '*' : '') . $collection);
68
+			$output->writeln('- '.(($collection === $this->configService->getInternalCollection()) ? '*' : '').$collection);
69 69
 		}
70 70
 
71 71
 		return 0;
Please login to merge, or discard this patch.
lib/Command/Reset.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -84,8 +84,8 @@  discard block
 block discarded – undo
84 84
 		$helper = $this->getHelper('question');
85 85
 
86 86
 		$output->writeln('<error>WARNING! You are about to reset your indexed documents:</error>');
87
-		$output->writeln('- provider: <info>' . (($provider === '') ? 'ALL' : $provider) . '</info>');
88
-		$output->writeln('- collection: <info>' . (($collection === '') ? 'ALL' : $collection) . '</info>');
87
+		$output->writeln('- provider: <info>'.(($provider === '') ? 'ALL' : $provider).'</info>');
88
+		$output->writeln('- collection: <info>'.(($collection === '') ? 'ALL' : $collection).'</info>');
89 89
 		$output->writeln('');
90 90
 
91 91
 		$question = new ConfirmationQuestion(
@@ -102,10 +102,10 @@  discard block
 block discarded – undo
102 102
 
103 103
 		$output->writeln('');
104 104
 		$output->writeln('<error>WARNING! This operation is not reversible.</error>');
105
-		$action = 'reset ' . (($provider === '') ? 'ALL' : $provider)
106
-			. ' ' . (($collection === '') ? 'ALL' : $collection);
105
+		$action = 'reset '.(($provider === '') ? 'ALL' : $provider)
106
+			. ' '.(($collection === '') ? 'ALL' : $collection);
107 107
 
108
-		$question = new Question('<comment>Please confirm this destructive operation by typing \'' . $action . '\'</comment>: ', '');
108
+		$question = new Question('<comment>Please confirm this destructive operation by typing \''.$action.'\'</comment>: ', '');
109 109
 
110 110
 		$helper = $this->getHelper('question');
111 111
 		$confirmation = $helper->ask($input, $output, $question);
Please login to merge, or discard this patch.
lib/Db/CoreRequestBuilder.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -202,8 +202,8 @@  discard block
 block discarded – undo
202 202
 	 */
203 203
 	private function limitToDBField(IQueryBuilder $qb, string $field, string $value) {
204 204
 		$expr = $qb->expr();
205
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
206
-		$field = $pf . $field;
205
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
206
+		$field = $pf.$field;
207 207
 
208 208
 		$qb->andWhere($expr->eq($field, $qb->createNamedParameter($value)));
209 209
 	}
@@ -215,8 +215,8 @@  discard block
 block discarded – undo
215 215
 	 */
216 216
 	private function limitToDBFieldInt(IQueryBuilder $qb, string $field, int $value) {
217 217
 		$expr = $qb->expr();
218
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
219
-		$field = $pf . $field;
218
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
219
+		$field = $pf.$field;
220 220
 
221 221
 		$qb->andWhere($expr->eq($field, $qb->createNamedParameter($value)));
222 222
 	}
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
 	 */
230 230
 	private function limitToDBFieldArray(IQueryBuilder $qb, string $field, array $values) {
231 231
 		$expr = $qb->expr();
232
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
233
-		$field = $pf . $field;
232
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
233
+		$field = $pf.$field;
234 234
 
235 235
 		if (!is_array($values)) {
236 236
 			$values = [$values];
@@ -250,8 +250,8 @@  discard block
 block discarded – undo
250 250
 	 */
251 251
 	protected function limitToQueuedIndexes(IQueryBuilder $qb) {
252 252
 		$expr = $qb->expr();
253
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
254
-		$qb->andWhere($expr->neq($pf . 'status', $qb->createNamedParameter(Index::INDEX_OK)));
253
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
254
+		$qb->andWhere($expr->neq($pf.'status', $qb->createNamedParameter(Index::INDEX_OK)));
255 255
 	}
256 256
 
257 257
 }
Please login to merge, or discard this patch.