nextcloud / fulltextsearch

lib/Model/Runner.php

Doc Comments +9 added lines, -4 removed lines

@@ -214,6 +214,11 @@ 
 		$this->infoUpdated();
 	}
 
+	/**
+	 * @param string $info
+	 * @param string $value
+	 * @param string $colored
+	 */
 	private function setInfoColored($info, $value, $colored) {
 		if ($colored === '') {
 			return;
@@ -257,7 +262,7 @@ 
 	}
 
 	/**
-	 * @param $key
+	 * @param string $key
 	 */
 	public function keyPressed($key) {
 		foreach ($this->methodOnKeyPress as $method) {
@@ -339,7 +344,7 @@ 
 
 
 	/**
-	 * @param $tick
+	 * @param integer $tick
 	 */
 	private function updateTick($tick) {
 		if (($this->ramTick + self::INFO_UPDATE) > $tick) {
@@ -353,8 +358,8 @@ 
 
 	/**
 	 * @deprecated - verifier l'interet !?
-	 * @param $reason
-	 * @param $stop
+	 * @param string $reason
+	 * @param boolean $stop
 	 */
 	public function exception($reason, $stop) {
 		if (!$stop) {

Spacing +3 added lines, -3 removed lines

@@ -235,7 +235,7 @@ 
 		}
 
 		if ($color !== '') {
-			$this->info[$info . 'Colored'] = '<' . $color . '>' . $value . '</' . $color . '>';
+			$this->info[$info.'Colored'] = '<'.$color.'>'.$value.'</'.$color.'>';
 		}
 
 
@@ -346,7 +346,7 @@ 
 			return;
 		}
 
-		$this->setInfo('_memory', round((memory_get_usage() / 1024 / 1024)) . ' MB');
+		$this->setInfo('_memory', round((memory_get_usage() / 1024 / 1024)).' MB');
 		$this->ramTick = $tick;
 	}
 
@@ -358,7 +358,7 @@ 
 	 */
 	public function exception($reason, $stop) {
 		if (!$stop) {
-			$this->output('Exception: ' . $reason);
+			$this->output('Exception: '.$reason);
 			// TODO: feed an array of exceptions for log;
 		}
 		$this->runningService->exception($this->tickId, $reason, $stop);

lib/Service/CliService.php

Doc Comments +1 added lines, -1 removed lines

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

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();
 	}

lib/Service/IndexService.php

Doc Comments +1 added lines, -1 removed lines

@@ -364,7 +364,7 @@
 	 * @param IndexDocument $document
 	 *
 	 * @return
-	 * @throws Exception
+	 Index @throws Exception
 	 */
 	public function indexDocument(
 		IFullTextSearchPlatform $platform, IFullTextSearchProvider $provider, $document

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 	public function indexProviderContentFromUser(
 		IFullTextSearchPlatform $platform, IFullTextSearchProvider $provider, $userId, $options
 	) {
-		$this->updateRunnerAction('generateIndex' . $provider->getName());
+		$this->updateRunnerAction('generateIndex'.$provider->getName());
 		$this->updateRunnerInfoArray(
 			[
 				'userId'       => $userId,

lib/Command/Test.php

Spacing +11 added lines, -11 removed lines

@@ -203,7 +203,7 @@ 
 			$output->write(' ', true);
 		}
 
-		$output->write($line . ' ', false);
+		$output->write($line.' ', false);
 	}
 
 
@@ -253,7 +253,7 @@ 
 		$this->output($output, 'Testing mocked provider: get indexable documents.');
 		$indexableDocuments =
 			$testProvider->generateIndexableDocuments(TestService::DOCUMENT_USER1);
-		$this->output($output, '(' . sizeof($indexableDocuments) . ' items)', false);
+		$this->output($output, '('.sizeof($indexableDocuments).' items)', false);
 		$this->output($output, true);
 	}
 
@@ -267,7 +267,7 @@ 
 	private function testLoadingPlatform($output) {
 		$this->output($output, 'Loading search platform.');
 		$testPlatform = $this->platformService->getPlatform();
-		$this->output($output, '(' . $testPlatform->getName() . ')', false);
+		$this->output($output, '('.$testPlatform->getName().')', false);
 		$this->output($output, true);
 
 		$this->output($output, 'Testing search platform.');
@@ -357,13 +357,13 @@ 
 			);
 
 			$this->output(
-				$output, '(size: ' . $indexDocument->getContentSize() . ')', false
+				$output, '(size: '.$indexDocument->getContentSize().')', false
 			);
 			$this->output($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()
 			);
 		}
 
@@ -536,7 +536,7 @@ 
 	) {
 		$this->output(
 			$output,
-			" - '" . $search . "'" . (($moreOutput === '') ? '' : ' - ' . $moreOutput . ' - ')
+			" - '".$search."'".(($moreOutput === '') ? '' : ' - '.$moreOutput.' - ')
 		);
 		$request = new SearchRequest();
 
@@ -544,7 +544,7 @@ 
 		$searchResult = $testPlatform->searchDocuments($testProvider, $access, $request);
 		$this->output(
 			$output,
-			'(result: ' . $searchResult->getCount() . ', expected: ' . json_encode($expected) . ')',
+			'(result: '.$searchResult->getCount().', expected: '.json_encode($expected).')',
 			false
 		);
 		$this->compareSearchResult($searchResult, $expected);
@@ -608,12 +608,12 @@ 
 	private function compareSearchResult(SearchResult $searchResult, $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));
 			}
 		}
 	}
@@ -626,7 +626,7 @@ 
 	 * @throws InterruptException
 	 */
 	private function pause(OutputInterface $output, $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/Live.php

Spacing +1 added lines, -1 removed lines

@@ -448,7 +448,7 @@
 		$index = $error['index'];
 		$errorIndex = '';
 		if ($index !== null) {
-			$errorIndex = $index->getProviderId() . ':' . $index->getDocumentId();
+			$errorIndex = $index->getProviderId().':'.$index->getDocumentId();
 		}
 
 		$this->runner->setInfoArray(

lib/Db/CoreRequestBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -181,8 +181,8 @@ 
 	 */
 	private function limitToDBField(IQueryBuilder &$qb, $field, $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];
@@ -202,8 +202,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/Command/Index.php

Spacing +1 added lines, -1 removed lines

@@ -448,7 +448,7 @@
 		$index = $error['index'];
 		$errorIndex = '';
 		if ($index !== null) {
-			$errorIndex = $index->getProviderId() . ':' . $index->getDocumentId();
+			$errorIndex = $index->getProviderId().':'.$index->getDocumentId();
 		}
 
 		$this->runner->setInfoArray(