nextcloud / fulltextsearch

lib/Model/SearchRequest.php

Doc Comments +1 added lines, -1 removed lines

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

lib/Service/CliService.php

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/Service/IndexService.php

Doc Comments +2 added lines, -2 removed lines

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

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,

lib/Service/TestService.php

Doc Comments +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
 
 
 	/**
-	 * @param IIndexOptions $options
+	 * @param \OCA\FullTextSearch\Model\IndexOptions $options
 	 *
 	 * @return IndexDocument
 	 */
@@ -103,7 +103,7 @@ 
 
 
 	/**
-	 * @param IIndexOptions $options
+	 * @param \OCA\FullTextSearch\Model\IndexOptions $options
 	 *
 	 * @return IndexDocument
 	 */

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
 	public function generateIndexDocumentContentLicense(IIndexOptions $options): IndexDocument {
 		$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) {
@@ -140,7 +140,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/Test.php

Spacing +12 added lines, -12 removed lines

@@ -215,7 +215,7 @@ 
 			$output->write(' ', true);
 		}
 
-		$output->write($line . ' ', false);
+		$output->write($line.' ', false);
 	}
 
 
@@ -276,7 +276,7 @@ 
 		$testProvider->setIndexOptions(new IndexOptions());
 		$indexableDocuments =
 			$testProvider->generateIndexableDocuments(TestService::DOCUMENT_USER1);
-		$this->output($output, '(' . sizeof($indexableDocuments) . ' items)', false);
+		$this->output($output, '('.sizeof($indexableDocuments).' items)', false);
 		$this->outputResult($output, true);
 	}
 
@@ -292,12 +292,12 @@ 
 		$wrapper = $this->platformService->getPlatform();
 		$testPlatform = $wrapper->getPlatform();
 
-		$this->output($output, '(' . $testPlatform->getName() . ')', false);
+		$this->output($output, '('.$testPlatform->getName().')', false);
 		$this->outputResult($output, true);
 
 		$this->output($output, 'Testing search platform.');
 		if (!$testPlatform->testPlatform()) {
-			throw new Exception ('Search platform (' . $testPlatform->getName() . ') down ?');
+			throw new Exception('Search platform ('.$testPlatform->getName().') down ?');
 		}
 		$this->outputResult($output, true);
 
@@ -393,13 +393,13 @@ 
 			);
 
 			$this->output(
-				$output, '(size: ' . $indexDocument->getContentSize() . ')', false
+				$output, '(size: '.$indexDocument->getContentSize().')', false
 			);
 			$this->outputResult($output, true);
 		} catch (Exception $e) {
 			throw new Exception(
-				"Issue while getting test document '" . TestService::DOCUMENT_TYPE_LICENSE
-				. "' from search platform: " . $e->getMessage()
+				"Issue while getting test document '".TestService::DOCUMENT_TYPE_LICENSE
+				. "' from search platform: ".$e->getMessage()
 			);
 		}
 
@@ -569,7 +569,7 @@ 
 	) {
 		$this->output(
 			$output,
-			" - '" . $search . "'" . (($moreOutput === '') ? '' : ' - ' . $moreOutput . ' - ')
+			" - '".$search."'".(($moreOutput === '') ? '' : ' - '.$moreOutput.' - ')
 		);
 		$request = new SearchRequest();
 
@@ -583,7 +583,7 @@ 
 
 		$this->output(
 			$output,
-			'(result: ' . $searchResult->getCount() . ', expected: ' . json_encode($expected) . ')',
+			'(result: '.$searchResult->getCount().', expected: '.json_encode($expected).')',
 			false
 		);
 		$this->compareSearchResult($searchResult, $expected);
@@ -647,12 +647,12 @@ 
 	private function compareSearchResult(SearchResult $searchResult, array $entries) {
 		$documents = $searchResult->getDocuments();
 		if (sizeof($documents) !== sizeof($entries)) {
-			throw new \Exception('Unexpected SearchResult: ' . json_encode($searchResult));
+			throw new \Exception('Unexpected SearchResult: '.json_encode($searchResult));
 		}
 
 		foreach ($documents as $document) {
 			if (!in_array($document->getId(), $entries)) {
-				throw new \Exception('Unexpected Document: ' . json_encode($document));
+				throw new \Exception('Unexpected Document: '.json_encode($document));
 			}
 		}
 	}
@@ -665,7 +665,7 @@ 
 	 * @throws InterruptException
 	 */
 	private function pause(OutputInterface $output, int $s) {
-		$this->output($output, 'Pausing ' . $s . ' seconds');
+		$this->output($output, 'Pausing '.$s.' seconds');
 
 		for ($i = 1; $i <= $s; $i++) {
 			if (time_nanosleep(1, 0) !== true) {

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 IndexDocument[] $result */
 		$result = $searchResult->getDocuments();
 		foreach ($result as $document) {
-			echo ' - ' . $document->getId() . ' score:' . $document->getScore() . "\n";
+			echo ' - '.$document->getId().' score:'.$document->getScore()."\n";
 		}
 	}
 

lib/Db/TickRequestBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -108,10 +108,10 @@
 	 * @return Tick
 	 */
 	protected function parseTickSelectSql(array $data): Tick {
-		$tick = new Tick($data['source'], (int) $data['id']);
+		$tick = new Tick($data['source'], (int)$data['id']);
 		$tick->setData(json_decode($data['data'], true))
-			 ->setTick((int) $data['tick'])
-			 ->setFirstTick((int) $data['first_tick'])
+			 ->setTick((int)$data['tick'])
+			 ->setFirstTick((int)$data['first_tick'])
 			 ->setStatus($data['status'])
 			 ->setAction($data['action']);
 

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/Cron/Index.php

Spacing +1 added lines, -1 removed lines

@@ -163,6 +163,6 @@
 	 *
 	 */
 	private function setLastErrReset() {
-		$this->configService->setAppValue(ConfigService::CRON_LAST_ERR_RESET, (string) time());
+		$this->configService->setAppValue(ConfigService::CRON_LAST_ERR_RESET, (string)time());
 	}
 }