nextcloud / fulltextsearch

lib/Service/SearchService.php

Spacing +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
 			try {
 				$rights->setCircles(Circles::joinedCircleIds($user->getUID()));
 			} catch (Exception $e) {
-				$this->miscService->log('Circles is set as enabled but: ' . $e->getMessage());
+				$this->miscService->log('Circles is set as enabled but: '.$e->getMessage());
 			}
 		}
 

lib/Service/CliService.php

Spacing +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
 			}
 		}
 
-		$this->display->setFormat(implode("\n", $format) . "\n");
+		$this->display->setFormat(implode("\n", $format)."\n");
 		$this->refreshInfo();
 		$this->display->start();
 	}

Doc Comments +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 
 	/**
 	 * @param string $panelId
-	 * @param array $lines
+	 * @param string[] $lines
 	 */
 	public function createPanel(string $panelId, array $lines) {
 		if (!is_array($lines)) {

lib/Model/SearchRequest.php

Doc Comments +1 added lines, -1 removed lines

@@ -631,7 +631,7 @@
 	}
 
 	/**
-	 * @param array $filters
+	 * @param string[] $filters
 	 *
 	 * @return ISearchRequest
 	 */

Spacing +4 added lines, -4 removed lines

@@ -169,7 +169,7 @@ 
 	 * @return ISearchRequest
 	 */
 	public function addSearch(string $search): ISearchRequest {
-		$this->search .= ' ' . $search;
+		$this->search .= ' '.$search;
 
 		return $this;
 	}
@@ -211,7 +211,7 @@ 
 
 		$options = ['is', 'show'];
 		if (in_array($kw, $options)) {
-			$this->addOption($kw . '_' . $value, '1');
+			$this->addOption($kw.'_'.$value, '1');
 
 			return true;
 		}
@@ -226,7 +226,7 @@ 
 		$valuedSubOptions = ['and'];
 		if (in_array($kw, $valuedSubOptions)) {
 			list($key, $value) = explode(':', $value, 2);
-			$this->addMultipleOption($kw . ':' . $key, $value);
+			$this->addMultipleOption($kw.':'.$key, $value);
 
 			return true;
 		}
@@ -538,7 +538,7 @@ 
 		foreach ($ak as $source) {
 			$tags = $this->subTags[$source];
 			foreach ($tags as $tag) {
-				$subTags[] = $source . '_' . $tag;
+				$subTags[] = $source.'_'.$tag;
 			}
 		}
 

lib/Db/CoreRequestBuilder.php

Spacing +8 added lines, -8 removed lines

@@ -191,8 +191,8 @@ 
 	 */
 	private function limitToDBField(IQueryBuilder &$qb, string $field, string $value) {
 		$expr = $qb->expr();
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
-		$field = $pf . $field;
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
+		$field = $pf.$field;
 
 		$qb->andWhere($expr->eq($field, $qb->createNamedParameter($value)));
 	}
@@ -204,8 +204,8 @@ 
 	 */
 	private function limitToDBFieldInt(IQueryBuilder &$qb, string $field, int $value) {
 		$expr = $qb->expr();
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
-		$field = $pf . $field;
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
+		$field = $pf.$field;
 
 		$qb->andWhere($expr->eq($field, $qb->createNamedParameter($value)));
 	}
@@ -218,8 +218,8 @@ 
 	 */
 	private function limitToDBFieldArray(IQueryBuilder &$qb, string $field, array $values) {
 		$expr = $qb->expr();
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
-		$field = $pf . $field;
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
+		$field = $pf.$field;
 
 		if (!is_array($values)) {
 			$values = [$values];
@@ -239,8 +239,8 @@ 
 	 */
 	protected function limitToQueuedIndexes(IQueryBuilder &$qb) {
 		$expr = $qb->expr();
-		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias . '.' : '';
-		$qb->andWhere($expr->neq($pf . 'status', $qb->createNamedParameter(Index::INDEX_OK)));
+		$pf = ($qb->getType() === QueryBuilder::SELECT) ? $this->defaultSelectAlias.'.' : '';
+		$qb->andWhere($expr->neq($pf.'status', $qb->createNamedParameter(Index::INDEX_OK)));
 	}
 
 }

lib/Service/IndexService.php

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
 		IFullTextSearchPlatform $platform, IFullTextSearchProvider $provider, string $userId,
 		IndexOptions $options
 	) {
-		$this->updateRunnerAction('generateIndex' . $provider->getName());
+		$this->updateRunnerAction('generateIndex'.$provider->getName());
 		$this->updateRunnerInfoArray(
 			[
 				'userId'          => $userId,

Doc Comments +2 added lines, -2 removed lines

@@ -210,7 +210,7 @@ 
 	/**
 	 * @param IFullTextSearchProvider $provider
 	 * @param IIndexDocument[] $documents
-	 * @param IIndexOptions $options
+	 * @param IndexOptions $options
 	 *
 	 * @return IIndexDocument[]
 	 * @throws Exception
@@ -491,7 +491,7 @@ 
 
 
 	/**
-	 * @param IIndex $index
+	 * @param Index $index
 	 */
 	private function updateIndexError(IIndex $index) {
 

lib/Service/TestService.php

Doc Comments +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 
 
 	/**
-	 * @param IIndexOptions $options
+	 * @param \OCA\FullTextSearch\Model\IndexOptions $options
 	 *
 	 * @return IIndexDocument
 	 */
@@ -104,7 +104,7 @@ 
 
 
 	/**
-	 * @param IIndexOptions $options
+	 * @param \OCA\FullTextSearch\Model\IndexOptions $options
 	 *
 	 * @return IIndexDocument
 	 */

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 	public function generateIndexDocumentContentLicense(IIndexOptions $options): IIndexDocument {
 		$indexDocument = $this->generateIndexDocument(self::DOCUMENT_TYPE_LICENSE);
 
-		$content = file_get_contents(__DIR__ . '/../../LICENSE');
+		$content = file_get_contents(__DIR__.'/../../LICENSE');
 		$indexDocument->setContent($content);
 
 		if ($options === null) {
@@ -142,7 +142,7 @@ 
 			$orig = call_user_func([$origIndex, $method]);
 			$compare = call_user_func([$compareIndex, $method]);
 			if ($orig !== $compare) {
-				throw new Exception($method . '() orig:' . $orig . ' compare:' . $compare);
+				throw new Exception($method.'() orig:'.$orig.' compare:'.$compare);
 			}
 		}
 	}

lib/Command/Search.php

Spacing +3 added lines, -3 removed lines

@@ -123,13 +123,13 @@
 	 */
 	private function displaySearchResult(ISearchResult $searchResult) {
 		/** @var SearchResult $searchResult */
-		echo '> ' . $searchResult->getProvider()
-								 ->getName() . "\n";
+		echo '> '.$searchResult->getProvider()
+								 ->getName()."\n";
 
 		/** @var IIndexDocument[] $result */
 		$result = $searchResult->getDocuments();
 		foreach ($result as $document) {
-			echo ' - ' . $document->getId() . ' score:' . $document->getScore() . "\n";
+			echo ' - '.$document->getId().' score:'.$document->getScore()."\n";
 		}
 	}
 

lib/Command/DocumentProvider.php

Spacing +3 added lines, -3 removed lines

@@ -126,12 +126,12 @@
 		$output->writeln(substr($content, 0, 80));
 
 		$parts = $indexDocument->getParts();
-		$output->writeln(sizeof($parts) . ' Part(s)');
+		$output->writeln(sizeof($parts).' Part(s)');
 		foreach (array_keys($parts) as $part) {
 			$output->writeln(
-				"'" . $part . "' " . substr($parts[$part], 0, 80) . '   (size: ' . strlen(
+				"'".$part."' ".substr($parts[$part], 0, 80).'   (size: '.strlen(
 					$parts[$part]
-				) . ')'
+				).')'
 			);
 		}
 

lib/Command/Index.php

Spacing +3 added lines, -3 removed lines

@@ -221,7 +221,7 @@ 
 				}
 				);
 			} catch (Throwable $t) {
-				$this->miscService->log($t->getMessage() . ' -- ' . $t->getTraceAsString());
+				$this->miscService->log($t->getMessage().' -- '.$t->getTraceAsString());
 				throw new Exception('Please install php-readline, or use --no-readline');
 			}
 		}
@@ -637,7 +637,7 @@ 
 		$index = $error['index'];
 		$errorIndex = '';
 		if ($index !== null) {
-			$errorIndex = $index->getProviderId() . ':' . $index->getDocumentId();
+			$errorIndex = $index->getProviderId().':'.$index->getDocumentId();
 		}
 
 		$width = $this->terminal->getWidth() - 13;
@@ -689,7 +689,7 @@ 
 		$index = $result['index'];
 		$resultIndex = '';
 		if ($index !== null) {
-			$resultIndex = $index->getProviderId() . ':' . $index->getDocumentId();
+			$resultIndex = $index->getProviderId().':'.$index->getDocumentId();
 		}